From: Erik Logtenberg <erik@logtenberg.eu>
To: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Cc: Patrick McGarry <patrick@inktank.com>,
Ceph Devel <ceph-devel@vger.kernel.org>,
ceph-users@lists.ceph.com
Subject: Re: Fwd: [ceph-users] Small fix for ceph.spec
Date: Tue, 30 Jul 2013 23:01:28 +0200 [thread overview]
Message-ID: <51F829A8.104@logtenberg.eu> (raw)
In-Reply-To: <51F771B2.1080606@bisect.de>
Hi,
I will report the issue there as well. Please note that Ceph seems to
support Fedora 17, even though that release is considered end-of-life by
Fedora. This issue with the leveldb package cannot be fixed for Fedora
17, only for 18 and 19.
So if Ceph wants to continue supporting Fedora 17, adding this
workaround seems to be the only way to get this (rather minor) bug fixed.
Kind regards,
Erik.
On 07/30/2013 09:56 AM, Danny Al-Gaaf wrote:
> Hi,
>
> then the Fedora package is broken. If you check the spec file of:
>
> http://dl.fedoraproject.org/pub/fedora/linux/updates/19/SRPMS/leveldb-1.12.0-3.fc19.src.rpm
>
>
> You can see the spec-file sets a:
>
> BuildRequires: snappy-devel
>
> But not the corresponding "Requires: snappy-devel" for the devel package.
>
> You should report this issue to your distribution, it needs to be fixed
> there instead of adding a workaround to the ceph spec.
>
> Regards,
>
> Danny
>
> Am 30.07.2013 09:42, schrieb Erik Logtenberg:
>> Hi,
>>
>> Fedora, in this case Fedora 19, x86_64.
>>
>> Kind regards,
>>
>> Erik.
>>
>>
>> On 07/30/2013 09:29 AM, Danny Al-Gaaf wrote:
>>> Hi,
>>>
>>> I think this is a bug in packaging of the leveldb package in this case
>>> since the spec-file already sets dependencies on on leveldb-devel.
>>>
>>> leveldb depends on snappy, therefore the leveldb package should set a
>>> dependency on snappy-devel for leveldb-devel (check the SUSE spec file
>>> for leveldb:
>>> https://build.opensuse.org/package/view_file/home:dalgaaf:ceph:extra/leveldb/leveldb.spec?expand=1).
>>> This way the RPM build process will pick up the correct packages needed
>>> to build ceph.
>>>
>>> Which distro do you use?
>>>
>>> Danny
>>>
>>> Am 30.07.2013 01:33, schrieb Patrick McGarry:
>>>> ---------- Forwarded message ----------
>>>> From: Erik Logtenberg <erik@logtenberg.eu>
>>>> Date: Mon, Jul 29, 2013 at 7:07 PM
>>>> Subject: [ceph-users] Small fix for ceph.spec
>>>> To: ceph-users@lists.ceph.com
>>>>
>>>>
>>>> Hi,
>>>>
>>>> The spec file used for building rpm's misses a build time dependency on
>>>> snappy-devel. Please see attached patch to fix.
>>>>
>>>> Kind regards,
>>>>
>>>> Erik.
>>>>
>>>> _______________________________________________
>>>> ceph-users mailing list
>>>> ceph-users@lists.ceph.com
>>>> http://lists.ceph.com/listinfo.cgi/ceph-users-ceph.com
>>>>
>>
prev parent reply other threads:[~2013-07-30 21:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <51F6F595.605@logtenberg.eu>
[not found] ` <51F6F595.605-nLcryBYLV+bMkAkIaI5Geg@public.gmane.org>
2013-07-29 23:29 ` Small fix for ceph.spec Patrick McGarry
2013-07-29 23:33 ` Fwd: [ceph-users] " Patrick McGarry
[not found] ` <CAP8Oy=KD4au7aaOqo1t6zvWP1JFnTtQELd8YunxXqyBaE4+wpg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-07-30 7:29 ` Fwd: " Danny Al-Gaaf
[not found] ` <51F76B43.4020801-2YacvwyR+KOzQB+pC5nmwQ@public.gmane.org>
2013-07-30 7:42 ` Erik Logtenberg
2013-07-30 7:56 ` Fwd: [ceph-users] " Danny Al-Gaaf
2013-07-30 21:01 ` Erik Logtenberg [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=51F829A8.104@logtenberg.eu \
--to=erik@logtenberg.eu \
--cc=ceph-devel@vger.kernel.org \
--cc=ceph-users@lists.ceph.com \
--cc=danny.al-gaaf@bisect.de \
--cc=patrick@inktank.com \
/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.