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

Previous Next

Package: guix-patches;

Reported by: Theodoros Foradis <theodoros.for <at> openmailbox.org>

Date: Mon, 12 Jun 2017 15:27:03 UTC

Severity: normal

Tags: patch

Done: ludo <at> gnu.org (Ludovic Courtès)

Bug is archived. No further changes may be made.

Full log


Message #254 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: Wed, 11 Oct 2017 10:19:27 +0200
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?

TIA!

Ludo’.




This bug report was last modified 7 years and 195 days ago.

Previous Next


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