Hi!
guix-comm...@gnu.org skribis:
> commit d5f344c987c8cc7b597e938c22e02edf1c4335f3
> Author: Tobias Geerinckx-Rice <m...@tobias.gr>
> AuthorDate: Fri Feb 21 05:19:27 2020 +0100
>
> gnu: libreoffice: Fix ‘soffice’ in a pure environment.
>
> * gnu/packages/libreoffice.scm (libreoffice)[arguments]: Refer to grep
> and coreutils by absolute file name in the soffice launcher script.
[...]
> + ;; Use store references for strictly necessary commands,
> + ;; but not for optional tools like ‘gdb’ and ‘valgrind’.
> + (for-each (lambda (command)
> + (substitute* "desktop/scripts/soffice.sh"
> + (((format #f"~a " command))
> + (format #f "~a " (which command)))))
> + (list "dirname" "grep" "uname"))
I strongly encourage using literal strings as patterns, it’s more
robust. In this case, you could replace the ‘for-each’ expression with
something like (untested):
(substitute* "desktop/scripts/soffice.sh"
(("\\<(dirname|grep|uname)\\>" _ cmomand)
(which command)))
WDYT?
Ludo’.
