[go: nahoru, domu]

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

ceph.spec.in:ownership of dirs extension #4793

Merged

Conversation

osynge
Copy link
@osynge osynge commented May 28, 2015

rpm lint says the following directories

/usr/share/ceph
/usr/lib/ceph

are un owned by any package and this causes
OBS to emit errors.

Signed-off-by: Owen Synge osynge@suse.com

@osynge
Copy link
Author
osynge commented May 28, 2015

Please note that if this and #4729 are merged OBS can build upstream master packages. (It fails due to other rpm lint warnings but hey I can fix that)

@ktdreyer
Copy link
Member

Looks good to me!

@tchaikov
Copy link
Contributor

s/ceph.spen.in/ceph.spec.in/

@osynge
Copy link
Author
osynge commented May 29, 2015

ops my spelling is so poor will fix now.

rpm lint says the following directories

/usr/share/ceph
/usr/lib/ceph

are un owned by any package and this causes
OBS to emit errors.

Signed-off-by: Owen Synge <osynge@suse.com>
@osynge osynge force-pushed the wip_ceph_spec_directories_not_owned_by_a_package branch from da39006 to 6f49597 Compare May 29, 2015 07:08
@osynge
Copy link
Author
osynge commented May 29, 2015

fixed commit message

@tchaikov tchaikov changed the title ceph.spen.in:ownership of dirs extension ceph.spec.in:ownership of dirs extension May 30, 2015
@ktdreyer
Copy link
Member
ktdreyer commented Jun 3, 2015

Looks good, merging.

By the way you may want to add osynge@suse.com to your GitHub account so the commits are linked with your GitHub username :)

@ktdreyer ktdreyer merged commit 6f49597 into ceph:master Jun 3, 2015
ktdreyer added a commit that referenced this pull request Jun 3, 2015
…d_by_a_package

ceph.spec.in:ownership of dirs extension

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
@ktdreyer
Copy link
Member
ktdreyer commented Jun 3, 2015

This didn't work out.

  error: File not found:
  /srv/autobuild-ceph/gitbuilder.git/build/rpmbuild/BUILDROOT/ceph-9.0.0-1225.ga16afdd.el6.x86_64/usr/lib/ceph

master's make install doesn't create these directories. Are you creating them some other way?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants