Bug 245639
Summary: | RFE: move check-buildroot, check-rpaths to rpm-build or redhat-rpm-config | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Ville Skyttä <ville.skytta> | ||||
Component: | redhat-rpm-config | Assignee: | Jeremy Katz <katzj> | ||||
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | low | Docs Contact: | |||||
Priority: | low | ||||||
Version: | rawhide | CC: | dcantrell, jonathan, opensource, pmatilai, rh-bugzilla | ||||
Target Milestone: | --- | Keywords: | Reopened | ||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2007-07-20 22:19:54 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Ville Skyttä
2007-06-25 20:14:48 UTC
Doing the work to get these into redhat-rpm-config. Since redhat-rpm-config will be providing the macro to run these, I think having them in /usr/lib/rpm/redhat/ is best for now. This is orthogonal to getting these in upstream RPM. If they ever show up in upstream RPM, we can adjust redhat-rpm-config to drop them and point to the new path for running them. I would suggest filing this RFE at rpm.org Trac. The bad thing about /usr/lib/rpm/redhat/ instead of /usr/lib/rpm/ is that a lot of people and setups have their ~/.rpmmacros and friends configured to invoke the scripts from /usr/lib/rpm/ - that's where rpmdevtools installs and rpmdev-setuptree configures them. I don't think these settings are fixable from a package, therefore /usr/lib/rpm/ would be very much recommended for backwards compatibility and less grief. http://people.redhat.com/jkeating/git/redhat-rpm-config.git has changes necessary to add these files to redhat-rpm-config, and enable them by default. (also I cleaned up the dist stuff in a second commit) Jeremy, can you apply this? This is part of the effort to remove the 'groups' repo in mock. Hrm. a bit of cross traffic there. I suppose I can make the scripts go into /usr/lib/rpm, will just mean special casing them in the Makefile, and I do so hate editing Makefiles :( Villa, care to provide a patch to Makefile that moves them somewhere different? Created attachment 158375 [details] Move check-* to /usr/lib/rpm, add test suite (In reply to comment #4) > Villa, That's wool in Finnish :) > care to provide a patch to Makefile that moves them somewhere different? Here goes. Also included is the check suite for check-rpaths from rpmdevtools. My git repo now has those changes in it. Thanks Ville (<-- look, I got it right!) FWIW, I've zero objections to adding the check-* scripts to upstream rpm even for 4.4.2.1, but enabling by default (or not) will need to go to redhat-rpm-config at this point. How soon would 4.4.2.1 show up? I'd like to get these things done ASAP as they are part of a larger change to mock. I can stuff the scripts in, bundle up a 4.4.2.1-rc2 tarball and have it in next rawhide build if you wish. Final 4.4.2.1 is due out very shortly anyway, these rc's are simply about being very cautious. That would be most awesome. IF you do that, I'll adjust my redhat-rpm-config changes to just enable it by default (and fix the %dist stuff too). Next rawhide build would be most awesome. http://people.redhat.com/jkeating/git/redhat-rpm-config.git now just has changes for enabling the post %install buildroot checking, and dist cleanups. Added to rpm 4.4.2.1-rc2 and currently building that for rawhide. Assuming nothing blows up, will be in next rawhide push. Confirmed, in the new rpm. I'll consider this closed. Thanks. One last bit: --- scripts/check-rpaths-worker~ 2007-07-04 14:57:19.000000000 +0300 +++ scripts/check-rpaths-worker 2007-07-05 23:41:11.000000000 +0300 @@ -65,4 +65,2 @@ * -* 'check-rpaths' is part of 'rpmdevtools'. -* ******************************************************************************* Looks like Panu got around to fixing that bit in hg some time ago. Thanks. |