all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#63726: time-machine without options does not get the latest commit
@ 2023-05-25 13:52 Simon Tournier
  2023-05-25 21:03 ` Ludovic Courtès
  0 siblings, 1 reply; 15+ messages in thread
From: Simon Tournier @ 2023-05-25 13:52 UTC (permalink / raw)
  To: 63726

Hi,

Well, I do not know if it is a bug or if I misuse something.  Feel free
to close if I am missing a point.

--8<---------------cut here---------------start------------->8---
$ guix describe
Generation 25	mai 19 2023 13:30:14	(current)
  guix 14c0380
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: 14c03807ba4bc81d42cf869f5b827f7da54ff843

$ git -C ~/.cache/guix/checkouts/pjmkglp4t7znuugeurpurzikxq3tnlaywmisyr27shj7apsnalwq log --oneline -1
14c03807ba4 (HEAD -> master) gnu: ruby-3.2: Upgrade to 3.2.2 [fixes CVE-2023-{28755, 28756}].
--8<---------------cut here---------------end--------------->8---

Well, that’s just a luck that both commit hash matches.

Now, what I am missing.  The manual says:

       As for ‘guix pull’, the absence of any options means that the latest
    commit on the master branch will be used.  The command

         guix time-machine -- build hello

       will thus build the package ‘hello’ as defined in the master branch,
    which is in general a newer revision of Guix than you have installed.
    Time travel works in both directions!

and I get:

--8<---------------cut here---------------start------------->8---
$ guix time-machine -- describe
  guix e499cb2
    repository URL: https://git.savannah.gnu.org/git/guix.git
    commit: e499cb2c12d7f1c6d2f004364c9cc7bdb7e38cd5
--8<---------------cut here---------------end--------------->8---

But I do not understand from where this commit hash is coming from.  And
please note that this commit seems coming from my previous experiments
reported in #63667 [1].

Well, the last commit seems a9cde26133b,

--8<---------------cut here---------------start------------->8---
$ git -C ~/.cache/guix/checkouts/pjmkglp4t7znuugeurpurzikxq3tnlaywmisyr27shj7apsnalwq log origin/master --oneline -1
a9cde26133b (origin/master, origin/HEAD) gnu: Add mouseloupe.
--8<---------------cut here---------------end--------------->8---

Therefore, let create a Guix revision:

--8<---------------cut here---------------start------------->8---
$ guix time-machine --commit=a9cde26133b -- describe
  guix a9cde26
    repository URL: https://git.savannah.gnu.org/git/guix.git
    commit: a9cde26133bd98498869a3528d9dff0d3f456a96
--8<---------------cut here---------------end--------------->8---

However, still this e499cb2 one:

--8<---------------cut here---------------start------------->8---
$ guix time-machine -- describe
  guix e499cb2
    repository URL: https://git.savannah.gnu.org/git/guix.git
    commit: e499cb2c12d7f1c6d2f004364c9cc7bdb7e38cd5
--8<---------------cut here---------------end--------------->8---

Cheers,
simon


1: https://issues.guix.gnu.org/issue/63667




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-25 13:52 bug#63726: time-machine without options does not get the latest commit Simon Tournier
@ 2023-05-25 21:03 ` Ludovic Courtès
  2023-05-26  9:31   ` Simon Tournier
                     ` (2 more replies)
  0 siblings, 3 replies; 15+ messages in thread
From: Ludovic Courtès @ 2023-05-25 21:03 UTC (permalink / raw)
  To: Simon Tournier; +Cc: Konrad Hinsen, 63726

Hi,

Simon Tournier <zimon.toutoune@gmail.com> skribis:

> Now, what I am missing.  The manual says:
>
>        As for ‘guix pull’, the absence of any options means that the latest
>     commit on the master branch will be used.  The command
>
>          guix time-machine -- build hello
>
>        will thus build the package ‘hello’ as defined in the master branch,
>     which is in general a newer revision of Guix than you have installed.
>     Time travel works in both directions!
>
> and I get:
>
> $ guix time-machine -- describe
>   guix e499cb2
>     repository URL: https://git.savannah.gnu.org/git/guix.git
>     commit: e499cb2c12d7f1c6d2f004364c9cc7bdb7e38cd5
>
>
> But I do not understand from where this commit hash is coming from.  And
> please note that this commit seems coming from my previous experiments
> reported in #63667 [1].
>
> Well, the last commit seems a9cde26133b,

