From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:54097)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1e2CFH-0006z0-2Q
	for guix-patches@gnu.org; Wed, 11 Oct 2017 04:20:08 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1e2CFG-0007gd-AD
	for guix-patches@gnu.org; Wed, 11 Oct 2017 04:20:03 -0400
Received: from debbugs.gnu.org ([208.118.235.43]:52534)
	by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)
	(Exim 4.71) (envelope-from <Debian-debbugs@debbugs.gnu.org>)
	id 1e2CFG-0007gU-6S
	for guix-patches@gnu.org; Wed, 11 Oct 2017 04:20:02 -0400
Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2)
	(envelope-from <Debian-debbugs@debbugs.gnu.org>) id 1e2CFG-00021L-0T
	for guix-patches@gnu.org; Wed, 11 Oct 2017 04:20:02 -0400
Subject: [bug#27344] [PATCH v3 6/8] gnu: Add freehdl.
Resent-Message-ID: <handler.27344.B27344.15077099777729@debbugs.gnu.org>
From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=)
References: <87lglnery8.fsf@foradis.org>
	<20170909175752.8566-1-theodoros@foradis.org>
	<20170909175752.8566-6-theodoros@foradis.org>
Date: Wed, 11 Oct 2017 10:19:27 +0200
In-Reply-To: <20170909175752.8566-6-theodoros@foradis.org> (Theodoros
	Foradis's message of "Sat, 9 Sep 2017 20:57:50 +0300")
Message-ID: <87sheqcd0w.fsf@gnu.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
List-Id: <guix-patches.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-patches/>
List-Post: <mailto:guix-patches@gnu.org>
List-Help: <mailto:guix-patches-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-patches>,
	<mailto:guix-patches-request@gnu.org?subject=subscribe>
Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org
Sender: "Guix-patches" <guix-patches-bounces+kyle=kyleam.com@gnu.org>
To: Theodoros Foradis <theodoros@foradis.org>
Cc: 27344@debbugs.gnu.org

Theodoros Foradis <theodoros@foradis.org> skribis:

> * gnu/packages/engineering.scm (freehdl): New variable.

I=E2=80=99ve applied the patch.  \o/

I have one question for which I couldn=E2=80=99t find an answer in the prev=
ious
discussions:

> +         (add-after 'configure 'patch-freehdl-pc
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "freehdl.pc"
> +               (("=3Dg\\+\\+")
> +                (string-append "=3D" (assoc-ref inputs "gcc")
> +                               "/bin/g++"))
> +               (("=3Dlibtool")
> +                (string-append "=3D" (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 =E2=80=98freehdl.pc=E2=80=99:

  cxx=3Dg++
  libtool=3Dlibtool

Now, whether this will work depends on whether/how users of =E2=80=98freehd=
l.pc=E2=80=99
use these variables.

Could you check whether that would work?

TIA!

Ludo=E2=80=99.