[Top][All Lists]

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

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

From: Theodoros Foradis
Subject: [bug#27344] [PATCH v3 6/8] gnu: Add freehdl.
Date: Thu, 12 Oct 2017 20:00:57 +0300

Ludovic Courtès writes:

> 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?

Yes, I have tested this and it should work. Freehdl is a VHDL to C
compiler, so the user should be responsible for having a gcc toolchain
in his environment to run the simulations.

Now, in regard to the qucs and qucs-s packages, for the VHDL simulation
to work, libtool should be propagated (?) as well, but I'll reply to
that patch separately.

We can remove this phase (patch-freehdl-pc) altogether. Shall I submit
this patch?

Theodoros Foradis

reply via email to

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