I had to check the code, but ‘guix time-machine’ follows the same logic
as ‘guix pull’: it reads ~/.config/guix/channels.scm, and so on (that’s
the ‘channel-list’ procedure).

I must say I’m surprised, but it’s always been this way.

Should we fix the doc or should we fix the code?…

Ludo’.




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-25 21:03 ` Ludovic Courtès
@ 2023-05-26  9:31   ` Simon Tournier
  2023-05-26 11:15   ` Konrad Hinsen
  2023-09-04 16:55   ` Simon Tournier
  2 siblings, 0 replies; 15+ messages in thread
From: Simon Tournier @ 2023-05-26  9:31 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Konrad Hinsen, 63726

[-- Attachment #1: Type: text/plain, Size: 4104 bytes --]

Hi,

On jeu., 25 mai 2023 at 23:03, Ludovic Courtès <ludo@gnu.org> wrote:

> I had to check the code, but ‘guix time-machine’ follows the same logic
> as ‘guix pull’: it reads ~/.config/guix/channels.scm, and so on (that’s
> the ‘channel-list’ procedure).

--8<---------------cut here---------------start------------->8---
$ cat ~/.config/guix/channels.scm
(use-modules (guix ci))

(list (channel-with-substitutes-available
       %default-guix-channel
       "https://ci.guix.gnu.org"))

$ guix time-machine -C ~/.config/guix/channels.scm -- describe
Updating channel 'guix' from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
substitute: updating substitutes from 'https://ci.guix.gnu.org'... 100.0%
 compute-guix-derivation  1013B                                                                                    373KiB/s 00:00 ▕██████████████████▏ 100.0%
Computing Guix derivation for 'x86_64-linux'... \
substitute: updating substitutes from 'https://ci.guix.gnu.org'... 100.0%
The following derivations will be built:
  /gnu/store/80sisjhmmb0sbvyylavm3c2rgjnhypw2-profile.drv
  /gnu/store/3213n6v442d1zv17xidg8s48hx02f433-guix-daemon.drv
  /gnu/store/77k19wqklyw4gvxqjjdg8ah4k9llb5c0-guix-command.drv
  /gnu/store/3cyv2vdk3v0asjqbspzlgbzibk1f50p2-guix-system-tests-modules.drv
  /gnu/store/9pq5bivbzg476svaqv38xjgwyhg0ml2k-guix-cli-core-modules.drv
  /gnu/store/gpjmcxvr10fnzjvkwr7yf2wpyrzyb5nm-guix-packages-base-modules.drv
  /gnu/store/gqamr3s5amc1ayqa4xbzv114zy6l8ac8-guix-system-modules.drv
  /gnu/store/xv2yv334dpxvq8bw605avhcf6dv0smyp-guix-home-modules.drv
  /gnu/store/yya1dwvx7svj588vnqznabqaa2ahv4a3-guix-cli-modules.drv

[...]

building profile with 1 package...
  guix 849286b
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: 849286ba66c96534bddc04df1a47d5692cbc977e

$ guix time-machine -- describe
  guix 849286b
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: 849286ba66c96534bddc04df1a47d5692cbc977e

$ mv ~/.config/guix/channels.scm /tmp/channels.scm

$ guix time-machine -C ~/.config/guix/channels.scm -- describe
guix time-machine: error: failed to load '/home/simon/.config/guix/channels.scm': No such file or directory

$ guix time-machine -- describe
Updating channel 'guix' from Git repository at 'https://git.savannah.gnu.org/git/guix.git'...
Authenticating channel 'guix', commits 9edb3f6 to 3299b40 (36 new commits)...
substitute: updating substitutes from 'https://ci.guix.gnu.org'... 100.0%
 compute-guix-derivation  1016B                                                                                    168KiB/s 00:00 ▕██████████████████▏ 100.0%
Computing Guix derivation for 'x86_64-linux'... -
substitute: updating substitutes from 'https://ci.guix.gnu.org'... 100.0%
The following derivations will be built:
  /gnu/store/4gm2ay6gjsjq6iclmbm4jqnvja12axy2-profile.drv
  /gnu/store/7b50p1xnh4r5ahfj4da8yb8xgzqdjsyv-guix-daemon.drv
  /gnu/store/qgcnzghwqxkak53rhnggl73d6r1jkq4q-guix-command.drv
  /gnu/store/1spf6ndyz62izy3xgr9v5jnjin305s2v-guix-cli-core-modules.drv
  /gnu/store/3fib3w2cjl83r75zfllby6wp3znh362p-guix-system-tests-modules.drv
  /gnu/store/7vn6pq6j5psjwlcgvl1nbgwlspamx1wx-guix-home-modules.drv
  /gnu/store/9qr483y1law33nqdwz7bjr32sbkka2xw-guix-cli-modules.drv
  /gnu/store/iq10xaghbj6435w4c6nmyqb3kwzs119f-guix-packages-base-modules.drv
  /gnu/store/wzssfkmqywrqm86vjlyxzpf26d4h30qd-guix-system-modules.drv

[...]

building profile with 1 package...
  guix 3299b40
    repository URL: https://git.savannah.gnu.org/git/guix.git
    branch: master
    commit: 3299b40b3599371890816b52d65a6a5ada1b1131
--8<---------------cut here---------------end--------------->8---

Hum.  Now, I am not seeing anymore e499cb2.  Related to #63667 [1]?

1: https://issues.guix.gnu.org/issue/63667


> Should we fix the doc or should we fix the code?…

What about this patch?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: doc.patch --]
[-- Type: text/x-diff, Size: 2058 bytes --]

