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

Previous Next

Full log

View this message in rfc822 format

From: Theodoros Foradis <theodoros <at> foradis.org>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 27344 <at> debbugs.gnu.org
Subject: [bug#27344] [PATCH v4 3/3] gnu: Add qucs-s.
Date: Wed, 08 Nov 2017 19:42:12 +0200

Hello Ludovic,

Ludovic Courtès writes:

> Theodoros Foradis <theodoros <at> foradis.org> skribis:
>
>> * gnu/packages/engineering.scm (qucs-s): New variable.
>
> There seems to be a lot of duplication with ‘qucs’.  Would it be
> possible to inherit from qucs, thereby reusing its phases, and simply
> provide any extra phase that’s needed?

Some paths are different in those two packages. For example:
Qucs:
> (substitute* '("qucs/qucs/qucsdigi"
>                "qucs/qucs/qucsdigilib"
>                "qucs/qucs/qucsveri")

> (wrap-program (string-append out "/bin/qucs")

Qucs-s:
> (substitute* '("qucs/qucsdigi"
>                "qucs/qucsdigilib"
>                "qucs/qucsveri")

> (let* ((out (assoc-ref outputs "out"))
>        (file (string-append out "/bin/qucs-s"))
>        (qucs (assoc-ref inputs "qucs"))
>        (qucsator (string-append qucs "/bin/qucsator")))
>   (wrap-program file

So qucs-s cannot inherit qucs, and this is why I dismissed that when I
looked at it in the past.

I think we could create a procedure, that takes the variables as
arguments and returns a qucs package. So that qucs-s inherits the
application of that procedure on different arguments.

Is this preferable to having two different packages with some identical
phases?

-- 
Theodoros Foradis

Previous Next

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