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

2013-09-25 Thread Nikita Karetnikov
> Frankly, I don’t like it. I think the default behavior should not be to > let the user shoot themself in the foot. > WDYT? OK, what about these patches? Can I push them? From 86d29a8b233d5cf6549fbcddf43b185086b7d178 Mon Sep 17 00:00:00 2001 From: Nikita Karetnikov Date: Wed, 25 Sep 2013 03:

Re: Generation 0

2013-09-25 Thread Nikita Karetnikov
>> There are two cases: >> >> 1. A generation does not exist. >> >> 2. The profile points to the zeroth generation, and there are no other >>generations. >> >> ‘tests/guix-package.sh’ tests the first case. Should we test the second >> case too? What would be the best way to do so? > The seco

Re: Generation 0

2013-09-25 Thread Ludovic Courtès
Nikita Karetnikov skribis: >>> What about these patches? Should the first patch also test the >>> following case? >>> >>> +(let ((numbers (generation-numbers profile))) >>> + (if (equal? numbers '(0)) >>> + (exit 1) >>> +

Re: FAIL: tests/guix-register.sh

2013-09-25 Thread Nikita Karetnikov
> Are you using a distro-provide SQLite package, or something you built > yourself? > If the problem is widespread, we can just remove this test. No need. I solved the problem by installing SQLite from Guix. pgpwaGLweKqG9.pgp Description: PGP signature

Re: Generation 0

2013-09-25 Thread Nikita Karetnikov
>> What about these patches? Should the first patch also test the >> following case? >> >> +(let ((numbers (generation-numbers profile))) >> + (if (equal? numbers '(0)) >> + (exit 1) >> + (for-each list-generation numbers))

Re: FAIL: tests/guix-register.sh

2013-09-25 Thread Ludovic Courtès
Nikita Karetnikov skribis: >> Could it be that something’s wrong with your setup? (Erroneous >> LD_LIBRARY_PATH, etc.) > > Maybe. This test also fails on a different machine of mine. > LD_LIBRARY_PATH is not set. Are you using a distro-provide SQLite package, or something you built yourself?

Re: FAIL: tests/guix-register.sh

2013-09-25 Thread Nikita Karetnikov
> Could it be that something’s wrong with your setup? (Erroneous > LD_LIBRARY_PATH, etc.) Maybe. This test also fails on a different machine of mine. LD_LIBRARY_PATH is not set. pgpMTcknFjC6O.pgp Description: PGP signature

Re: Goals for 0.4

2013-09-25 Thread Ludovic Courtès
Andreas Enge skribis: > On Thu, Aug 29, 2013 at 10:42:23PM +0200, Andreas Enge wrote: >> On Thu, Aug 29, 2013 at 02:34:27PM +0200, Ludovic Courtès wrote: >> > • Manual: improve as we see fit; notably add a section on font usage >> > for X applications. >> I would volunteer for this one. > >

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

2013-09-25 Thread Ludovic Courtès
Nikita Karetnikov skribis: >> Oh. I’d prefer if it would delete all generations but the current one. > >> That’s a fairly useful pattern: suppose you want to make space on your >> disk (and you know you won’t need to roll back), it’d be handy to run: > >> guix package --delete-generations && g

Re: [PATCH] guix package: Show which generation is the current one.

2013-09-25 Thread Ludovic Courtès
Nikita Karetnikov skribis: >> I’m suggesting to have a procedure like this: > >> (define (current-generation-number profile) >> ;; Return the current generation number of PROFILE. >> ...) > > We already have such a procedure. It’s called ‘generation-number’. Oh, OK. > What do you thi

Re: Generation 0

2013-09-25 Thread Ludovic Courtès
Nikita Karetnikov skribis: > What about these patches? Should the first patch also test the > following case? > > +(let ((numbers (generation-numbers profile))) > + (if (equal? numbers '(0)) > + (exit 1) > + (for-each lis

Re: Goals for 0.4

2013-09-25 Thread Andreas Enge
On Thu, Aug 29, 2013 at 10:42:23PM +0200, Andreas Enge wrote: > On Thu, Aug 29, 2013 at 02:34:27PM +0200, Ludovic Courtès wrote: > > • Manual: improve as we see fit; notably add a section on font usage > > for X applications. > I would volunteer for this one. The main problem that gtk+ appli