tech-pkg archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: pkgsrc/mk/pbulk

Jonathan Perkin <> writes:

> * On 2014-07-15 at 22:29 BST, Aleksej Saushev wrote:
>> Module Name: pkgsrc
>> Committed By:        asau
>> Date:                Tue Jul 15 21:29:39 UTC 2014
>> Added Files:
>>      pkgsrc/mk/pbulk:
>> Log Message:
>> Add helper script for simplified pbulk setup.
> Please rename it to or similar, as I clearly asked for
> in my review.  There are a number of different ways to implement pbulk
> helper scripts, and it needs to be clear what each does.
> You should also add your README as a starting point for documenting
> each approach.


Jonathan's rationale for naming seems very sensible to me.  Could you
rename it, or explain why the -simple or -single name causes problems?
(It seems we are on the cusp of getting more scripts and pbulk docs,
which is great!)


Attachment: pgpTV7n7d9LIV.pgp
Description: PGP signature

Home | Main Index | Thread Index | Old Index