guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] guix package: Add '--delete-generations'.


From: Nikita Karetnikov
Subject: Re: [PATCH] guix package: Add '--delete-generations'.
Date: Tue, 24 Sep 2013 11:21:29 +0400

>>>> I had noticed that ‘--roll-back’ doesn’t output anything with
>>>> ‘--dry-run’, so I implemented ‘--delete-generations’ similarly.  Maybe
>>>> it would be better to print something.  WDYT?

>>> Agreed (in a separate patch.)

I don’t think that I’ll have time for this before the release.  I’d
like to be able to test the MIPS binaries.

>>> “Delete the generations matching @var{patterns} or ... when omitted.”

>>> Or what actually?  I would expect it to delete all the generations but
>>> the current one when PATTERN is omitted, right?

OK.  Should it delete the current generation when it’s specified
explicitly, e.g., ‘--delete-generations=5’?  What about other patterns?

>>>> +(define (link-to-empty-environment generation)
>>>> +  "Link GENERATION, a string, to the empty environment."

>>> s/environment/profile/ maybe?  (I know there are other inconsistencies
>>> in these files.)

OK.

>>> Ideally this factorization would go in a patch of its own, before the
>>> one that adds --delete-generations.  Is that doable for you?

>> I think so.

Sorry, I’m not sure what have to be changed.  (I’ve just looked
through the module.)  Do you have time to work on this?

>>>> +  (define (apply-to-generations function profile pattern)

>>> s/function/proc/ to follow the convention.

OK.

Attachment: pgpNOv7Vqj0Bo.pgp
Description: PGP signature


reply via email to

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