Skip to contentSkip to conversation

Add missing buildrequires for Fedora

PATCH In review Erik Logtenberg 2 participants 2 messages Jul 30, 2013 – Aug 1, 2013 1 message not in the archive

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Add missing buildrequires for Fedora

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.

---

ceph.spec renamed from ceph.spec-orig +2 -0
@@ -42,6 +42,8 @@
 BuildRequires:  libxml2-devel
 BuildRequires:  libuuid-devel
 BuildRequires:  leveldb-devel > 1.2
+BuildRequires:  snappy-devel
+BuildRequires:  junit
 
 #################################################################################
 # specific

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

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:

Quoted 11 lines

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.

---