unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#71580] [PATCH] doc: Add message for common error about make check-system
@ 2024-06-15 21:42 Richard Sent
  2024-06-17 21:07 ` Maxim Cournoyer
  2024-06-18  1:02 ` [bug#71580] [PATCH v2] " Richard Sent
  0 siblings, 2 replies; 6+ messages in thread
From: Richard Sent @ 2024-06-15 21:42 UTC (permalink / raw)
  To: 71580
  Cc: Richard Sent, Florian Pelz, Ludovic Courtès,
	Matthew Trzcinski, Maxim Cournoyer

* doc/contributing.texi (Running the Test Suite): Mention that make clean-go
may need to be run before running make check-system when previous builds were
run with different work trees. See https://issues.guix.gnu.org/47573.

Change-Id: I4b68d3a05f1425505816db969284487d725840d6
---
Sending a followup rebased on current master.

 doc/contributing.texi | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index 938c8bfdb1..c9591dc9bf 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -411,6 +411,20 @@ Running the Test Suite
 substitutes are available for their dependencies (@pxref{Substitutes}).
 Some of them require a lot of storage space to hold VM images.
 
+If you encounter an error like:
+
+@example
+Compiling Scheme modules...
+ice-9/eval.scm:142:16: In procedure compile-top-call:
+error: all-system-tests: unbound variable
+hint: Did you forget `(use-modules (gnu tests))'?
+@end example
+
+@noindent
+there may be inconsistency in the work tree from previous builds.  To
+resolve this, try running @command{make clean-go} followed by
+@command{make}.
+
 Again in case of test failures, please send @email{bug-guix@@gnu.org}
 all the details.
 

base-commit: 612e4dd98f7d1d015e405af9d029bede3fe3c280
-- 
2.45.1





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

* [bug#71580] [PATCH] doc: Add message for common error about make check-system
  2024-06-15 21:42 [bug#71580] [PATCH] doc: Add message for common error about make check-system Richard Sent
@ 2024-06-17 21:07 ` Maxim Cournoyer
  2024-06-18  1:02 ` [bug#71580] [PATCH v2] " Richard Sent
  1 sibling, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-17 21:07 UTC (permalink / raw)
  To: Richard Sent; +Cc: Ludovic Courtès, Florian Pelz, 71580, Matthew Trzcinski

Hi Richard,

Richard Sent <richard@freakingpenguin.com> writes:

> * doc/contributing.texi (Running the Test Suite): Mention that make clean-go
> may need to be run before running make check-system when previous builds were
> run with different work trees. See https://issues.guix.gnu.org/47573.
>
> Change-Id: I4b68d3a05f1425505816db969284487d725840d6
> ---
> Sending a followup rebased on current master.
>
>  doc/contributing.texi | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
>
> diff --git a/doc/contributing.texi b/doc/contributing.texi
> index 938c8bfdb1..c9591dc9bf 100644
> --- a/doc/contributing.texi
> +++ b/doc/contributing.texi
> @@ -411,6 +411,20 @@ Running the Test Suite
>  substitutes are available for their dependencies (@pxref{Substitutes}).
>  Some of them require a lot of storage space to hold VM images.
>  
> +If you encounter an error like:
> +
> +@example
> +Compiling Scheme modules...
> +ice-9/eval.scm:142:16: In procedure compile-top-call:
> +error: all-system-tests: unbound variable
> +hint: Did you forget `(use-modules (gnu tests))'?
> +@end example
> +
> +@noindent
> +there may be inconsistency in the work tree from previous builds.  To

perhaps        ^ an inconsistency?  alternatively, 'inconsistencies'.

> +resolve this, try running @command{make clean-go} followed by
> +@command{make}.
> +
>  Again in case of test failures, please send @email{bug-guix@@gnu.org}
>  all the details.

I guess the fact that there is an open issue from a different perhaps
about this problem on the tracker means it's bit enough people that it
merits to be mentioned in the doc.

Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

-- 
Thanks,
Maxim




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

* [bug#71580] [PATCH v2] doc: Add message for common error about make check-system
  2024-06-15 21:42 [bug#71580] [PATCH] doc: Add message for common error about make check-system Richard Sent
  2024-06-17 21:07 ` Maxim Cournoyer
@ 2024-06-18  1:02 ` Richard Sent
  2024-06-18 12:13   ` Maxim Cournoyer
  1 sibling, 1 reply; 6+ messages in thread
From: Richard Sent @ 2024-06-18  1:02 UTC (permalink / raw)
  To: 71580
  Cc: Richard Sent, Florian Pelz, Ludovic Courtès,
	Matthew Trzcinski, Maxim Cournoyer

* doc/contributing.texi (Running the Test Suite): Mention that make clean-go
may need to be run before running make check-system when previous builds were
run with different work trees. See https://issues.guix.gnu.org/47573.

Change-Id: I4b68d3a05f1425505816db969284487d725840d6
---
Adjusted the phrasing. If we think it's best to hold on merging this
until we determine how common of an occurrence this is or (better yet)
resolve the issue entirely, there's no complaints by me. I'll leave
that decision up to you all. :)

 doc/contributing.texi | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/doc/contributing.texi b/doc/contributing.texi
index 938c8bfdb1..6f0acc8c1f 100644
--- a/doc/contributing.texi
+++ b/doc/contributing.texi
@@ -411,6 +411,20 @@ Running the Test Suite
 substitutes are available for their dependencies (@pxref{Substitutes}).
 Some of them require a lot of storage space to hold VM images.
 
+If you encounter an error like:
+
+@example
+Compiling Scheme modules...
+ice-9/eval.scm:142:16: In procedure compile-top-call:
+error: all-system-tests: unbound variable
+hint: Did you forget `(use-modules (gnu tests))'?
+@end example
+
+@noindent
+there may be inconsistencies in the work tree from previous builds.  To
+resolve this, try running @command{make clean-go} followed by
+@command{make}.
+
 Again in case of test failures, please send @email{bug-guix@@gnu.org}
 all the details.
 

base-commit: b993f4735d41e690dbafb8ee2e17fce996a8cf20
-- 
2.45.1





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

* [bug#71580] [PATCH v2] doc: Add message for common error about make check-system
  2024-06-18  1:02 ` [bug#71580] [PATCH v2] " Richard Sent
@ 2024-06-18 12:13   ` Maxim Cournoyer
  2024-06-19  8:48     ` pelzflorian
  2024-06-24  4:09     ` bug#71580: " Maxim Cournoyer
  0 siblings, 2 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-18 12:13 UTC (permalink / raw)
  To: Richard Sent; +Cc: Ludovic Courtès, Florian Pelz, 71580, Matthew Trzcinski

Richard Sent <richard@freakingpenguin.com> writes:

> * doc/contributing.texi (Running the Test Suite): Mention that make clean-go
> may need to be run before running make check-system when previous builds were
> run with different work trees. See https://issues.guix.gnu.org/47573.
>
> Change-Id: I4b68d3a05f1425505816db969284487d725840d6

Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

-- 
Thanks,
Maxim




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

* [bug#71580] [PATCH v2] doc: Add message for common error about make check-system
  2024-06-18 12:13   ` Maxim Cournoyer
@ 2024-06-19  8:48     ` pelzflorian
  2024-06-24  4:09     ` bug#71580: " Maxim Cournoyer
  1 sibling, 0 replies; 6+ messages in thread
From: pelzflorian @ 2024-06-19  8:48 UTC (permalink / raw)
  To: Maxim Cournoyer
  Cc: Richard Sent, Ludovic Courtès, 71580, Matthew Trzcinski

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


I cannot push; my computer broke; but this patch should not be stuck because of me.  However, it is not clear why `make clean-go' helps.  I tried to switch branches, but cannot reproduce the issue and do not get this kind of error message.

Regards,
Florian
 

[-- Attachment #2: Type: text/html, Size: 309 bytes --]

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

* bug#71580: [PATCH v2] doc: Add message for common error about make check-system
  2024-06-18 12:13   ` Maxim Cournoyer
  2024-06-19  8:48     ` pelzflorian
@ 2024-06-24  4:09     ` Maxim Cournoyer
  1 sibling, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-24  4:09 UTC (permalink / raw)
  To: Richard Sent
  Cc: Ludovic Courtès, Florian Pelz, 71580-done, Matthew Trzcinski

Hi,

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

> Richard Sent <richard@freakingpenguin.com> writes:
>
>> * doc/contributing.texi (Running the Test Suite): Mention that make clean-go
>> may need to be run before running make check-system when previous builds were
>> run with different work trees. See https://issues.guix.gnu.org/47573.
>>
>> Change-Id: I4b68d3a05f1425505816db969284487d725840d6
>
> Reviewed-by: Maxim Cournoyer <maxim.cournoyer@gmail>

Applied locally.  Will push shortly.

-- 
Thanks,
Maxim




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

end of thread, other threads:[~2024-06-24  4:14 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-15 21:42 [bug#71580] [PATCH] doc: Add message for common error about make check-system Richard Sent
2024-06-17 21:07 ` Maxim Cournoyer
2024-06-18  1:02 ` [bug#71580] [PATCH v2] " Richard Sent
2024-06-18 12:13   ` Maxim Cournoyer
2024-06-19  8:48     ` pelzflorian
2024-06-24  4:09     ` bug#71580: " Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).