Bug 1004751 - glusterfs-api.pc.in contains an rpath
Summary: glusterfs-api.pc.in contains an rpath
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: libgfapi
Version: 3.4.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Kaleb KEITHLEY
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On: 1002220
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-05 12:09 UTC by Kaleb KEITHLEY
Modified: 2015-12-01 16:45 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 1002220
Environment:
Last Closed: 2014-01-17 16:37:27 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Kaleb KEITHLEY 2013-09-05 12:09:29 UTC 
+++ This bug was initially created as a clone of Bug #1002220 +++

Description of problem:

pkg-config --libs glusterfs-api
-Wl,-R/usr/lib64 -lgfapi -lglusterfs -lgfrpc -lgfxdr

Shouldn' t contain an rpath.

--- Additional comment from Anand Avati on 2013-08-28 13:31:09 EDT ---

REVIEW: http://review.gluster.org/5717 (api: glusterfs-api.pc.in (and libgfchangelog.pc.in) use rpath (-Wl,-R...)) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle@redhat.com)

--- Additional comment from Kaleb KEITHLEY on 2013-08-28 14:40:48 EDT ---

Note that the library itself does not have an RPATH. Only the package config erroneously reports that there's an RPATH.

--- Additional comment from Anand Avati on 2013-08-28 18:24:31 EDT ---

COMMIT: http://review.gluster.org/5717 committed in master by Anand Avati (avati@redhat.com) 
------
commit 8fcf91b51ccd457d417e71c401f6e6af1dffc607
Author: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Date:   Wed Aug 28 13:28:29 2013 -0400

    api: glusterfs-api.pc.in (and libgfchangelog.pc.in) use rpath (-Wl,-R...)
    
    Change-Id: Icdd23f1c31421c882db106cdad1b1406569594ca
    BUG: 1002220
    Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
    Reviewed-on: http://review.gluster.org/5717
    Tested-by: Gluster Build System <jenkins@build.gluster.com>
    Reviewed-by: Anand Avati <avati@redhat.com>
Comment 1 Anand Avati 2013-09-05 13:59:13 UTC 
REVIEW: http://review.gluster.org/5822 (api: glusterfs-api.pc.in use rpath (-Wl,-R...) (backport BZ 1002220)) posted (#2) for review on release-3.4 by Kaleb KEITHLEY (kkeithle@redhat.com)
Comment 2 Anand Avati 2013-09-10 00:28:19 UTC 
COMMIT: http://review.gluster.org/5822 committed in release-3.4 by Anand Avati (avati@redhat.com) 
------
commit 972e9155656e2211a62ab030e3e044964b934a85
Author: Kaleb S. KEITHLEY <kkeithle@redhat.com>
Date:   Thu Sep 5 09:10:30 2013 -0400

    api: glusterfs-api.pc.in use rpath (-Wl,-R...) (backport BZ 1002220)
    
    Change-Id: Ie36d7c3c058c2e738b0fb1ffae116ed43d573474
    Signed-off-by: Kaleb S. KEITHLEY <kkeithle@redhat.com>
    BUG: 1004751
    Reviewed-on: http://review.gluster.org/5822
    Tested-by: Gluster Build System <jenkins@build.gluster.com>
    Reviewed-by: Anand Avati <avati@redhat.com>
Note You need to log in before you can comment on or make changes to this bug.