cvs commit: ports/archivers/p5-Archive-Any pkg-plist ports/archivers/p5-Archive-SimpleExtractor pkg-plist ports/archivers/p5-Compress-LZF pkg-plist ports/archivers/p5-Compress-LZO pkg-plist ports/archivers/p5-Compress-LZW pkg-plist ports/archiver

[Stanislav Sedov]

On Fri, 24 Sep 2010 00:27:28 -0700
Doug Barton <dougb at FreeBSD.org> mentioned:

> 
> I think Philip described the standard very well in the portion of the 
> log that is quoted above. Personally, I've never seen or used such a 
> comment in any FreeBSD port, and can't imagine any reason why someone 
> would want to.

I disagree.  The only thing he described is that he preferred one way
over another as the standard one, and I have not seen any comments
about who it was discussed with.  Looks like it was one man decision
from 3rd party person perspective.  At a bare minimum there should be
a list of person who took part in the discussion, and what was the
conclusion, because the commit itself, as was noted by osa@, has no
evident benefit.

> It made the ports tree a little smaller? And personally I think more 
> standardization is a good thing in areas that have no material effect on 
> the port, or the maintainer's creativity.

If there going to be a standard, it should be documented first.  From my
POV I can see a benefit of having a @comment in pkg-plist, as it allows
one to find out what revision this file was.  I'm not sure that it was
the original intention, though.

Deciding what's standard and what's not only by use number is not gonna
work.  I'm pretty sure that most ports doesn't fully respect PREFIX or
CFLAGS, but it's not a reason to drop it from ports that do.  Or changing
all ports with dynamic pkg-lists to use static one just because most
ports do this.

-- 
Stanislav Sedov
ST4096-RIPE
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-ports/attachments/20100924/9af71747/attachment.pgp