From 63b862ed9f3d3b019af3c6926bc1737f573bdfe6 Mon Sep 17 00:00:00 2001
Message-Id: <63b862ed9f3d3b019af3c6926bc1737f573bdfe6.1685093404.git.zimon.toutoune@gmail.com>
From: Simon Tournier <zimon.toutoune@gmail.com>
Date: Fri, 26 May 2023 11:26:32 +0200
Subject: [PATCH] doc: Reword guix time-machine without option.

* doc/guix.texi (Invoking guix time-machine): Reword the behaviour in the
absence of any option.
---
 doc/guix.texi | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 31dc33fb97..54f8a20196 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -4958,16 +4958,20 @@ Invoking guix time-machine
 @xref{Channels} for more information.
 @end table
 
-As for @command{guix pull}, the absence of any options means that the
-latest commit on the master branch will be used.  The command
+Similarly as @command{guix pull}, the absence of any options means that
+either the user's @file{~/.config/guix/channels.scm} file, or either the
+system-wide @file{/etc/guix/channels.scm} file, or either the built-in
+default channels specified in the @code{%default-channels} will be
+honored, in that order.  The command
 
 @example
 guix time-machine -- build hello
 @end example
 
-will thus build the package @code{hello} as defined in the master branch,
-which is in general a newer revision of Guix than you have installed.
-Time travel works in both directions!
+will thus build the package @code{hello} as defined by the previous
+@dfn{channels} file (@pxref{Channels}).  In general, it builds a newer
+revision of Guix than you have installed.  Time travel works in both
+directions!
 
 Note that @command{guix time-machine} can trigger builds of channels and
 their dependencies, and these are controlled by the standard build

base-commit: 99fc7e5dd3c9c92f9b6c2e51db9dc5b0481a56b2
prerequisite-patch-id: 7ff5f10954ba2e9663c6e7b15e7aee448c8b74c2
prerequisite-patch-id: cb31c4d7ad8d27df155dcf57b61116048a6b0681
prerequisite-patch-id: f256649d0fac1d618b07615c97c46e66436b9262
-- 
2.38.1


