Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hammer: redhat-lsb-core dependency was dropped, but is still needed #7876

Merged
1 commit merged into from Mar 11, 2016

Conversation

smithfarm
Copy link
Contributor

The lsb_release executable is being run in multiple places, not least in
src/common/util.cc, which calls it via shell in the collect_sys_info() code
path.

This patch addresses this issue on SUSE- and Debian-derivatives, as well
as reinstating the dependency for RHEL/Fedora after it was dropped in
1560057.

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

Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit b87f4ef)

Conflicts:
	ceph.spec.in
             The jewel specfile has diverged considerably from hammer:
             systemd, package split, etc. This is more of a hand backport
             than a cherry-pick.
@smithfarm smithfarm self-assigned this Mar 2, 2016
@smithfarm smithfarm added this to the hammer milestone Mar 2, 2016
ghost pushed a commit that referenced this pull request Mar 4, 2016
…ped, but is still needed

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Mar 11, 2016

It passed a run of the upgrade suite : http://tracker.ceph.com/issues/14692#note-31

ghost pushed a commit that referenced this pull request Mar 11, 2016
hammer: redhat-lsb-core dependency was dropped, but is still needed

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit dd66eb9 into ceph:hammer Mar 11, 2016
@smithfarm smithfarm deleted the wip-14946-hammer branch March 11, 2016 05:08
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant