[MacPorts] #43339: Update arb to 6.0
MacPorts
noreply at macports.org
Sun Apr 13 02:06:33 PDT 2014
#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 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.
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.
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
}}}
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.
--
Ticket URL: <https://trac.macports.org/ticket/43339#comment:1>
MacPorts <http://www.macports.org/>
Ports system for OS X
More information about the macports-tickets
mailing list