#43339: Update arb to 6.0 ------------------------------+--------------------------------- Reporter: matt.cottrell@… | Owner: macports-tickets@… Type: update | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.2.1 Resolution: | Keywords: haspatch maintainer Port: arb | ------------------------------+--------------------------------- Comment (by matt.cottrell@…): Replying to [comment:1 ryandesign@…]:
Thanks. Some remarks about your changes:
You've set the version field to 6.0, but it appears to actually be downloading version 6.0-rc1. You shouldn't deceive the user as to what version they're going to get. If this is 6.0-rc1, put 6.0-rc1 in the version field. We usually update ports to stable versions, not release candidates or other development releases. However if you as maintainer feel that 6.0-rc1 is more stable or more usable than the official latest stable release 5.5 then that's ok.
The Portfile now accurately reflects the release candidate status of this arb version. I've been in frequent contact with the upstream developers and they agree that 6.0 rc1 is a better option for users than the the old 5.5 version.
You've added a dependency on `port:glib2`. You should instead write `path:lib/pkgconfig/glib-2.0.pc:glib2` so that glib2-devel could satisfy the dependency.
Thanks for the correct syntax fort glib2.
You've added this block: {{{ # https://trac.macports.org/ticket/33756 if {${configure.compiler} == "llvm-gcc-4.2"} { configure.compiler clang } }}}
The correct way to express this is:
{{{ compiler.blacklist *llvm-gcc-4.2 }}}
Thanks for the blacklist syntax.
This will blacklist both the Xcode and MacPorts versions of llvm-gcc and
will use the next best compiler, which will most likely be a version of clang.
However, you should fix the ticket citation. #33756 is about clang not
working; it says nothing about llvm-gcc-4.2 not working (quite the opposite, in fact).
You added this check:
{{{ #Adjusting configure flags for Clang on Lion (ticket #34545) if {${configure.compiler} == "clang"} { configure.args-append -CXX_FLAGS="-Wno-reserved-user-defined-
literal -stdlib=libc++"
}}}
Note that MacPorts may select a different clang compiler, i.e. one provided by MacPorts rather than one from Xcode. To target any version of clang, you should usually use:
{{{ if {[string match *clang* ${configure.compiler}]} { }}}
However, setting `-stdlib=libc++` is suspect. Have you read wiki:FAQ#libcpp? You cite #34545, but that's about kdevplatform, not arb, and the code in question, although it was added to kdevplatform 22 months ago, was subsequently changed.
I don't know why I included that Lion check. It must have crept in from old versions on my system. A revised Portile.diff is attached. -- Ticket URL: <https://trac.macports.org/ticket/43339#comment:2> MacPorts <http://www.macports.org/> Ports system for OS X