[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#27344] [PATCH v2 06/12] gnu: Add freehdl.
From: |
Theodoros Foradis |
Subject: |
[bug#27344] [PATCH v2 06/12] gnu: Add freehdl. |
Date: |
Sat, 24 Jun 2017 12:34:46 +0300 |
User-agent: |
mu4e 0.9.18; emacs 25.2.1 |
Hello,
>> + (add-before 'configure 'patch-pkg-config
>> + (lambda _
>> + (substitute* "freehdl/freehdl-config"
>> + (("pkg-config") (which "pkg-config"))
>> + (("cat") (which "cat")))
>> + #t))
>
> freehdl-config is installed, but it uses a path to pkg-config which is from a
> native-input (so it will not be retained).
>
> Does this ever work again after removing pkg-config from your profile and
> then doing "guix gc" ? I doubt it...
>
>> + (add-after 'configure 'patch-freehdl-pc
>> + (lambda _
>> + (substitute* "freehdl.pc"
>> + (("=g\\+\\+") (string-append "=" (which "g++")))
>> + (("=libtool") (string-append "=" (which "libtool"))))
>> + #t))
>
> Same here...
>
>> + (add-after 'install-scripts 'make-wrapper
>> + (lambda* (#:key outputs #:allow-other-keys)
>> + (let ((out (assoc-ref outputs "out")))
>> + (wrap-program (string-append out "/bin/freehdl-config")
>> + `("PKG_CONFIG_PATH" ":" prefix (,(string-append out
>> "/lib/pkgconfig")))))
>
>
>> + #t)))))
>> + (inputs
>> + `(("perl" ,perl)))
>> + (native-inputs
>> + `(("pkg-config" ,pkg-config)
>> + ("libtool" ,libtool)))
>
>
> I'm not sure how to proceed so I'll stop at this point (with freehdl not in)
> until the path is clear :)
You are correct, I totally missed the issue there. I can think of two
possible ways to solve the issue, though I may be making wrong
assumptions.
- Move pkg-config to inputs, and disable cross-compilation of the
package, if possible.
- Have both pkg-configs at build time. Use the native pkg-config in
building and reference the non-native one in freehdl-config. Is this
possible with guix?
--
Theodoros Foradis
- [bug#27344] [PATCH 01/12] gnu: Add harminv., (continued)
- [bug#27344] [PATCH v2 01/12] gnu: Add harminv., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 02/12] gnu: Add guile-libctl., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 05/12] gnu: Add adms., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 03/12] gnu: Add mpb., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 07/12] gnu: Add asco., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 06/12] gnu: Add freehdl., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 04/12] gnu: Add meep., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 10/12] gnu: Add xyce-parallel., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 11/12] gnu: Add qucs., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 09/12] gnu: Add xyce-serial., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 08/12] gnu: Add ngspice., Theodoros Foradis, 2017/06/22
- [bug#27344] [PATCH v2 12/12] gnu: Add qucs-s., Theodoros Foradis, 2017/06/22