[MacPorts] #16589: DEFECT: ffmpeg -version fails to report correct svn revision
#16589: DEFECT: ffmpeg -version fails to report correct svn revision --------------------------------+------------------------------------------- Reporter: db.evans@gmail.com | Owner: macports-tickets@lists.macosforge.org Type: defect | Status: new Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.7.0 Keywords: ffmpeg | Port: ffmpeg --------------------------------+------------------------------------------- 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. }}} -- Ticket URL: <http://trac.macports.org/ticket/16589> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16589: DEFECT: ffmpeg -version fails to report correct svn revision ---------------------------------+------------------------------------------ Reporter: db.evans@gmail.com | Owner: macsforever2000@macports.org Type: defect | Status: assigned Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.7.0 Resolution: | Keywords: ffmpeg Port: ffmpeg | ---------------------------------+------------------------------------------ Changes (by macsforever2000@macports.org): * owner: macports-tickets@lists.macosforge.org => macsforever2000@macports.org * status: new => assigned -- Ticket URL: <http://trac.macports.org/ticket/16589#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16589: DEFECT: ffmpeg -version fails to report correct svn revision ---------------------------------+------------------------------------------ Reporter: db.evans@gmail.com | Owner: macsforever2000@macports.org Type: defect | Status: closed Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.7.0 Resolution: fixed | Keywords: ffmpeg Port: ffmpeg | ---------------------------------+------------------------------------------ Changes (by macsforever2000@macports.org): * status: assigned => closed * resolution: => fixed Comment: Committed revision r40473. Thanks and sorry for the delay! -- Ticket URL: <http://trac.macports.org/ticket/16589#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16589: DEFECT: ffmpeg -version fails to report correct svn revision ---------------------------------+------------------------------------------ Reporter: db.evans@gmail.com | Owner: macsforever2000@macports.org Type: defect | Status: closed Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.7.0 Resolution: fixed | Keywords: ffmpeg Port: ffmpeg | ---------------------------------+------------------------------------------ Comment(by macsforever2000@macports.org): 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! -- Ticket URL: <http://trac.macports.org/ticket/16589#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#16589: DEFECT: ffmpeg -version fails to report correct svn revision ---------------------------------+------------------------------------------ Reporter: db.evans@gmail.com | Owner: macsforever2000@macports.org Type: defect | Status: closed Priority: Normal | Milestone: Port Bugs Component: ports | Version: 1.7.0 Resolution: fixed | Keywords: ffmpeg Port: ffmpeg | ---------------------------------+------------------------------------------ Comment(by ryandesign@macports.org): Replying to [comment:3 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. -- Ticket URL: <http://trac.macports.org/ticket/16589#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts