diff mbox

Add missing buildrequires for Fedora

Message ID 51F770E9.6060004@logtenberg.eu (mailing list archive)
State New, archived
Headers show

Commit Message

Erik Logtenberg July 30, 2013, 7:53 a.m. UTC
Hi,

This patch adds two buildrequires to the ceph.spec file, that are needed
to build the rpms under Fedora. Danny Al-Gaaf commented that the
snappy-devel dependency should actually be added to the leveldb-devel
package. I will try to get that fixed too, in the mean time, this patch
does make sure Ceph builds on Fedora.

Signed-off-by: Erik Logtenberg <erik@logtenberg.eu>
---

Comments

Danny Al-Gaaf Aug. 1, 2013, 1:14 p.m. UTC | #1
Hi,

I've opened a pull request with some additional fixes for this issue:
https://github.com/ceph/ceph/pull/478

Danny

Am 30.07.2013 09:53, schrieb Erik Logtenberg:
> Hi,
> 
> This patch adds two buildrequires to the ceph.spec file, that are needed
> to build the rpms under Fedora. Danny Al-Gaaf commented that the
> snappy-devel dependency should actually be added to the leveldb-devel
> package. I will try to get that fixed too, in the mean time, this patch
> does make sure Ceph builds on Fedora.
> 
> Signed-off-by: Erik Logtenberg <erik@logtenberg.eu>
> ---
> 

--
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
diff mbox

Patch

--- ceph.spec-orig	2013-07-30 00:24:54.705000000 +0200
+++ ceph.spec	2013-07-30 01:20:23.593000000 +0200
@@ -42,6 +42,8 @@ 
 BuildRequires:  libxml2-devel
 BuildRequires:  libuuid-devel
 BuildRequires:  leveldb-devel > 1.2
+BuildRequires:  snappy-devel
+BuildRequires:  junit
 
 #################################################################################
 # specific