Bug 118112
Summary: | patch to build subversion perl modules | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Steven Pritchard <steve> | ||||||||
Component: | subversion | Assignee: | Joe Orton <jorton> | ||||||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||||||
Severity: | medium | Docs Contact: | |||||||||
Priority: | medium | ||||||||||
Version: | rawhide | Keywords: | FutureFeature | ||||||||
Target Milestone: | --- | ||||||||||
Target Release: | --- | ||||||||||
Hardware: | All | ||||||||||
OS: | Linux | ||||||||||
Whiteboard: | |||||||||||
Fixed In Version: | 1.0.0-3 | Doc Type: | Enhancement | ||||||||
Doc Text: | Story Points: | --- | |||||||||
Clone Of: | Environment: | ||||||||||
Last Closed: | 2004-03-12 10:13:53 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
Steven Pritchard
2004-03-12 01:30:23 UTC
Created attachment 98477 [details]
spec changes
Comment on attachment 98477 [details]
spec changes
That first hunk is bogus. Plus libsvn_swig_perl-1.so.0 ends up in the main
package instead of in the -perl subpackage. I'll fix both and resubmit.
Created attachment 98480 [details]
spec changes, take 2
Unfortunately, this patch uglies up the spec file just a bit. I couldn't think
of a good way to say that we want libsvn_swig_perl and nothing else in /usr/lib
without spelling it out.
The good news is that it seems to install properly. The main package is in no
way dependent on the -perl subpackage.
%exclude is your friend :) Thanks a lot Steve, this looks great; I added this in with changes to: - use %exclude to put libsvn_swig_perl in the -perl subpackage - make the Perl module .so's chmod 755 so they get stripped - remove the unnecessary libdir/libsvn_swig_*.{a,la,so} I'll attach the new spec file Created attachment 98481 [details]
spec file -3
I either forgot about, or never knew about, %exclude. Either way, I'll keep that in mind for next time. :-) Thanks for taking a look at this so quickly. Oh, what about those empty .bs files; are they needed? I get the impression (doing some quick reading on ExtUtils::Mkbootstrap) that those files shouldn't be created zero-length, so there's probably a bug there somewhere. I'd guess that means it is OK to clean them out. Perhaps cturner would have a better answer? Looks like zero-length .bs files can safely be removed from some googling; I'll do that for the next build. Meanwhile, this is built for Raw Hide - thanks again. |