unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: tbb: Add rpath to LDFLAGS.
@ 2015-04-27  9:23 Ricardo Wurmus
  2015-04-27 14:21 ` Andreas Enge
  0 siblings, 1 reply; 2+ messages in thread
From: Ricardo Wurmus @ 2015-04-27  9:23 UTC (permalink / raw)
  To: guix-devel

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

Hi Guix,

the attached patch is required to fix the build of tbb.  When tests are
enabled, some additional binaries are built whose rpath is not properly
set.  This patch sets the rpath in LDFLAGS, thereby solving this
problem.

~~ Ricardo


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-tbb-Set-rpath-in-LDFLAGS.patch --]
[-- Type: text/x-patch, Size: 889 bytes --]

From de82a09eaf5037287cf1740aab7d1ef8f748cda8 Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
Date: Mon, 27 Apr 2015 11:21:08 +0200
Subject: [PATCH] gnu: tbb: Set rpath in LDFLAGS.

* gnu/packages/tbb.scm (tbb)[arguments]: Set rpath to "/lib" output directory
  in LDFLAGS.
---
 gnu/packages/tbb.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/tbb.scm b/gnu/packages/tbb.scm
index 3c41141..5bfbeea 100644
--- a/gnu/packages/tbb.scm
+++ b/gnu/packages/tbb.scm
@@ -46,6 +46,8 @@
     (build-system gnu-build-system)
     (arguments
      `(#:test-target "test"
+       #:make-flags (list (string-append "LDFLAGS = -Wl,-rpath="
+                                         (assoc-ref %outputs "out") "/lib"))
        #:phases (alist-replace
                  'configure
                  (lambda* (#:key outputs #:allow-other-keys)
-- 
2.1.0


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

* Re: [PATCH] gnu: tbb: Add rpath to LDFLAGS.
  2015-04-27  9:23 [PATCH] gnu: tbb: Add rpath to LDFLAGS Ricardo Wurmus
@ 2015-04-27 14:21 ` Andreas Enge
  0 siblings, 0 replies; 2+ messages in thread
From: Andreas Enge @ 2015-04-27 14:21 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

On Mon, Apr 27, 2015 at 11:23:39AM +0200, Ricardo Wurmus wrote:
> the attached patch is required to fix the build of tbb.

Looks good!

Andreas

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

end of thread, other threads:[~2015-04-27 14:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-27  9:23 [PATCH] gnu: tbb: Add rpath to LDFLAGS Ricardo Wurmus
2015-04-27 14:21 ` Andreas Enge

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).