All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Brunner <chb@muc.de>
To: Sage Weil <sage@newdream.net>
Cc: ceph-devel@vger.kernel.org
Subject: Re: [PATCH 0/4] ceph.spec.in: fixes and additions for the rpm spec file
Date: Fri, 24 Sep 2010 23:04:20 +0200	[thread overview]
Message-ID: <20100924210420.GA10944@dell.home> (raw)
In-Reply-To: <AANLkTinG-3x8STo4kMgYjvNL1UZ8z3aCce0s9JX7OnhZ@mail.gmail.com>

On Fri, Sep 24, 2010 at 08:50:04PM +0200, Christian Brunner wrote:
> 2010/7/9 Sage Weil <sage@newdream.net>:
> >
> > There's one other pitfall I ran into with the debs... by default symbols
> > are stripped from all binaries, but the libcls_rbd.so shared object needs
> > to _not_ be stripped, or else the class loading doesn't work.  I have no
> > idea where/when/if this is done during the rpmbuild.  The version in the
> > built package should look like so:
> >
> > $ nm .libs/libcls_rbd.so
> > 000000000000797d T __cls_init
> > 000000000020ce50 D __cls_name
> > 000000000020ce7c B __cls_name__rbd
> > 000000000020ce78 B __cls_ver__1_1
> > 000000000020ce48 D __cls_ver_maj
> > 000000000020ce4c D __cls_ver_min
> 
> After setting up a new build system I ran into this, too. I've now
> looked at this a bit closer and was able to find a workaround. A patch
> for the spec file is attached.
> 
> Christian

I'm sorry. Sending the patch with cut and paste was a bad idea. Here is
the patch again...

---
 ceph.spec.in |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/ceph.spec.in b/ceph.spec.in
index c92ff9d..ab91f7b 100644
--- a/ceph.spec.in
+++ b/ceph.spec.in
@@ -75,6 +75,11 @@ mkdir -p $RPM_BUILD_ROOT%{_localstatedir}/lib/ceph/tmp/
 mkdir -p $RPM_BUILD_ROOT%{_localstatedir}/log/ceph/
 mkdir -p $RPM_BUILD_ROOT%{_localstatedir}/log/ceph/stat
 
+# To avoid that libcls_rbd.so.1.0.0 is beeing stiped by find-debuginfo.sh
+# we petend that the debug symbols have already been extracted
+mkdir -p $RPM_BUILD_ROOT/usr/lib/debug/%{_libdir}/rados-classes/
+touch $RPM_BUILD_ROOT/usr/lib/debug/%{_libdir}/rados-classes/libcls_rbd.so.1.0.0.debug
+
 %clean
 rm -rf $RPM_BUILD_ROOT
 
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2010-09-24 19:02 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-09  9:11 [PATCH 0/4] ceph.spec.in: fixes and additions for the rpm spec file Christian Brunner
2010-07-09 16:41 ` Sage Weil
2010-09-24 18:50   ` Christian Brunner
2010-09-24 21:04     ` Christian Brunner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100924210420.GA10944@dell.home \
    --to=chb@muc.de \
    --cc=ceph-devel@vger.kernel.org \
    --cc=sage@newdream.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.