GNU bug report logs - #27344
[PATCH 0/12] Add computational software and circuit simulators

Previous Next

Full log

Message #269 received at 27344 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Theodoros Foradis <theodoros <at> foradis.org>
Cc: 27344 <at> debbugs.gnu.org
Subject: Re: [bug#27344] [PATCH v3 6/8] gnu: Add freehdl.
Date: Fri, 13 Oct 2017 10:21:47 +0200

Theodoros Foradis <theodoros <at> foradis.org> skribis:

> Ludovic Courtès writes:
>
>> Theodoros Foradis <theodoros <at> foradis.org> 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.

Just to be clear: does FreeHDL invoke the C compiler by itself, or is it
up to the user to compile the C code produced by FreeHDL?

In the former case, FreeHDL should definitely keep a reference to the C
compiler; otherwise it wouldn’t work out-of-the-box.  In the latter
case, it doesn’t have to keep a reference to the C compiler.

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

I can do it on your behalf if you confirm that it’s FreeHDL does not
invoke the C compiler directly.

Thank you!

Ludo’.

Previous Next

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