guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#27344] [PATCH v3 6/8] gnu: Add freehdl.


From: Ludovic Courtès
Subject: [bug#27344] [PATCH v3 6/8] gnu: Add freehdl.
Date: Wed, 11 Oct 2017 10:19:27 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

Theodoros Foradis <address@hidden> skribis:

> * gnu/packages/engineering.scm (freehdl): New variable.

I’ve applied the patch.  \o/

I have one question for which I couldn’t find an answer in the previous
discussions:

> +         (add-after 'configure 'patch-freehdl-pc
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "freehdl.pc"
> +               (("=g\\+\\+")
> +                (string-append "=" (assoc-ref inputs "gcc")
> +                               "/bin/g++"))
> +               (("=libtool")
> +                (string-append "=" (assoc-ref inputs "libtool")
> +                               "/bin/libtool")))
> +             #t))

The effect of this is to retain a reference to GCC, Libtool, and all
their dependencies, which makes the closure of FreeHDL much bigger.

We could avoid it by just putting this in ‘freehdl.pc’:

  cxx=g++
  libtool=libtool

Now, whether this will work depends on whether/how users of ‘freehdl.pc’
use these variables.

Could you check whether that would work?

TIA!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]