guix-patches
[Top][All Lists]
Advanced

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

[bug#27344] [PATCH v3 7/8] gnu: Add qucs.


From: Theodoros Foradis
Subject: [bug#27344] [PATCH v3 7/8] gnu: Add qucs.
Date: Thu, 12 Oct 2017 20:07:26 +0300

Hello,

Ludovic Courtès writes:

> Hi,
>
> Theodoros Foradis <address@hidden> skribis:
>
>> * gnu/packages/engineering.scm (qucs): New variable.
>
> [...]
>
>> +    (propagated-inputs
>> +     `(("gcc-toolchain" ,gcc-toolchain-5))) ; for freehdl
>
> What’s the reason for this?
>
> Propagated inputs should be used sparsely because they “pollute” user
> profiles.  Here we’d be forcing a big package, and a specific version of
> that, in user profiles, which is not OK.
>
> Apart from that the patch LGTM though, so if we can remove these two
> lines, we can commit.

Freehdl is a VHDL to C compiler, so for qucs and qucs-s to be able to do
VHDL simulations, there should be a gcc toolchain and libtool in the
user environment (libtool should be propagated as well, after the phase
removal from the freehdl package).

I think it *should* be OK to remove gcc-toolchain from the
propagated-inputs, if we provide the user with some kind of message,
that they should add a gcc-toolchain and libtool in their environment if
they want to do VHDL simulations.

Maybe it should be preferable to add a message along those lines in the
package descriptions of those 2 packages (qucs and qucs-s)?

WDYT?

-- 
Theodoros Foradis





reply via email to

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