GNU bug report logs - #38528
Add test for `guix pull'

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 38528 in the body.
You can then email your comments to 38528 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: bug-guix <at> gnu.org
Subject: Add test for `guix pull'
Date: Sun, 08 Dec 2019 16:39:35 +0100

[Message part 1 (text/plain, inline)]

It would be nice to add a `guix pull' test as part of the test suite.
This was discussed in bug #38455.

The test should run `guix pull' but not apply change any symlink.

Note that `guix pull --dry-run' does not catch enough errors.

The current way to do it is to run

  guix pull --url=/path/to/checkout --profile=/tmp/guix.master

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #8 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Sun, 8 Dec 2019 21:59:41 +0100

Hi,

Thank Pierre to raise this issue.

On Sun, 8 Dec 2019 at 16:40, Pierre Neidhardt <mail <at> ambrevar.xyz> wrote:

> The current way to do it is to run
>
>   guix pull --url=/path/to/checkout --profile=/tmp/guix.master

When I put that in a test -- say tests/guix-pull.sh -- troubles
arises... The main one appears because there is no user (chroot
environment I guess). Currently, Guix looks at $USER and $LOGNAME then
goes in "default" and applies "Migrating" profile code. Note that it
appears too with non login shell as reported here [1], perhaps not
related.

[1] https://lists.gnu.org/archive/html/guix-devel/2019-12/msg00021.html

What is the best:
    - set manually the variable USER to `whoami`?
or - change to scheme code to call `whoami`? Does Guile expose that?


All the best,
simon

Message #11 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Tue, 10 Dec 2019 18:13:28 +0100

Hi,

Pierre Neidhardt <mail <at> ambrevar.xyz> skribis:

> The current way to do it is to run
>
>   guix pull --url=/path/to/checkout --profile=/tmp/guix.master

“make as-derivation” achieves more or less the same.

Now, as you know, this can be CPU-intensive, so I’m afraid we cannot
have “make check” do that systematically.

Thanks,
Ludo’.

Message #14 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Wed, 11 Dec 2019 01:11:47 +0100

[Message part 1 (text/plain, inline)]

Ludovic Courtès <ludo <at> gnu.org> writes:

> “make as-derivation” achieves more or less the same.

Is this be good enough to test guix pull?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Thu, 12 Dec 2019 14:12:14 +0100

Hi,

Pierre Neidhardt <mail <at> ambrevar.xyz> skribis:

> Ludovic Courtès <ludo <at> gnu.org> writes:
>
>> “make as-derivation” achieves more or less the same.
>
> Is this be good enough to test guix pull?

It should be, though I noticed that Mark reported a bug recently so we
need to check.

Ludo’.

Message #20 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Thu, 12 Dec 2019 16:27:01 +0100

Hi Pierre,

On Wed, 11 Dec 2019 at 01:12, Pierre Neidhardt <mail <at> ambrevar.xyz> wrote:
>
> Ludovic Courtès <ludo <at> gnu.org> writes:
>
> > “make as-derivation” achieves more or less the same.
>
> Is this be good enough to test guix pull?

Have you tried to run the current test suite on the commit you
recently sent breaking "guix pull"?

(I do not remember which commit it was and who sent the fix. If you
indicate me the commit or patch or bug-number introducing the break of
guix pull, I can try.)

All the best,
simon

Message #23 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Fri, 13 Dec 2019 10:45:51 +0100

[Message part 1 (text/plain, inline)]

I tried it on two recent commits
(including eb8aad6a23442cf7b23e0df88b89b4cd030dfbf5) and I get

--8<---------------cut here---------------start------------->8---
$ guix environment guix -- make as-derivation
Building Guix in Guix...
warning: 'nix-server-socket' is deprecated, use 'store-connection-socket' instead
warning: 'nix-server-major-version' is deprecated, use 'store-connection-major-version' instead
warning: 'nix-server-minor-version' is deprecated, use 'store-connection-minor-version' instead
fetching path `/gnu/store/qa3sp0f1qpqhrzb0v9y5kfgyr61dh70r-xz-5.2.4.tar.gz'...
building path(s) `/gnu/store/aqv75406kbs6s1sp3z9x5fbl9c7y7psn-ld-wrapper-boot0-0'
Downloading https://ci.guix.gnu.org/nar/qa3sp0f1qpqhrzb0v9y5kfgyr61dh70r-xz-5.2.4.tar.gz...
building "/gnu/store/w8fw0vzvai0yw5akfq3gd94avb7jvd4h-binutils-cross-boot0-2.32"/bin/ld wrapper in "/gnu/store/aqv75406kbs6s1sp3z9x5fbl9c7y7psn-ld-wrapper-boot0-0"
Backtrace:
In ice-9/boot-9.scm:
 157: 11 [catch #t #<catch-closure c900a0> ...]
In unknown file:
   ?: 10 [apply-smob/1 #<catch-closure c900a0>]
In ice-9/boot-9.scm:
  63: 9 [call-with-prompt prompt0 ...]
In ice-9/eval.scm:
 432: 8 [eval # #]
In ice-9/boot-9.scm:
2320: 7 [save-module-excursion #<procedure cc1b80 at ice-9/boot-9.scm:3961:3 ()>]
3966: 6 [#<procedure cc1b80 at ice-9/boot-9.scm:3961:3 ()>]
1645: 5 [%start-stack load-stack #<procedure cbd2e0 at ice-9/boot-9.scm:3957:10 ()>]
1650: 4 [#<procedure cc3060 ()>]
In unknown file:
   ?: 3 [primitive-load "/gnu/store/7pcp12416a7gjfm18pk87hradjgrnnrz-ld-wrapper-boot0-0-guile-builder"]
In ice-9/eval.scm:
 387: 2 [eval # ()]
 432: 1 [eval # #]
In unknown file:
   ?: 0 [copy-file "/gnu/store/2p4rab987bknvsqm46rxkz7d55wyqpfy-ld-wrapper.in" ...]

ERROR: In procedure copy-file:
ERROR: In procedure copy-file: No such file or directory
builder for `/gnu/store/dvrl5vf6ghxx9w70gacrcamqaxbiwlrf-ld-wrapper-boot0-0.drv' failed with exit code 1
cannot build derivation `/gnu/store/f3ip8y1kw6py0hb597yr8mklsj7wxra2-bash-minimal-5.0.7.drv': 1 dependencies couldn't be built
killing process 14973
cannot build derivation `/gnu/store/gf0ic28nrb8ydz89c5hmjzvs7ndg149n-binutils-2.32.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/f0zz87120k9sb88lijb87dcb0skw5vzz-bzip2-1.0.6.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/s1x51d8rvl845pxsmgg865z4rw6fkvlc-coreutils-8.31.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/1w2h1h84ir7hi27kgiyxm1kcfzv7yrc0-diffutils-3.7.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/w88snqxx45n9m6iyhl491kwlcpjgmqfv-file-5.33.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/dnvs0wggv8gnk27qravhnssgprc6qndc-findutils-4.6.0.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/33asax0rynrvqk9ay6z131ziibym2s9f-gawk-5.0.1.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/s37yl9v9ivhlag4drlxwffjlcmy4yygj-gcc-7.4.0.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/501b6362yp7ng5cdpqg5z08j3c80wcgq-glibc-2.29.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/my7j43l4sspjwxva6gn8jslk5qyaplqi-gmp-6.1.2.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/02dfgh08n9zsa0gkvimh9675430frll7-grep-3.3.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/fnd0k2sdqnin6p1788qjbma37fg3f85n-guile-2.2.6.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/nnp7zxpnjk01zfqm9f4kvdk18g2008ka-gzip-1.10.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/z2ndd6klhz3kl2g048krf3mq576zq7cj-gzip-1.10.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/xxns752z4znp24c46l5aq8j0lvds9s07-libgc-7.6.12.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/v35m6i04n4qbkwbahi7pa89rrlzzr49w-libltdl-2.4.6.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/hm5i7xmvv40ds9ymnnqn4cy75jb9d9y4-libunistring-0.9.10.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/5ab03vwm2hkk24r0i3xgj2q13fzxn995-make-4.2.1.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/72g1a2nhn6h055fa2aqga7mgck0fqc12-patch-2.7.6.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/c9xax133y945rszbmax006jjsadvdiyg-sed-4.7.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/c8m06xvadb4rqa8znpf16xziamh3kc7l-tar-1.32.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/6lwh9va6krsazd53y4fpid8isv0lbz7g-xz-5.2.4.drv': 1 dependencies couldn't be built
cannot build derivation `/gnu/store/yxz3jnqi0scmyp70yg4w0mwqfr6n9n9w-ghostscript-9.27.drv': 1 dependencies couldn't be built
error: build of `/gnu/store/yxz3jnqi0scmyp70yg4w0mwqfr6n9n9w-ghostscript-9.27.drv' failed
make: *** [Makefile:5712: as-derivation] Error 1
--8<---------------cut here---------------end--------------->8---

