Re: 13/15: gnu: libreoffice: Fix ‘soffice’ in a pure environment.

["Ludovic Courtès" <ludo@gnu.org>]

Hi!

guix-commits@gnu.org skribis:

> commit d5f344c987c8cc7b597e938c22e02edf1c4335f3
> Author: Tobias Geerinckx-Rice <me@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’.