[bug#60079] [PATCH] guix: gexp: Clarify what output can be in gexp module commentary.

[bug#60079] [PATCH] guix: gexp: Clarify what output can be in gexp module commentary.

[jgart via Guix-patches via]

* guix/gexp.scm (Commentary): Clarify comment.

Hi Ludo et al,

WDYT

all best,

jgart
---
 guix/gexp.scm | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/guix/gexp.scm b/guix/gexp.scm
index 5f92174a2c..33d0d8487d 100644
--- a/guix/gexp.scm
+++ b/guix/gexp.scm
@@ -139,10 +139,11 @@ (define-module (guix gexp)
 ;;; This module implements "G-expressions", or "gexps".  Gexps are like
 ;;; S-expressions (sexps), with two differences:
 ;;;
-;;;   1. References (un-quotations) to derivations or packages in a gexp are
-;;;      replaced by the corresponding output file name; in addition, the
-;;;      'ungexp-native' unquote-like form allows code to explicitly refer to
-;;;      the native code of a given package, in case of cross-compilation;
+;;;   1. References (un-quotations) to derivations or packages in a gexp
+;;;   are replaced by the corresponding output store path or file name;
+;;;   in addition, the 'ungexp-native' unquote-like form allows code to
+;;;   explicitly refer to the native code of a given package, in case
+;;;   of cross-compilation;
 ;;;
 ;;;   2. Gexps embed information about the derivations they refer to.
 ;;;
-- 
2.38.1

[bug#60079] [PATCH] guix: gexp: Clarify what output can be in gexp module commentary.

[Ludovic Courtès]

Hi jgart,

jgart <jgart@dismail.de> skribis:

> --- a/guix/gexp.scm
> +++ b/guix/gexp.scm
> @@ -139,10 +139,11 @@ (define-module (guix gexp)
>  ;;; This module implements "G-expressions", or "gexps".  Gexps are like
>  ;;; S-expressions (sexps), with two differences:
>  ;;;
> -;;;   1. References (un-quotations) to derivations or packages in a gexp are
> -;;;      replaced by the corresponding output file name; in addition, the
> -;;;      'ungexp-native' unquote-like form allows code to explicitly refer to
> -;;;      the native code of a given package, in case of cross-compilation;
> +;;;   1. References (un-quotations) to derivations or packages in a gexp
> +;;;   are replaced by the corresponding output store path or file name;
> +;;;   in addition, the 'ungexp-native' unquote-like form allows code to
> +;;;   explicitly refer to the native code of a given package, in case
> +;;;   of cross-compilation;

I prefer not to apply this patch: it worsens indentation and adds the
words “store path”, which AFAICS bring nothing.

Thanks anyway for taking a look,
Ludo’.

[bug#60079] [PATCH] guix: gexp: Clarify what output can be in gexp module commentary.

[zimoun]

Hi,

On Wed, 21 Dec 2022 at 14:31, Ludovic Courtès <ludo@gnu.org> wrote:

>> --- a/guix/gexp.scm
>> +++ b/guix/gexp.scm
>> @@ -139,10 +139,11 @@ (define-module (guix gexp)
>>  ;;; This module implements "G-expressions", or "gexps".  Gexps are like
>>  ;;; S-expressions (sexps), with two differences:
>>  ;;;
>> -;;;   1. References (un-quotations) to derivations or packages in a gexp are
>> -;;;      replaced by the corresponding output file name; in addition, the
>> -;;;      'ungexp-native' unquote-like form allows code to explicitly refer to
>> -;;;      the native code of a given package, in case of cross-compilation;
>> +;;;   1. References (un-quotations) to derivations or packages in a gexp
>> +;;;   are replaced by the corresponding output store path or file name;
>> +;;;   in addition, the 'ungexp-native' unquote-like form allows code to
>> +;;;   explicitly refer to the native code of a given package, in case
>> +;;;   of cross-compilation;
>
> I prefer not to apply this patch: it worsens indentation and adds the
> words “store path”, which AFAICS bring nothing.

I agree.  Closing?

> Thanks anyway for taking a look,

Thanks.

Cheers,
simon

[bug#60079] [PATCH] guix: gexp: Clarify what output can be in gexp module commentary.

[jgart via Guix-patches via]

Sure, feel free to close this one. Thanks for the review.

all best,

jgart