Any idea?

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #26 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Fri, 13 Dec 2019 12:20:07 +0100

Hi Pierre,

On Fri, 13 Dec 2019 at 10:45, Pierre Neidhardt <mail <at> ambrevar.xyz> wrote:

> I tried it on two recent commits
> (including eb8aad6a23442cf7b23e0df88b89b4cd030dfbf5) and I get

Is it a pushed commit? Or a local one?

[...]

> Any idea?

I have no idea. It can only propose to try to reproduce. If you send
what is your Guix commit (to know in which environment you run) and
your Git commit in which you run "make as-derivation".


Cheers,
simon

Message #29 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Fri, 13 Dec 2019 13:19:28 +0100

On Fri, 13 Dec 2019 at 12:20, zimoun <zimon.toutoune <at> gmail.com> wrote:

> On Fri, 13 Dec 2019 at 10:45, Pierre Neidhardt <mail <at> ambrevar.xyz> wrote:
>
> > I tried it on two recent commits
> > (including eb8aad6a23442cf7b23e0df88b89b4cd030dfbf5) and I get
>
> Is it a pushed commit? Or a local one?

My bad. I find the commit. :-)
But I do not experiment the issue you described.
No idea...

Cheers,
simon

Message #32 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: Ludovic Courtès <ludo <at> gnu.org>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Fri, 13 Dec 2019 13:27:24 +0100

