[144726] trunk/dports/science
Clemens Lang
cal at macports.org
Sat Jan 16 13:01:15 PST 2016
Hi Ryan,
On Sat, Jan 16, 2016 at 09:25:15AM -0600, Ryan Schmidt wrote:
> Instead of overriding the fetch, checksum and extract phases, you
> should declare that the port has no distfiles, by writing on a line by
> itself:
>
> distfiles
>
> With no distfiles, the master_sites line isn't needed either. ("port
> lint" will complain about missing master_sites, but that's a bug in
> "port lint".)
>
> > +test {}
>
> The test phase is disabled by default. There is no need to override it
> with an empty block.
[...]
> Why construct this file in the build phase? Why not just put the real
> file in the files directory and copy it from there? Wouldn't that be
> much more straightforward?
I've fixed these comments in https://trac.macports.org/changeset/144734.
> The whitespace of this portfile does not conform to what the modeline
> says the whitespace is. (Indents should be at multiples of 4 spaces,
> not 2.)
Formatted to conform to the modeline in
https://trac.macports.org/changeset/144735.
--
Clemens
More information about the macports-dev
mailing list