bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-08-25 Thread Josselin Poiret via Bug reports for GNU Guix
Hi everyone,

Simon Tournier  writes:

> Nitpick: Missing double space for sentences.
>
> Otherwise, LGTM.

Pushed as 9f68a2a9c41166ec5ac24c082bcd96c433dd2ede.

Closing.

Best,
-- 
Josselin Poiret


signature.asc
Description: PGP signature


bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-04-13 Thread Skyler via Bug reports for GNU Guix
> The later, I'd say; we're all volunteers :-).

Fair enough!





bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-04-12 Thread Maxim Cournoyer
Hello,

Skyler via Bug reports for GNU Guix  writes:

> Is there a normal timeline for patch review or is it mor of an "as resources 
> allow" type of deal?

The later, I'd say; we're all volunteers :-).

-- 
Thanks,
Maxim





bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-04-12 Thread Simon Tournier
Hi,

On jeu., 30 mars 2023 at 22:42, Josselin Poiret via Bug reports for GNU Guix 
 wrote:

> +Note that it is necessary to include the main @code{python} package in
> +this command even if it is already installed into your environment. This
> +is so that the shell environment knows to set @env{PYTHONPATH} and other
> +related variables. The shell environment cannot check the previously
> +installed environment, because then it would be non-deterministic.  This
> +is true for most libraries: their corresponding language package should
> +be included in the shell invocation.

Nitpick: Missing double space for sentences.

Otherwise, LGTM.


Cheers,
simon






bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-04-12 Thread Josselin Poiret via Bug reports for GNU Guix
Hi Skyler,

Skyler  writes:

> Is there a normal timeline for patch review or is it mor of an "as resources 
> allow" type of deal?

Unfortunately no, there are no guarantees: someone needs to take some
time to look at this, and we can't force anyone to do it.  The fact that
it's in a bug report instead of a patch submission probably doesn't
help, let me see if I can fix this with a debbugs command.

Best,
-- 
Josselin Poiret


signature.asc
Description: PGP signature


bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-04-11 Thread Skyler via Bug reports for GNU Guix
Is there a normal timeline for patch review or is it mor of an "as resources 
allow" type of deal?

--- Original Message ---
On Thursday, March 30th, 2023 at 1:42 PM, Josselin Poiret  
wrote:


> * doc/guix.texi (Invoking guix shell): Explain that python must be included in
> the invocation even if it is already available in the external environment.
> 
> Suggested-by: Skyler skyv...@protonmail.com
> 
> ---
> doc/guix.texi | 10 +-
> 1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/guix.texi b/doc/guix.texi
> index c49e51b72e..879a526b77 100644
> --- a/doc/guix.texi
> +++ b/doc/guix.texi
> @@ -96,7 +96,7 @@ Copyright @copyright{} 2021 Domagoj Stolfa@*
> Copyright @copyright{} 2021 Hui Lu@*
> Copyright @copyright{} 2021 pukkamustard@*
> Copyright @copyright{} 2021 Alice Brenon@*
> -Copyright @copyright{} 2021, 2022 Josselin Poiret@*
> +Copyright @copyright{} 2021, 2022, 2023 Josselin Poiret@*
> Copyright @copyright{} 2021 muradm@*
> Copyright @copyright{} 2021, 2022 Andrew Tropin@*
> Copyright @copyright{} 2021 Sarah Morgensen@*
> @@ -5959,6 +5959,14 @@ building or downloading any missing package, and runs 
> the
> guix shell python python-numpy -- python3
> @end example
> 
> +Note that it is necessary to include the main @code{python} package in
> +this command even if it is already installed into your environment. This
> +is so that the shell environment knows to set @env{PYTHONPATH} and other
> +related variables. The shell environment cannot check the previously
> +installed environment, because then it would be non-deterministic. This
> +is true for most libraries: their corresponding language package should
> +be included in the shell invocation.
> +
> @quotation Note
> @cindex shebang, for @command{guix shell}
> @command{guix shell} can be also be used as a script interpreter, also
> 
> base-commit: be5e280e5fe26f93bd5a6e3f76e4502edb913a94
> --
> 2.39.2





bug#62448: [PATCH] doc: Note that `guix shell` should contain base language packages.

2023-03-30 Thread Josselin Poiret via Bug reports for GNU Guix
* doc/guix.texi (Invoking guix shell): Explain that python must be included in
the invocation even if it is already available in the external environment.

Suggested-by: Skyler 
---
 doc/guix.texi | 10 +-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index c49e51b72e..879a526b77 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -96,7 +96,7 @@ Copyright @copyright{} 2021 Domagoj Stolfa@*
 Copyright @copyright{} 2021 Hui Lu@*
 Copyright @copyright{} 2021 pukkamustard@*
 Copyright @copyright{} 2021 Alice Brenon@*
-Copyright @copyright{} 2021, 2022 Josselin Poiret@*
+Copyright @copyright{} 2021, 2022, 2023 Josselin Poiret@*
 Copyright @copyright{} 2021 muradm@*
 Copyright @copyright{} 2021, 2022 Andrew Tropin@*
 Copyright @copyright{} 2021 Sarah Morgensen@*
@@ -5959,6 +5959,14 @@ building or downloading any missing package, and runs the
 guix shell python python-numpy -- python3
 @end example
 
+Note that it is necessary to include the main @code{python} package in
+this command even if it is already installed into your environment. This
+is so that the shell environment knows to set @env{PYTHONPATH} and other
+related variables. The shell environment cannot check the previously
+installed environment, because then it would be non-deterministic.  This
+is true for most libraries: their corresponding language package should
+be included in the shell invocation.
+
 @quotation Note
 @cindex shebang, for @command{guix shell}
 @command{guix shell} can be also be used as a script interpreter, also

base-commit: be5e280e5fe26f93bd5a6e3f76e4502edb913a94
-- 
2.39.2