]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph.spec.in:BuildRequires sharutils
authorOwen Synge <osynge@suse.com>
Mon, 8 Jun 2015 15:48:55 +0000 (17:48 +0200)
committerNathan Cutler <ncutler@suse.com>
Mon, 26 Oct 2015 11:40:54 +0000 (12:40 +0100)
The uudecode binary is used to build Java-related components, and
uudecode is provided by the sharutils package on all supported
RPM platforms. When building with "--without=cephfs_java",
sharutils is not needed.

Thanks to Nathan Cutler <ncutler@suse.cz> for going into the
details with me.

On OBS without this patch we get the error message:

[  170s] -----------------------------------------------------------------
[  170s] ----- building ceph.spec (user abuild)
[  170s] -----------------------------------------------------------------
[  170s] -----------------------------------------------------------------
[  170s] + exec rpmbuild -ba --define '_srcdefattr (-,root,root)' --nosignature --define 'disturl obs://build.opensuse.org/home:osynge:ceph:wip:wip_obs_fedora/Fedora_20/c0bbbc1e62228ca956ac3d367edc4fba-master' /home/abuild/rpmbuild/SOURCES/ceph.spec
[  170s] error: Failed build dependencies:
[  170s]    sharutils is needed by ceph-1:2+git.1435043747.c1bd02c-1.1.x86_64

With this patch we can build fedora 22 and fedora 20 rpms fine.

Signed-off-by: Owen Synge <osynge@suse.com>
(cherry picked from commit 43c1784640538b813f0575c50f721ac911a30b3e)

Conflicts:
ceph.spec.in
            _with_systemd -> trivial resolution

ceph.spec.in

index d62e6f1cb1377fc0f6a28ea80ca1791d0b1d362e..19730bf6a13579657aa5f9c860c6eb205d31bf83 100644 (file)
@@ -39,6 +39,9 @@ Requires:     util-linux
 Requires:      hdparm
 Requires:      cryptsetup
 Requires(post):        binutils
+%if 0%{with cephfs_java}
+BuildRequires: sharutils
+%endif
 BuildRequires: gcc-c++
 BuildRequires: boost-devel
 %if 0%{defined suse_version}
@@ -79,10 +82,6 @@ BuildRequires:       net-tools
 #################################################################################
 # specific
 #################################################################################
-%if ! 0%{?rhel} || 0%{?fedora}
-BuildRequires: sharutils
-%endif
-
 %if 0%{defined suse_version}
 %if 0%{?suse_version} > 1210
 Requires:      gptfdisk