Ticket #458 (closed defect: fixed)

Opened 9 months ago

Last modified 9 months ago

Broken support for xvid.mov

Reported by: MyCometG3 Owned by:
Priority: normal Milestone:
Component: Perian-Main Version: 1.1.4
Severity: normal Keywords:
Cc:

Description

Tested svn rev 1224, and found it seems to break support for xvid.mov produced by xvidEncoder.

  • xvid.mov is MPEG-4 ASP stream - compatible with 1.1.3/1.1.4. svn 1224 seems to be incompatible.

Please investigate. (I also tried to build 1.2b1, but unable to make...)

Attachments

testsample1.zip Download (1.0 MB) - added by MyCometG3 9 months ago.
test sample movies
testsample2.zip Download (3.0 MB) - added by MyCometG3 9 months ago.
test sample movies
r1120Patch.diff Download (1.7 KB) - added by gbooker 9 months ago.

Change History

Changed 9 months ago by MyCometG3

test sample movies

Changed 9 months ago by MyCometG3

test sample movies

Changed 9 months ago by astrange

It's not that it's incompatible, but it seems Apple's broken decoder is being used for mp4v still. Reverting r1120 will fix it, I think.

(I also tried to build 1.2b1, but unable to make...)

What happened instead?

Changed 9 months ago by MyCometG3

What happened instead?

Just Missing libraries.

Four external libs (ffmpeg, libebml, libmatroska, a52codec) are not downloaded when

$ svn checkout http://svn.perian.org/branches

All branch tree would not include them. Seems to require additional steps.

Changed 9 months ago by gbooker

Reverting r1120 will do some of it (though reverting it isn't entirely correct). In addition, we have to have a bigger version number since Apple's component uses 0x00030011 now, where we use 0x00030006. These together, and we override Apple's component every time, but neither is sufficient on its own.

The following patch will do the job, without incrementing any other version numbers (not that I see any harm in doing so). This is the minimal change in the binary that makes it work.

Changed 9 months ago by gbooker

Changed 9 months ago by MyCometG3

Applied diff. It seems to be fixed. Thank you.

Changed 9 months ago by astrange

  • status changed from new to closed
  • resolution set to fixed

(In [1226]) Apply patch overriding Apple's mp4v component again.

Closes #458.

Note: See TracTickets for help on using tickets.