]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
Packaging: Removing conditional BuildRequires for btrfs-progs 8016/head
authorErwan Velu <erwan@redhat.com>
Thu, 10 Mar 2016 08:15:14 +0000 (09:15 +0100)
committerErwan Velu <erwan@redhat.com>
Thu, 10 Mar 2016 08:15:14 +0000 (09:15 +0100)
commit2294c73d391e415be5299ff773519af2d2544151
treecc9e2db8dda32dece4d86720dec1b2e000a6061e
parent44f7b87263681e7a755837602065769a91e156bd
Packaging: Removing conditional BuildRequires for btrfs-progs

By pull request 7742, the btrfs-progs package was considered as a BuildRequires
only when --with tests was engaged like :

    if %{with tests}
    BuildRequires: btrfsprogs
    %endif

That's perfectly valid for a spec file.

The issue we have is the following :
- yum-builddep called by install-deps.sh is used to prepare the build env by
installing the needed BuildRequires.
- %{with test} is defined by using a %bcond_with
- yum-builddep doesn't consider %{with test} as valid
- yum-builddep doesn't install the btrfs package

As per discussions with the yum team, there is no way to engage conditional flags with
yum-builddep.

So this patch, as per discussions with Nathan Cutler & Loic Dachary, is removing
the condition arond the BuildRequires.

Note that all BuildRequires defined with a %bcond_with would be affected by this
issue. The current specfile only have %bcond_without conditional BuildRequires
which is fine.

Fixes: #15042
Signed-off-by: Erwan Velu <erwan@redhat.com>
ceph.spec.in