guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] gnu: Add perl-db_file.


From: Andreas Enge
Subject: Re: [PATCH] gnu: Add perl-db_file.
Date: Tue, 26 Jul 2016 22:48:37 +0200
User-agent: Mutt/1.6.1 (2016-04-27)

Hello,

On Tue, Jul 26, 2016 at 09:01:38PM +0200, Danny Milosavljevic wrote:
> Yeah, I thought about it but decided against it - there are very few options 
> in that file, substitute* can't substitute entire lines (or only at the 
> beginning of the line) or entire words (so it's not safe), the user is 
> supposed to set PREFIX and HASH (it's just a coincidence we didn't have to 
> change them) and if we did that then new versions of the package could sneak 
> in new options we wouldn't notice but we should have changed. Better for it 
> to fail instead of silently doing something strange.

well, substitute* is closer to a diff - it makes it easy to see what actually
changes. And I would make the converse argument of you - for the next version,
if more things change (an option is added to the file, for instance), then
the substitute* still has a chance to get a working package. Imagine that
someone without knowledge of the package might be the one to update it, for
instance.

I pushed with tiny changes to the description.

Thanks!

Andreas




reply via email to

[Prev in Thread] Current Thread [Next in Thread]