Subject: Re: [bug#77527] [PATCH v4 5/6] guix-install.sh: Group profile sourcing.

From: 宋文武 <[email-masked]>
Date: Sat, 19 Apr 2025 10:45:22 +0800

Hilton Chain <[email-masked]> writes:

> [...]
> +  if [ \"$GUIX_PROFILE\" = \"$HOME/.config/guix/current\" ]; then
> +    # Expose the latest Guix modules to Guile so guix shell and repls spawned by
> +    # e.g. Geiser work out of the box.
> +    case $GUILE_LOAD_PATH in
> +      *$GUIX_PROFILE/share/guile/site/3.0*) ;;
> +      *) export GUILE_LOAD_PATH=\"$GUIX_PROFILE/share/guile/site/3.0${GUILE_LOAD_PATH:+:}$GUILE_LOAD_PATH\" ;;
> +    esac
> +    case $GUILE_LOAD_COMPILED_PATH in
> +      *$GUIX_PROFILE/lib/guile/3.0/site-ccache*) ;;
> +      *) export
> GUILE_LOAD_COMPILED_PATH=\"$GUIX_PROFILE/lib/guile/3.0/site-ccache${GUILE_LOAD_COMPILED_PATH:+:}$GUILE_LOAD_COMPILED_PATH\"
> ;;
Maybe not needed, since 'guix repl' handle this.

> [...]
> -# Clean up after ourselves.
> -unset GUIX_PROFILE

I think it's still needed, since after the for loop the variable is
still visible.

Otherwise look good to me, thanks!