[Message part 1 (text/plain, inline)]

guix commit: 9fcf282.

> Is it a pushed commit? Or a local one?

Yes, it is pushed of course! :)

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #35 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528 <at> debbugs.gnu.org, zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: bug#38528: Add test for `guix pull'
Date: Sat, 14 Dec 2019 15:11:05 +0100

Hi,

Pierre Neidhardt <mail <at> ambrevar.xyz> skribis:

> $ guix environment guix -- make as-derivation
> Building Guix in Guix...
> warning: 'nix-server-socket' is deprecated, use 'store-connection-socket' instead
> warning: 'nix-server-major-version' is deprecated, use 'store-connection-major-version' instead
> warning: 'nix-server-minor-version' is deprecated, use 'store-connection-minor-version' instead
> fetching path `/gnu/store/qa3sp0f1qpqhrzb0v9y5kfgyr61dh70r-xz-5.2.4.tar.gz'...
> building path(s) `/gnu/store/aqv75406kbs6s1sp3z9x5fbl9c7y7psn-ld-wrapper-boot0-0'
> Downloading https://ci.guix.gnu.org/nar/qa3sp0f1qpqhrzb0v9y5kfgyr61dh70r-xz-5.2.4.tar.gz...
> building "/gnu/store/w8fw0vzvai0yw5akfq3gd94avb7jvd4h-binutils-cross-boot0-2.32"/bin/ld wrapper in "/gnu/store/aqv75406kbs6s1sp3z9x5fbl9c7y7psn-ld-wrapper-boot0-0"
> Backtrace:
> In ice-9/boot-9.scm:
>  157: 11 [catch #t #<catch-closure c900a0> ...]
> In unknown file:
>    ?: 10 [apply-smob/1 #<catch-closure c900a0>]
> In ice-9/boot-9.scm:
>   63: 9 [call-with-prompt prompt0 ...]
> In ice-9/eval.scm:
>  432: 8 [eval # #]
> In ice-9/boot-9.scm:
> 2320: 7 [save-module-excursion #<procedure cc1b80 at ice-9/boot-9.scm:3961:3 ()>]
> 3966: 6 [#<procedure cc1b80 at ice-9/boot-9.scm:3961:3 ()>]
> 1645: 5 [%start-stack load-stack #<procedure cbd2e0 at ice-9/boot-9.scm:3957:10 ()>]
> 1650: 4 [#<procedure cc3060 ()>]
> In unknown file:
>    ?: 3 [primitive-load "/gnu/store/7pcp12416a7gjfm18pk87hradjgrnnrz-ld-wrapper-boot0-0-guile-builder"]
> In ice-9/eval.scm:
>  387: 2 [eval # ()]
>  432: 1 [eval # #]
> In unknown file:
>    ?: 0 [copy-file "/gnu/store/2p4rab987bknvsqm46rxkz7d55wyqpfy-ld-wrapper.in" ...]
>
> ERROR: In procedure copy-file:
> ERROR: In procedure copy-file: No such file or directory
> builder for `/gnu/store/dvrl5vf6ghxx9w70gacrcamqaxbiwlrf-ld-wrapper-boot0-0.drv' failed with exit code 1

This is weird…  All I can say it that it works for me.  :-)

Could you find out (1) why it’s trying to rebuild ‘ld-wrapper-boot0’
(which means rebuilding the world), and (2) why this particular build is
failing?

Thanks,
Ludo’.

Message #38 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 38528 <at> debbugs.gnu.org, zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: bug#38528: Add test for `guix pull'
Date: Mon, 16 Dec 2019 14:19:53 +0100

[Message part 1 (text/plain, inline)]

It works now, however it does not detect breakages like "guix pull -p
..." does.

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #41 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528 <at> debbugs.gnu.org, zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: bug#38528: Add test for `guix pull'
Date: Mon, 16 Dec 2019 15:36:23 +0100

Hi Pierre,

Pierre Neidhardt <mail <at> ambrevar.xyz> skribis:

> It works now, however it does not detect breakages like "guix pull -p
> ..." does.

Do you have a specific example in mind?  They normally build the exact
same thing.

All that said, I think it’s fine to recommend “guix pull -p” in the
manual, but I just wanted to make sure people know about “make
as-derivation”, which can be slightly more convenient in cases where you
don’t want to commit the change to test it.

Thanks,
Ludo’.

Message #44 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: Pierre Neidhardt <mail <at> ambrevar.xyz>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 38528 <at> debbugs.gnu.org, zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: bug#38528: Add test for `guix pull'
Date: Mon, 16 Dec 2019 15:39:12 +0100

[Message part 1 (text/plain, inline)]

Yes, I testsed on commit 
614a1e3fa2d731d4719f03912b1b87fb4fd309cb, which should be broken.

-- 
Pierre Neidhardt
https://ambrevar.xyz/

[signature.asc (application/pgp-signature, inline)]

Message #47 received at 38528 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Pierre Neidhardt <mail <at> ambrevar.xyz>, 38528 <at> debbugs.gnu.org
Subject: Re: bug#38528: Add test for `guix pull'
Date: Mon, 16 Dec 2019 16:27:08 +0100

Hi Ludo,

On Mon, 16 Dec 2019 at 15:36, Ludovic Courtès <ludo <at> gnu.org> wrote:

> All that said, I think it’s fine to recommend “guix pull -p” in the
> manual, but I just wanted to make sure people know about “make
> as-derivation”, which can be slightly more convenient in cases where you
> don’t want to commit the change to test it.

Both can be mentioned in the manual because they target different usage, IMHO.


All the best,
simon

Message #52 received at 38528-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Pierre Neidhardt <mail <at> ambrevar.xyz>
Cc: 38528-done <at> debbugs.gnu.org, Ludovic Courtès <ludo <at> gnu.org>,
 zimoun <zimon.toutoune <at> gmail.com>
Subject: Re: bug#38528: Add test for `guix pull'
Date: Fri, 19 Aug 2022 10:31:05 -0400

Hi,

Pierre Neidhardt <mail <at> ambrevar.xyz> writes:

> Yes, I testsed on commit
> 614a1e3fa2d731d4719f03912b1b87fb4fd309cb, which should be broken.

Closing this forgotten "bug".

Please do submit a patch if you'd like to document this.

Thanks,

Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.