[PATCH] doc: Add cindex & command to show versions of dependencies

DoneSubmitted by swedebugia.
Details
2 participants
  • swedebugia
  • zimoun
Owner
unassigned
Severity
normal
S
S
swedebugia wrote on 8 Nov 2018 17:00
(address . guix-patches@gnu.org)
3daf961d-eab0-305f-b6e3-76f38b0643da@riseup.net
Hi

Here is my first patch!

--
Cheers
Swedebugia
From 331dfcdcdf75f943f2395fe829703726200cbb5b Mon Sep 17 00:00:00 2001
From: swedebugia <swedebugia@riseup.net>
Date: Thu, 8 Nov 2018 16:14:06 +0100
Subject: [PATCH] doc: Add cindex & command to show versions of dependencies

* doc/guix.texi: Add 2 cindex in 2 sections
* doc/guix.texi (Running the Test Suite): Add command to show versions of dependencies
---
doc/guix.texi | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)

Toggle diff (39 lines)
diff --git a/doc/guix.texi b/doc/guix.texi
index 441b65bca..0b906edc1 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -52,6 +52,7 @@ Copyright @copyright{} 2018 Pierre-Antoine Rouby@*
 Copyright @copyright{} 2018 Gábor Boskovits@*
 Copyright @copyright{} 2018 Florian Pelz@*
 Copyright @copyright{} 2018 Laura Lazzati@*
+Copyright @copyright{} 2018 swedebugia@*
 
 Permission is granted to copy, distribute and/or modify this document
 under the terms of the GNU Free Documentation License, Version 1.3 or
@@ -602,6 +603,7 @@ would find yourself badly handicapped by the lack of the @command{guix}
 command.  In other words, do not remove @code{guix} by running
 @code{guix package -r guix}.
 
+@cindex producing the binary guix tarball
 The binary installation tarball can be (re)produced and verified simply
 by running the following command in the Guix source tree:
 
@@ -743,8 +745,15 @@ make check TESTS="tests/base64.scm" SCM_LOG_DRIVER_FLAGS="--brief=no"
 Upon failure, please email @email{bug-guix@@gnu.org} and attach the
 @file{test-suite.log} file.  Please specify the Guix version being used
 as well as version numbers of the dependencies (@pxref{Requirements}) in
-your message.
+your message, e.g. by running this command:
 
+@example
+guile --version | head -1 && guix --version | head -1 && \
+gcc --version | head -1 && echo -n ``libgcrypt `` && \
+libgcrypt-config --version && make --version | head -1
+@end example
+
+@cindex whole-system test suite
 Guix also comes with a whole-system test suite that tests complete
 GuixSD operating system instances.  It can only run on systems where
 Guix is already installed, using:
-- 
2.18.0
S
S
swedebugia wrote on 3 Jan 2019 17:21
(address . 33318@debbugs.gnu.org)
8cf7f956-35c1-a028-3f50-13718ba5de30@riseup.net
On 2018-11-08 17:00, swedebugia wrote:
Toggle quote (5 lines)
> Hi
>
> Here is my first patch!
>

Ping

--
Cheers Swedebugia
Z
Z
zimoun wrote on 7 Apr 13:45 +0200
Re: bug#33318: [PATCH] doc: Add cindex & command to show versions of dependencies
(name . swedebugia)(address . swedebugia@riseup.net)(address . 33318@debbugs.gnu.org)
86czht3ynu.fsf@gmail.com
Hi,

Looking at the status of old patches, I hit this one patch#33318:


Thanks for the patch.


On Thu, 08 Nov 2018 at 17:00, swedebugia <swedebugia@riseup.net> wrote:

Toggle quote (17 lines)
> +@cindex producing the binary guix tarball
> The binary installation tarball can be (re)produced and verified simply
> by running the following command in the Guix source tree:
>
> @@ -743,8 +745,15 @@ make check TESTS="tests/base64.scm" SCM_LOG_DRIVER_FLAGS="--brief=no"
> Upon failure, please email @email{bug-guix@@gnu.org} and attach the
> @file{test-suite.log} file. Please specify the Guix version being used
> as well as version numbers of the dependencies (@pxref{Requirements}) in
> -your message.
> +your message, e.g. by running this command:
>
> +@example
> +guile --version | head -1 && guix --version | head -1 && \
> +gcc --version | head -1 && echo -n ``libgcrypt `` && \
> +libgcrypt-config --version && make --version | head -1
> +@end example

Well, I do not know if this example is really useful.

I mean, if one is able to successfully run ’make’ without running before
“guix shell -D guix”, then this person probably already know how to
provide the versions of the dependencies, I guess. :-)

Moreover, such failures are rare so once the bug report open, we can ask
more details if the initial report does not contain enough.

I propose to close this submission without applying. WDYT?


Cheers,
simon
Z
Z
zimoun wrote on 19 May 23:26 +0200
control message for bug #33318
(address . control@debbugs.gnu.org)
86czg98b9z.fsf@gmail.com
tags 33318 + moreinfo
quit
Z
Z
zimoun wrote on 23 Jun 10:22 +0200
Re: bug#33318: [PATCH] doc: Add cindex & command to show versions of dependencies
(address . 33318-done@debbugs.gnu.org)
86o7yj94c1.fsf_-_@gmail.com
Hi,

On Thu, 07 Apr 2022 at 13:45, zimoun <zimon.toutoune@gmail.com> wrote:
Toggle quote (2 lines)
> On Thu, 08 Nov 2018 at 17:00, swedebugia <swedebugia@riseup.net> wrote:

[...]

Toggle quote (6 lines)
>> +@example
>> +guile --version | head -1 && guix --version | head -1 && \
>> +gcc --version | head -1 && echo -n ``libgcrypt `` && \
>> +libgcrypt-config --version && make --version | head -1
>> +@end example

[...]

Toggle quote (2 lines)
> I propose to close this submission without applying. WDYT?

Without more info since couple of months, I am closing.


Cheers,
simon
Closed
?
Your comment

This issue is archived.

To comment on this conversation send email to 33318@debbugs.gnu.org