[-- Attachment #3: Type: text/plain, Size: 16 bytes --]



Cheers,
simon

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-25 21:03 ` Ludovic Courtès
  2023-05-26  9:31   ` Simon Tournier
@ 2023-05-26 11:15   ` Konrad Hinsen
  2023-05-30 13:32     ` Simon Tournier
  2023-09-04 16:55   ` Simon Tournier
  2 siblings, 1 reply; 15+ messages in thread
From: Konrad Hinsen @ 2023-05-26 11:15 UTC (permalink / raw)
  To: Ludovic Courtès, Simon Tournier; +Cc: 63726

Hi,

> Should we fix the doc or should we fix the code?…

I vote for fixing the doc. Two reasons:

 1. Having "guix time-machine" and "guix pull" behave in the same way
    is desirable. Less cognitive load for users.

 2. What the doc says cannot be implemented in general. "The latest
    commit on the master branch" makes no sense for a setup with
    multiple channels, some of which may not even have a branch called
    "master".

Cheers,
  Konrad.




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-26 11:15   ` Konrad Hinsen
@ 2023-05-30 13:32     ` Simon Tournier
  2023-06-01 15:01       ` Konrad Hinsen
  0 siblings, 1 reply; 15+ messages in thread
From: Simon Tournier @ 2023-05-30 13:32 UTC (permalink / raw)
  To: Konrad Hinsen, Ludovic Courtès; +Cc: 63726

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

Hi Konrad,

On ven., 26 mai 2023 at 13:15, Konrad Hinsen <konrad.hinsen@fastmail.net> wrote:
> Hi,
>
>> Should we fix the doc or should we fix the code?…
>
> I vote for fixing the doc. Two reasons:

What about this patch?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: p.patch --]
[-- Type: text/x-diff, Size: 2058 bytes --]

From 63b862ed9f3d3b019af3c6926bc1737f573bdfe6 Mon Sep 17 00:00:00 2001
Message-Id: <63b862ed9f3d3b019af3c6926bc1737f573bdfe6.1685093404.git.zimon.toutoune@gmail.com>
From: Simon Tournier <zimon.toutoune@gmail.com>
Date: Fri, 26 May 2023 11:26:32 +0200
Subject: [PATCH] doc: Reword guix time-machine without option.

* doc/guix.texi (Invoking guix time-machine): Reword the behaviour in the
absence of any option.
---
 doc/guix.texi | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 31dc33fb97..54f8a20196 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -4958,16 +4958,20 @@ Invoking guix time-machine
 @xref{Channels} for more information.
 @end table
 
-As for @command{guix pull}, the absence of any options means that the
-latest commit on the master branch will be used.  The command
+Similarly as @command{guix pull}, the absence of any options means that
+either the user's @file{~/.config/guix/channels.scm} file, or either the
+system-wide @file{/etc/guix/channels.scm} file, or either the built-in
+default channels specified in the @code{%default-channels} will be
+honored, in that order.  The command
 
 @example
 guix time-machine -- build hello
 @end example
 
-will thus build the package @code{hello} as defined in the master branch,
-which is in general a newer revision of Guix than you have installed.
-Time travel works in both directions!
+will thus build the package @code{hello} as defined by the previous
+@dfn{channels} file (@pxref{Channels}).  In general, it builds a newer
+revision of Guix than you have installed.  Time travel works in both
+directions!
 
 Note that @command{guix time-machine} can trigger builds of channels and
 their dependencies, and these are controlled by the standard build

base-commit: 99fc7e5dd3c9c92f9b6c2e51db9dc5b0481a56b2
prerequisite-patch-id: 7ff5f10954ba2e9663c6e7b15e7aee448c8b74c2
prerequisite-patch-id: cb31c4d7ad8d27df155dcf57b61116048a6b0681
prerequisite-patch-id: f256649d0fac1d618b07615c97c46e66436b9262
-- 
2.38.1


