bug#38528: Add test for `guix pull'

2022-08-19 Thread Maxim Cournoyer
Hi, Pierre Neidhardt 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

bug#38528: Add test for `guix pull'

2019-12-16 Thread zimoun
Hi Ludo, On Mon, 16 Dec 2019 at 15:36, Ludovic Courtès 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 t

bug#38528: Add test for `guix pull'

2019-12-16 Thread Pierre Neidhardt
Yes, I testsed on commit 614a1e3fa2d731d4719f03912b1b87fb4fd309cb, which should be broken. -- Pierre Neidhardt https://ambrevar.xyz/ signature.asc Description: PGP signature

bug#38528: Add test for `guix pull'

2019-12-16 Thread Ludovic Courtès
Hi Pierre, Pierre Neidhardt 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 wa

bug#38528: Add test for `guix pull'

2019-12-14 Thread Ludovic Courtès
Hi, Pierre Neidhardt 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 > w

bug#38528: Add test for `guix pull'

2019-12-13 Thread zimoun
On Fri, 13 Dec 2019 at 12:20, zimoun wrote: > On Fri, 13 Dec 2019 at 10:45, Pierre Neidhardt 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 ex

bug#38528: Add test for `guix pull'

2019-12-13 Thread zimoun
Hi Pierre, On Fri, 13 Dec 2019 at 10:45, Pierre Neidhardt 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 w

bug#38528: Add test for `guix pull'

2019-12-12 Thread zimoun
Hi Pierre, On Wed, 11 Dec 2019 at 01:12, Pierre Neidhardt wrote: > > Ludovic Courtès 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"

bug#38528: Add test for `guix pull'

2019-12-12 Thread Ludovic Courtès
Hi, Pierre Neidhardt skribis: > Ludovic Courtès 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’.

bug#38528: Add test for `guix pull'

2019-12-10 Thread Ludovic Courtès
Hi, Pierre Neidhardt 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 systematical

bug#38528: Add test for `guix pull'

2019-12-08 Thread zimoun
Hi, Thank Pierre to raise this issue. On Sun, 8 Dec 2019 at 16:40, Pierre Neidhardt 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 beca

bug#38528: Add test for `guix pull'

2019-12-08 Thread Pierre Neidhardt
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/c