Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=563318 Jonathan Dieter <jdieter@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+ --- Comment #13 from Jonathan Dieter <jdieter@xxxxxxxxx> 2010-04-26 09:50:15 EDT --- As I was playing around with this today, I noticed that the man pages for cauthtool and rados are in the ceph-fuse package, while the binaries are in the base package. We probably want to move them to the base package as I'm assuming cauthtool and rados aren't just needed for the fuse client. As for the exit() in shared libraries, your explanation makes sense, so we'll call that good. FWIW, I did build the package on my system with the updated spec and haven't hit any glitches yet (though I haven't set up a ceph cluster yet). I'm going to call this approved, with the proviso that the man pages get moved (or binaries the other way if they are specific to the fuse client). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review