Re: PR/47769 CVS commit: pkgsrc/textproc/ispell-ro

[Alistair Crooks <agc%pkgsrc.org@localhost>]

The following reply was made to PR pkg/47769; it has been noted by GNATS.

From: Alistair Crooks <agc%pkgsrc.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: rodent%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost, 
pkgsrc-bugs%NetBSD.org@localhost,
        kre%munnari.OZ.AU@localhost
Subject: Re: PR/47769 CVS commit: pkgsrc/textproc/ispell-ro
Date: Tue, 30 Apr 2013 05:45:33 +0200

 On Tue, Apr 30, 2013 at 02:10:07AM +0000, rodent%NetBSD.org@localhost wrote:
 > The following reply was made to PR pkg/47769; it has been noted by GNATS.
 > 
 > From: rodent%NetBSD.org@localhost
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: 
 > Subject: Re: PR/47769 CVS commit: pkgsrc/textproc/ispell-ro
 > Date: Mon, 29 Apr 2013 22:06:05 -0400
 > 
 >  On Tue, Apr 30, 2013 at 12:40:06AM +0000, Robert Elz wrote:
 >  >  [...]
 >  >  it is possible that
 >  >  there might be a distfile with a name that contains no version, but for
 >  >  which the chances of any modifications are vanishingly small (say a 
 > historical
 >  >  version of some source file, perhaps "original-grep.tgz" or something)
 >  >  where use of a DIST_SUBDIR might not be required.   On the other hand, 
 > when
 >  >  it is quite likely that a distfile will alter, then using a DIST_SUBDIR 
 > if
 >  >  it has no version in its name makes sense.
 >  >  
 >  >  In any case, if a package has a DIST_SUBDIR it is always wrong to remove 
 > it
 >  >  unless the distfile name changes.
 >  
 >  Please, note all of this in the documentation.
 
 I would really prefer it if, instead of requiring this, prescribing
 that, and proscribing something else, judgement was used as to the
 best way of doing things.
 
 It's much more scalable, and less stressful for all concerned.
 
 Thanks,
 Alistair