GNU bug report logs - #73682
[PATCH 0/3] gnu: varnish: Update to 7.6.0. [security fixes]

Previous Next

Package: guix-patches;

Reported by: Nicolas Graves <ngraves <at> ngraves.fr>

Date: Mon, 7 Oct 2024 21:56:02 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 73682 <at> debbugs.gnu.org
Subject: [bug#73682] [PATCH 3/3] gnu: varnish: Improve style.
Date: Tue, 15 Oct 2024 18:25:55 +0200
Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> * gnu/packages/web.scm (varnish): Improve style.
>   [arguments]: Rewrite using gexps.
>   [inputs]: Replace coreutils by coreutils-minimal, python by
>   python-minimal. Add glibc.

[...]

> +             ;; Varnish uses GCC to compile VCL, so wrap it with required GCC
> +             ;; environment variables to avoid propagating them to profiles.
> +             (lambda _
> +               (wrap-program (string-append #$output "/sbin/varnishd")
>                   ;; Add binutils to PATH so gcc finds the 'as' executable.
> -                 `("PATH" ":" prefix (,PATH))
> +                 `("PATH" ":" prefix (,(dirname (which "as"))))
>                   ;; Make sure 'crti.o' et.al is found.
> -                 `("LIBRARY_PATH" ":" prefix (,LIBRARY_PATH)))))))))
> +                 `("LIBRARY_PATH" ":" prefix
> +                    (,#$(file-append (this-package-input "glibc") "/lib"))))))))))

If we assume we’ll never want to cross-compile Varnish (sounds
reasonable to me), use:

  (dirname (search-input-file inputs "/lib/libc.so"))

and do *not* add ‘glibc’ as an input, as this effectively adds a
dependency on a second glibc.

Ludo’.




This bug report was last modified 212 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.