unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#71695: guix-install.sh REQUIRE check for SYSV_INIT_REQUIRE
@ 2024-06-21 10:15 Nyi Nyi Zaw
  2024-06-22 13:12 ` Richard Sent
  0 siblings, 1 reply; 2+ messages in thread
From: Nyi Nyi Zaw @ 2024-06-21 10:15 UTC (permalink / raw)
  To: 71695

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

Cause>>

>
>   MX linux 23.3(Debian 12.5 “bookworm” base) return sysv-init as 
> lowercase with dash
>   which cause install script fails with sysv-init_REQUIRE[@]: invalid 
> variable name
>

ERROR>>

> https://www.gnu.org/software/guix/
> This script installs GNU Guix on your system
> To uninstall, pass in the '--uninstall' parameter.
> Press return to continue...
> [1718953785.295]: Starting installation (Fri Jun 21 01:39:45 PM +0630 
> 2024)
> [1718953785.306]: [ INFO ] init system is: sysv-init
> ./guix-install.sh: line 164: sysv-init_REQUIRE[@]: invalid variable name

Fix Patch >> Convert sysv-init to SYSV_INIT_REQUIRE to match with 
variable name

>   --- guix-install.sh     2024-06-21 15:22:03.000000000 +0630
>   +++ guix-install-patched.sh     2024-06-21 16:09:17.454668652 +0630
>   @@ -157,7 +157,9 @@
>
>   add_init_sys_require()
>   { # Add the elements of FOO_INIT_SYS to REQUIRE
>       -    local init_require="${INIT_SYS}_REQUIRE[@]"
>       + # Convert INIT_SYS return string to uppercase and replace dash 
> with underscore
>       +    local init_require="${INIT_SYS^^}_REQUIRE[@]"
>       + init_require="${init_require//-/_}"
> if[[ !-z "$init_require"]]; then
> # Have to add piecemeal because ${!foo[@]} performs direct array key
> # expansion, not indirect plain array expansion.



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

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

* bug#71695: guix-install.sh REQUIRE check for SYSV_INIT_REQUIRE
  2024-06-21 10:15 bug#71695: guix-install.sh REQUIRE check for SYSV_INIT_REQUIRE Nyi Nyi Zaw
@ 2024-06-22 13:12 ` Richard Sent
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Sent @ 2024-06-22 13:12 UTC (permalink / raw)
  To: Nyi Nyi Zaw; +Cc: 71695

The fix looks good to me. I don't have commit access so you may want to
send it off to guix-patches so it's not lost.
(info "(guix) Submitting Patches")

If you can, please submit in plaintext. The formatting here looks odd. :)

-- 
Take it easy,
Richard Sent
Making my computer weirder one commit at a time.




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

end of thread, other threads:[~2024-06-22 13:13 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-21 10:15 bug#71695: guix-install.sh REQUIRE check for SYSV_INIT_REQUIRE Nyi Nyi Zaw
2024-06-22 13:12 ` Richard Sent

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).