DEFECT: ffmpeg -version fails to report correct svn revision

[db.evans@…]

Building using the existing Portfile, the command ffmpeg -version yields

FFmpeg version UNKNOWN, Copyright (c) 2000-2008 Fabrice Bellard, et al.

The ffmpeg makefiles depend on the existence of svn metadata to determine the correct svn revision but the current Portfile fetches the distribution using svn export resulting in no metadata.

The attached patch to the Portfile uses svn co in the fetch phase to insure that the correct svn metadata is included. In addition, a pre-fetch phase is included to forcably delete a pre-existing distribution, prior to the fetch phase, if it does not contain metadata. This ensures that an existing installation will correctly upgrade in the presence of a pre-existing svn distribution without metadata.

After building with the modified Portfile, ffmpeg -version yields

FFmpeg version SVN-r14381, Copyright (c) 2000-2008 Fabrice Bellard, et al.

[db.evans@…]

Patch for Portfile

[macsforever2000@…]

Committed revision r40473. Thanks and sorry for the delay!

[macsforever2000@…]

ryandesign at macports.org had this comment about the patch: "But this just forces the working copy to be discarded and checked out again every time you try to install. I thought the point of having a working copy was so that it could be reused. It would be nice if the working copy were only deleted in the pre-fetch phase if the revision number of the working copy (see the "svnversion" command) doesn't match the revision the port wants to check out."

If you provide an updated patch, I will commit it. Thanks!

[ryandesign@…]

Replying to macsforever2000@…:

ryandesign at macports.org had this comment about the patch: "But this just forces the working copy to be discarded and checked out again every time you try to install. I thought the point of having a working copy was so that it could be reused. It would be nice if the working copy were only deleted in the pre-fetch phase if the revision number of the working copy (see the "svnversion" command) doesn't match the revision the port wants to check out." If you provide an updated patch, I will commit it. Thanks!

Nevermind, please ignore me. I misread the patch. It's fine as it is. I see now that it only deletes the distpath directory if it is an export, not a working copy.

[anonymous]

Milestone Port Bugs deleted