]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
ceph.spec.in: add missing BuildRequires from SUSE block
authorLoic Dachary <ldachary@redhat.com>
Sat, 6 Jun 2015 08:20:02 +0000 (10:20 +0200)
committerLoic Dachary <ldachary@redhat.com>
Sat, 6 Jun 2015 10:05:19 +0000 (12:05 +0200)
commit34e4dd9c8a562b48d6b807b76e4fc3046fc6574b
treef0842956cceb963cd672577754b620cdae212caf
parenta8500ce981c563ece34e7171571b6b56663ae6b5
ceph.spec.in: add missing BuildRequires from SUSE block

The %endif removed by f94f23297c05019db754ec53b4ceaea7706152c9 is restored.
The %else removed by 75e87a20dac05441b97ce99dea76fff8e3a863ac is restored.
The stray %endif added by d8abde3338b0c7df373b762e35099ad5123866bf is removed.

May 29th, in d8abde3338b0c7df373b762e35099ad5123866bf Owen added a stray
%endif after

   BuildRequires: gperftools-devel

around line 133. June 3rd, in f94f23297c05019db754ec53b4ceaea7706152c9
Ken correctly attributed the error

     "error: /srv/autobuild-ceph/gitbuilder.git/build/ceph.spec:140: Got a
      %endif with no %if"

to a stray %endif but did not remove the one causing problem and in
doing so created another problem. June 4th, in
75e87a20dac05441b97ce99dea76fff8e3a863ac Owen incorrectly fixed
this new problem by removing the %else that is near

   BuildRequires: gperftools-devel

around line 116, instead of reverting
f94f23297c05019db754ec53b4ceaea7706152c9. As a consequence the
ceph.spec.in became syntactically correct but implemented an if/else
logic different from what was intended originally and a number of
BuildRequires became exclusive to SUSE and were not installed for CentOS
7 etc.

http://tracker.ceph.com/issues/11901 Fixes: #11901

Signed-off-by: Loic Dachary <ldachary@redhat.com>
ceph.spec.in