unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#41254] [PATCH] gnu: boost: Add which to native-inputs.
@ 2020-05-14 10:13 Vincent Legoll
  2020-05-14 11:23 ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: Vincent Legoll @ 2020-05-14 10:13 UTC (permalink / raw)
  To: 41254; +Cc: Vincent Legoll

To fix:
./bootstrap.sh: line 28: which: command not found

* gnu/packages/boost.scm (boost)[native-inputs]: Add which.
---
 gnu/packages/boost.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/boost.scm b/gnu/packages/boost.scm
index 2f2ca289ab..f6f442c5bf 100644
--- a/gnu/packages/boost.scm
+++ b/gnu/packages/boost.scm
@@ -38,6 +38,7 @@
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system trivial)
   #:use-module (gnu packages)
+  #:use-module (gnu packages base)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages icu4c)
   #:use-module (gnu packages perl)
@@ -82,7 +83,8 @@
        ,@(if (%current-target-system)
              '()
              `(("python" ,python-wrapper)))
-       ("tcsh" ,tcsh)))
+       ("tcsh" ,tcsh)
+       ("which" ,which)))
     (arguments
      `(#:imported-modules ((guix build python-build-system)
                            ,@%gnu-build-system-modules)
-- 
2.26.2





^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [bug#41254] [PATCH] gnu: boost: Add which to native-inputs.
  2020-05-14 10:13 [bug#41254] [PATCH] gnu: boost: Add which to native-inputs Vincent Legoll
@ 2020-05-14 11:23 ` Tobias Geerinckx-Rice via Guix-patches via
  2020-05-14 11:37   ` Vincent Legoll
  0 siblings, 1 reply; 4+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-14 11:23 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: 41254

[-- Attachment #1: Type: text/plain, Size: 391 bytes --]

Vincent,

Thanks for the patch!

Vincent Legoll 写道:
> To fix:
> ./bootstrap.sh: line 28: which: command not found

I'm not convinced that increasing the number of inputs to silence 
(harmless) warnings in the build logs is a way to go.

Is there another reason to add ‘which’?  The resulting boost 
appears identical to one built without it.

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#41254] [PATCH] gnu: boost: Add which to native-inputs.
  2020-05-14 11:23 ` Tobias Geerinckx-Rice via Guix-patches via
@ 2020-05-14 11:37   ` Vincent Legoll
  2020-05-14 12:13     ` bug#41254: " Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: Vincent Legoll @ 2020-05-14 11:37 UTC (permalink / raw)
  To: Tobias Geerinckx-Rice; +Cc: 41254

On 14/05/2020 13:23, Tobias Geerinckx-Rice wrote:
> I'm not convinced that increasing the number of inputs to silence 
> (harmless) warnings in the build logs is a way to go.

It did not look harmless to me...

 >  The resulting boost appears identical to one built without it.

...but it looks like I'm wrong.

> Is there another reason to add ‘which’? 

Nope, feel free to close the issue

-- 
Vincent Legoll




^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#41254: [PATCH] gnu: boost: Add which to native-inputs.
  2020-05-14 11:37   ` Vincent Legoll
@ 2020-05-14 12:13     ` Tobias Geerinckx-Rice via Guix-patches via
  0 siblings, 0 replies; 4+ messages in thread
From: Tobias Geerinckx-Rice via Guix-patches via @ 2020-05-14 12:13 UTC (permalink / raw)
  To: Vincent Legoll; +Cc: 41254-done

[-- Attachment #1: Type: text/plain, Size: 497 bytes --]

Vincent Legoll 写道:
> On 14/05/2020 13:23, Tobias Geerinckx-Rice wrote:
>> I'm not convinced that increasing the number of inputs to 
>> silence
>> (harmless) warnings in the build logs is a way to go.
>
> It did not look harmless to me...

I understand.  It's actually quite common (as these things go) but 
tends to go unnoticed exactly because it's harmless.  Once you 
notice it you see it everywhere.

> Nope, feel free to close the issue

Done!

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-05-14 12:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-14 10:13 [bug#41254] [PATCH] gnu: boost: Add which to native-inputs Vincent Legoll
2020-05-14 11:23 ` Tobias Geerinckx-Rice via Guix-patches via
2020-05-14 11:37   ` Vincent Legoll
2020-05-14 12:13     ` bug#41254: " Tobias Geerinckx-Rice via Guix-patches via

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).