All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Add missing buildrequires for Fedora
@ 2013-07-30  7:53 Erik Logtenberg
       [not found] ` <51F770E9.6060004-nLcryBYLV+bMkAkIaI5Geg@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Erik Logtenberg @ 2013-07-30  7:53 UTC (permalink / raw)
  To: Ceph Devel; +Cc: ceph-users

[-- Attachment #1: Type: text/plain, Size: 380 bytes --]

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

[-- Attachment #2: add-missing-buildrequires-for-fedora.patch --]
[-- Type: text/x-patch, Size: 372 bytes --]

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Add missing buildrequires for Fedora
       [not found] ` <51F770E9.6060004-nLcryBYLV+bMkAkIaI5Geg@public.gmane.org>
@ 2013-08-01 13:14   ` Danny Al-Gaaf
  0 siblings, 0 replies; 2+ messages in thread
From: Danny Al-Gaaf @ 2013-08-01 13:14 UTC (permalink / raw)
  To: Erik Logtenberg; +Cc: Ceph Devel, ceph-users-idqoXFIVOFJgJs9I8MT0rw

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-nLcryBYLV+bMkAkIaI5Geg@public.gmane.org>
> ---
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-08-01 13:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-30  7:53 [PATCH] Add missing buildrequires for Fedora Erik Logtenberg
     [not found] ` <51F770E9.6060004-nLcryBYLV+bMkAkIaI5Geg@public.gmane.org>
2013-08-01 13:14   ` Danny Al-Gaaf

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.