Ticket #458 (closed defect: fixed)

Opened 3 months ago

Last modified 3 months ago

Broken support for xvid.mov

Reported by: MyCometG3 Assigned to:
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 (1.0 MB) - added by MyCometG3 on 12/02/09 08:25:21.
test sample movies
testsample2.zip (3.0 MB) - added by MyCometG3 on 12/02/09 08:27:07.
test sample movies
r1120Patch.diff (1.7 kB) - added by gbooker on 12/02/09 18:42:26.

Change History

12/02/09 08:25:21 changed by MyCometG3

  • attachment testsample1.zip added.

test sample movies

12/02/09 08:27:07 changed by MyCometG3

  • attachment testsample2.zip added.

test sample movies

12/02/09 10:51:11 changed 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?

12/02/09 17:02:53 changed 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.

12/02/09 18:42:01 changed 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.

12/02/09 18:42:26 changed by gbooker

  • attachment r1120Patch.diff added.

12/02/09 20:06:11 changed by MyCometG3

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

12/14/09 21:11:09 changed by astrange

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

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

Closes #458.