New Port: Slrn-devel
Juan Manuel Palacios
jmpp at macports.org
Sat Feb 17 17:32:56 PST 2007
On Feb 16, 2007, at 12:45 PM, Sbranzo wrote:
> Just a head-up for a new port submitted a while ago:
> http://trac.macports.org/projects/macports/ticket/11299
>
> Gufo
From my comment in the ticket:
Committed in r22124, thanks for the contribution! Notes of interest:
1) "revision" defaults to zero, so no need to state it explicitly if
that's the value used.
2) Number of files should be kept to a minimum, so a patch like that is
unnecessary as users using the port are already using MacPorts, no need
to instruct them to type port install to install the software.
3) No need to pass --prefix=${prefix} to the configure script, that's
basically what MacPorts is for ;-)
4) Don't hardcode /opt/local in the Portfile, that's the default value
of ${prefix} but users are free to choose another one; Portfile should
be abstracted from explicit values as much as possible.
Other than that, great work, keep those contributions coming!
-jmpp
PS: Ticket severity should not be Crash/Data loss ;-)
More information about the macports-dev
mailing list