[-- Attachment #3: Type: text/plain, Size: 15 bytes --]


Cheers,
simon

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-30 13:32     ` Simon Tournier
@ 2023-06-01 15:01       ` Konrad Hinsen
  2023-08-11 15:36         ` Ludovic Courtès
  0 siblings, 1 reply; 15+ messages in thread
From: Konrad Hinsen @ 2023-06-01 15:01 UTC (permalink / raw)
  To: Simon Tournier, Ludovic Courtès; +Cc: 63726

[-- Attachment #1: Type: text/plain, Size: 854 bytes --]

Hi Simon,

> What about this patch?

That's a good start! But I didn't quite understand what "as defined by the previous
+@dfn{channels} file" means, so I did some more research and some more
rewriting, and came up with the attached patch. That one is clear for
me, but maybe not for others...

In the course of this research, I noticed that channel files are used in
Guix in two varieties for rather different purposes, but this doesn't
seem to be explained anywhere. "guix describe -f channels" produces
channel files with a pinned commit for each repository. That's what we
usually give to time-machine. That feature of channel files seems even
undocumented. The chapter on channels discusses the channel files we
typically use with "guix pull", and which refer to a branch and thus its
last commit, rather than to a pinned commit.

Cheers,
  Konrad.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-doc-Reword-guix-time-machine-without-option.patch --]
[-- Type: text/x-diff, Size: 2077 bytes --]

From cbe372191a2daea7b62d8558422f08bc6ed0e047 Mon Sep 17 00:00:00 2001
From: Konrad Hinsen <konrad.hinsen@fastmail.net>
Date: Thu, 1 Jun 2023 16:55:33 +0200
Subject: [PATCH] doc: Reword guix time-machine without option.

* doc/guix.texi (Invoking guix time-machine): Reword the behaviour in the
absence of any option.
---
 doc/guix.texi | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 7f8d8d66e9..e4daffe4dd 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -4959,16 +4959,25 @@ Invoking guix time-machine
 @xref{Channels} for more information.
 @end table
 
-As for @command{guix pull}, the absence of any options means that the
-latest commit on the master branch will be used.  The command
+In the absence of any options, the target revision will be determined as
+for @command{guix pull}, i.e. from the user's
+@file{~/.config/guix/channels.scm} file, or, if no such file exists,
+from the system-wide @file{/etc/guix/channels.scm} file, or, if that
+file does not exist either, from the built-in default channels specified
+in the @code{%default-channels} variable.  Note that these three
+fallbacks are usually channel definitions pointing to @i{branches} in
+git repositories, rather then to specific @i{commits}, as the channel
+definitions produced by @command{guix describe} do. A branch-based
+channel definition refers to the latest commit on that branch. The
+command
 
 @example
 guix time-machine -- build hello
 @end example
 
-will thus build the package @code{hello} as defined in the master branch,
-which is in general a newer revision of Guix than you have installed.
-Time travel works in both directions!
+will thus build the package @code{hello} as defined by the current state
+of the default channels. In general, it builds a @i{newer} revision of
+Guix than you have installed.  Time travel works in both directions!
 
 Note that @command{guix time-machine} can trigger builds of channels and
 their dependencies, and these are controlled by the standard build
-- 
2.40.1


^ permalink raw reply related	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-06-01 15:01       ` Konrad Hinsen
@ 2023-08-11 15:36         ` Ludovic Courtès
  2023-08-14 10:13           ` Konrad Hinsen
  0 siblings, 1 reply; 15+ messages in thread
From: Ludovic Courtès @ 2023-08-11 15:36 UTC (permalink / raw)
  To: Konrad Hinsen, Simon Tournier; +Cc: 63726

Hi Konrad & Simon,

I had forgotten about this issue (my apologies…) and stumbled upon it
again recently, which led me to approach it a bit differently:

  https://issues.guix.gnu.org/65229

Let me know what you think!

Thanks,
Ludo’.




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-11 15:36         ` Ludovic Courtès
@ 2023-08-14 10:13           ` Konrad Hinsen
  2023-08-16 13:51             ` Simon Tournier
  2023-08-17  8:42             ` Ludovic Courtès
  0 siblings, 2 replies; 15+ messages in thread
From: Konrad Hinsen @ 2023-08-14 10:13 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 63726, Simon Tournier

Hi Ludo,

> I had forgotten about this issue (my apologies…) and stumbled upon it
> again recently, which led me to approach it a bit differently:
>
>   https://issues.guix.gnu.org/65229
>
> Let me know what you think!

This looks good to me. In practice, I doubt anyone would use -q with
time-machine, because it makes more sense to specify an explicit channel
file every time. But for pull, I can well imaging using it in a
reproducibility-oriented setting.

Cheers,
  Konrad




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-14 10:13           ` Konrad Hinsen
@ 2023-08-16 13:51             ` Simon Tournier
  2023-08-17  8:42             ` Ludovic Courtès
  1 sibling, 0 replies; 15+ messages in thread
From: Simon Tournier @ 2023-08-16 13:51 UTC (permalink / raw)
  To: Konrad Hinsen, Ludovic Courtès; +Cc: 63726

Hi,

On Mon, 14 Aug 2023 at 12:13, Konrad Hinsen <konrad.hinsen@fastmail.net> wrote:

>> Let me know what you think!

LGTM! :-)


> This looks good to me. In practice, I doubt anyone would use -q with
> time-machine, because it makes more sense to specify an explicit channel
> file every time. But for pull, I can well imaging using it in a
> reproducibility-oriented setting.

Well, I am also lacking imagination for “guix time-machine -q”. :-)

Moreover, the option precedence (commit, branch, channel file, etc.) is
never clear for me.  Hum, maybe I could check if it’s documented… ;-)


Cheers,
simon






^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-14 10:13           ` Konrad Hinsen
  2023-08-16 13:51             ` Simon Tournier
@ 2023-08-17  8:42             ` Ludovic Courtès
  2023-08-17 13:41               ` Maxim Cournoyer
  1 sibling, 1 reply; 15+ messages in thread
From: Ludovic Courtès @ 2023-08-17  8:42 UTC (permalink / raw)
  To: Konrad Hinsen; +Cc: 63726, Simon Tournier

Hey!

Konrad Hinsen <konrad.hinsen@fastmail.net> skribis:

> This looks good to me. In practice, I doubt anyone would use -q with
> time-machine, because it makes more sense to specify an explicit channel
> file every time.

Yeah.  The use case I have in mind is something like:

  guix time-machine -q --commit=XYZ -- build hello

Useful, for example, to share a way to reproduce a bug, making sure
there’s nothing but the ‘guix’ channel.

Thanks for taking a look!

Ludo’.




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-17  8:42             ` Ludovic Courtès
@ 2023-08-17 13:41               ` Maxim Cournoyer
  2023-08-17 14:07                 ` Ludovic Courtès
  0 siblings, 1 reply; 15+ messages in thread
From: Maxim Cournoyer @ 2023-08-17 13:41 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Konrad Hinsen, 63726, Simon Tournier

Hi,

Ludovic Courtès <ludovic.courtes@inria.fr> writes:

> Hey!
>
> Konrad Hinsen <konrad.hinsen@fastmail.net> skribis:
>
>> This looks good to me. In practice, I doubt anyone would use -q with
>> time-machine, because it makes more sense to specify an explicit channel
>> file every time.
>
> Yeah.  The use case I have in mind is something like:
>
>   guix time-machine -q --commit=XYZ -- build hello
>
> Useful, for example, to share a way to reproduce a bug, making sure
> there’s nothing but the ‘guix’ channel.

The '-q' short option by itself is a bit opaque / hard to memorize;
perhaps complementing it with a self explanatory
'--ignore-channel-files' long option name would be a good idea?

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-17 13:41               ` Maxim Cournoyer
@ 2023-08-17 14:07                 ` Ludovic Courtès
  2023-08-17 14:39                   ` Maxim Cournoyer
  0 siblings, 1 reply; 15+ messages in thread
From: Ludovic Courtès @ 2023-08-17 14:07 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: Konrad Hinsen, 63726, Simon Tournier

Hi,

Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:

> Ludovic Courtès <ludovic.courtes@inria.fr> writes:
>
>> Hey!
>>
>> Konrad Hinsen <konrad.hinsen@fastmail.net> skribis:
>>
>>> This looks good to me. In practice, I doubt anyone would use -q with
>>> time-machine, because it makes more sense to specify an explicit channel
>>> file every time.
>>
>> Yeah.  The use case I have in mind is something like:
>>
>>   guix time-machine -q --commit=XYZ -- build hello
>>
>> Useful, for example, to share a way to reproduce a bug, making sure
>> there’s nothing but the ‘guix’ channel.
>
> The '-q' short option by itself is a bit opaque / hard to memorize;
> perhaps complementing it with a self explanatory
> '--ignore-channel-files' long option name would be a good idea?

Yes, I did that in v2, as Simon suggested.

(‘-q’ seems to be a relatively common convention: emacs, guile, and also
‘guix repl’.)

Ludo’.




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-08-17 14:07                 ` Ludovic Courtès
@ 2023-08-17 14:39                   ` Maxim Cournoyer
  0 siblings, 0 replies; 15+ messages in thread
From: Maxim Cournoyer @ 2023-08-17 14:39 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Konrad Hinsen, 63726, Simon Tournier

Hi Ludovic,

Ludovic Courtès <ludovic.courtes@inria.fr> writes:

> Hi,
>
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Ludovic Courtès <ludovic.courtes@inria.fr> writes:
>>
>>> Hey!
>>>
>>> Konrad Hinsen <konrad.hinsen@fastmail.net> skribis:
>>>
>>>> This looks good to me. In practice, I doubt anyone would use -q with
>>>> time-machine, because it makes more sense to specify an explicit channel
>>>> file every time.
>>>
>>> Yeah.  The use case I have in mind is something like:
>>>
>>>   guix time-machine -q --commit=XYZ -- build hello
>>>
>>> Useful, for example, to share a way to reproduce a bug, making sure
>>> there’s nothing but the ‘guix’ channel.
>>
>> The '-q' short option by itself is a bit opaque / hard to memorize;
>> perhaps complementing it with a self explanatory
>> '--ignore-channel-files' long option name would be a good idea?
>
> Yes, I did that in v2, as Simon suggested.

Oh, I had missed that.  Thank you!

> (‘-q’ seems to be a relatively common convention: emacs, guile, and also
> ‘guix repl’.)

I think it's also commonly used as '--quiet', in other circles :-).

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-05-25 21:03 ` Ludovic Courtès
  2023-05-26  9:31   ` Simon Tournier
  2023-05-26 11:15   ` Konrad Hinsen
@ 2023-09-04 16:55   ` Simon Tournier
  2023-09-08 15:39     ` Ludovic Courtès
  2 siblings, 1 reply; 15+ messages in thread
From: Simon Tournier @ 2023-09-04 16:55 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Konrad Hinsen, 63726

Hi,

On Thu, 25 May 2023 at 23:03, Ludovic Courtès <ludo@gnu.org> wrote:

>> Now, what I am missing.  The manual says:
>>
>>        As for ‘guix pull’, the absence of any options means that the latest
>>     commit on the master branch will be used.  The command
>>
>>          guix time-machine -- build hello
>>
>>        will thus build the package ‘hello’ as defined in the master branch,
>>     which is in general a newer revision of Guix than you have installed.
>>     Time travel works in both directions!

[...]

> Should we fix the doc or should we fix the code?…

All seems fixed so let close it?

Cheers,
simon




^ permalink raw reply	[flat|nested] 15+ messages in thread

* bug#63726: time-machine without options does not get the latest commit
  2023-09-04 16:55   ` Simon Tournier
@ 2023-09-08 15:39     ` Ludovic Courtès
  0 siblings, 0 replies; 15+ messages in thread
From: Ludovic Courtès @ 2023-09-08 15:39 UTC (permalink / raw)
  To: Simon Tournier; +Cc: Konrad Hinsen, 63726-done

Simon Tournier <zimon.toutoune@gmail.com> skribis:

> On Thu, 25 May 2023 at 23:03, Ludovic Courtès <ludo@gnu.org> wrote:
>
>>> Now, what I am missing.  The manual says:
>>>
>>>        As for ‘guix pull’, the absence of any options means that the latest
>>>     commit on the master branch will be used.  The command
>>>
>>>          guix time-machine -- build hello
>>>
>>>        will thus build the package ‘hello’ as defined in the master branch,
>>>     which is in general a newer revision of Guix than you have installed.
>>>     Time travel works in both directions!
>
> [...]
>
>> Should we fix the doc or should we fix the code?…
>
> All seems fixed so let close it?

Yes, done!




^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2023-09-08 15:40 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-25 13:52 bug#63726: time-machine without options does not get the latest commit Simon Tournier
2023-05-25 21:03 ` Ludovic Courtès
2023-05-26  9:31   ` Simon Tournier
2023-05-26 11:15   ` Konrad Hinsen
2023-05-30 13:32     ` Simon Tournier
2023-06-01 15:01       ` Konrad Hinsen
2023-08-11 15:36         ` Ludovic Courtès
2023-08-14 10:13           ` Konrad Hinsen
2023-08-16 13:51             ` Simon Tournier
2023-08-17  8:42             ` Ludovic Courtès
2023-08-17 13:41               ` Maxim Cournoyer
2023-08-17 14:07                 ` Ludovic Courtès
2023-08-17 14:39                   ` Maxim Cournoyer
2023-09-04 16:55   ` Simon Tournier
2023-09-08 15:39     ` Ludovic Courtès

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.