[MacPorts] #40658: flac @1.2.1 upgrade to 1.3.0
#40658: flac @1.2.1 upgrade to 1.3.0 ----------------------------+-------------------------------- Reporter: james.george@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Keywords: haspatch | Port: flac ----------------------------+-------------------------------- I have created a patch to the flac @1.2.1 port to retrieve and install flac 1.3.0 from its new home at xiph.org. One of the patches from the old 1.2.1 port no longer applies and didn't seem relevant so I've removed it from the patchfiles list, though it still exists, commented out in the diff. Built and tested locally and it worked for me - 10.8.5 XCode5 MBP. Also built the xmms_plugin variant, though didn't test it. FLAC 1.3.0 doesn't bring much new other than a few bug fixes and improved handling of high bit rate files. Replay gain processing for 24bit files being what I was after. :-) -- Ticket URL: <https://trac.macports.org/ticket/40658> MacPorts <http://www.macports.org/> Ports system for OS X
#40658: flac @1.2.1 upgrade to 1.3.0 -----------------------------+-------------------------- Reporter: james.george@… | Owner: ryandesign@… Type: update | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: | Keywords: haspatch Port: flac | -----------------------------+-------------------------- Changes (by ryandesign@…): * status: new => assigned * owner: macports-tickets@… => ryandesign@… * cc: nomaintainer@… (removed) Comment: Thanks. -- Ticket URL: <https://trac.macports.org/ticket/40658#comment:1> MacPorts <http://www.macports.org/> Ports system for OS X
#40658: flac @1.2.1 upgrade to 1.3.0 -----------------------------+-------------------------- Reporter: james.george@… | Owner: ryandesign@… Type: update | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: | Keywords: haspatch Port: flac | -----------------------------+-------------------------- Comment (by ryandesign@…): Replying to [ticket:40658 james.george@…]:
One of the patches from the old 1.2.1 port no longer applies and didn't seem relevant so I've removed it from the patchfiles list, though it still exists, commented out in the diff.
In Tcl, a backslash at the end of a line extends a comment to the next line. So you've in fact commented out ''all'' the patches, not just the one. I'll go through and check what all the patches were doing and see if they're still useful. Also note that you can more simply use an xz distfile with the statement "`use_xz yes`". -- Ticket URL: <https://trac.macports.org/ticket/40658#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X
#40658: flac @1.2.1 upgrade to 1.3.0 -----------------------------+-------------------------- Reporter: james.george@… | Owner: ryandesign@… Type: update | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: | Keywords: haspatch Port: flac | -----------------------------+-------------------------- Comment (by james.george@…): Replying to [comment:2 ryandesign@…]:
In Tcl, a backslash at the end of a line extends a comment to the next line. So you've in fact commented out ''all'' the patches, not just the one. I'll go through and check what all the patches were doing and see if they're still useful.
Also note that you can more simply use an xz distfile with the statement "`use_xz yes`".
Really? Even in a comment line? That would explain the really odd error that I saw when I commented it out in-place. I shall have to keep that in mind. I missed use_xz when I read the Guide. Do you want me to rework the patch? Cheers! -- Ticket URL: <https://trac.macports.org/ticket/40658#comment:3> MacPorts <http://www.macports.org/> Ports system for OS X
#40658: flac @1.2.1 upgrade to 1.3.0 -----------------------------+-------------------------- Reporter: james.george@… | Owner: ryandesign@… Type: update | Status: assigned Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: | Keywords: haspatch Port: flac | -----------------------------+-------------------------- Comment (by ryandesign@…): Replying to [comment:3 james.george@…]:
Replying to [comment:2 ryandesign@…]:
In Tcl, a backslash at the end of a line extends a comment to the next line. So you've in fact commented out ''all'' the patches, not just the one. I'll go through and check what all the patches were doing and see if they're still useful.
Really? Even in a comment line?
Really! Removing that line shows that at least one of the other patches doesn't even apply.
Do you want me to rework the patch?
That's ok, I'll work on it. I just also want to check all the other things the portfile already does because I'm not sure if they're still useful or correct. -- Ticket URL: <https://trac.macports.org/ticket/40658#comment:4> MacPorts <http://www.macports.org/> Ports system for OS X
#40658: flac @1.2.1 upgrade to 1.3.0 -----------------------------+-------------------------- Reporter: james.george@… | Owner: ryandesign@… Type: update | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.2.0 Resolution: fixed | Keywords: haspatch Port: flac | -----------------------------+-------------------------- Changes (by ryandesign@…): * status: assigned => closed * resolution: => fixed Comment: Committed with changes in r111885. -- Ticket URL: <https://trac.macports.org/ticket/40658#comment:5> MacPorts <http://www.macports.org/> Ports system for OS X
participants (1)
-
MacPorts