Bug 176721

Summary: perl-RPm-Specfile: update request to 1.19
Product: [Fedora] Fedora Reporter: Jose Pedro Oliveira <jose.p.oliveira.oss>
Component: perl-RPM-SpecfileAssignee: Jason Vas Dias <jvdias>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: perl-devel
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://search.cpan.org/dist/RPM-Specfile/
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-01-19 19:17:48 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 Flags
Update, specfile rewrite none

Description Jose Pedro Oliveira 2005-12-30 19:57:11 UTC
Version-Release number of selected component (if applicable):
rawhide: perl-RPM-Specfile-1.17-2.src.rpm

Expected results:
version 1.19 in rawhide

Additional info:
Diff from RPM-Specfile-1.17 to RPM-Specfile-1.19
http://search.cpan.org/diff?from=RPM-Specfile-1.17&to=RPM-Specfile-1.19

Comment 1 Ville Skyttä 2006-01-06 10:14:37 UTC
Created attachment 122862 [details]
Update, specfile rewrite

* Fri Jan  6 2006 Ville Skyttä <ville.skytta at iki.fi> - 1.19-1
- 1.19 (#176721).
- Rewrite specfile using fedora-rpmdevtools' spec template, fixes #176888.

Comment 2 Ville Skyttä 2006-01-06 10:18:58 UTC
Hm, %{?dist} in Release probably has no meaning Core packages.

Comment 3 Jason Vas Dias 2006-01-10 16:37:31 UTC
Thanks Ville - package upgraded to RPM::Spec 1.19, and attached .spec 
file submitted with a few minor modifications to perl-RPM-Specfile-1.19-1 .


Comment 4 Ville Skyttä 2006-01-10 21:54:14 UTC
This change needs to be reverted, it results in an unowned directory:

-%{perl_vendorlib}/RPM/
+%{perl_vendorlib}/RPM/*


Comment 5 Jason Vas Dias 2006-01-12 21:48:45 UTC
I'm not sure about this.

There are loads of perl RPM::* modules, all of which use the
%{perl_vendorlib}/RPM/ directory - so I changed the .spec file
to make the package own %{perl_vendorlib}/RPM/* instead of 
%{perl_vendorlib}/RPM/ - I think that makes more sense .




Comment 6 Ville Skyttä 2006-01-13 15:55:14 UTC
None of the dependencies of perl-RPM-Specfile own the %{perl_vendorlib}/RPM
directory, so this package needs to own it.

It's just the usual case of unowned directories where 1) they get left behind
when erasing the package -> unneeded trash, and 2) with an umask of 007 or 077
for root when installing the package, they'll end up unreadable by world and/or
group -> broken out of the box for everyone besides root.  (I may have forgotten
some other problems.)

Bugzilla is full of reports like these, in particular for perl module packages
created by an old version of cpanflute which didn't do the right thing with dir
ownerships.  Luckily the vast majority of them have been already fixed.

(I see you did similar modifications to the perl-XML-Grove specfile I submitted;
that's different in the sense that its prerequisites own the %{perl_vendor*}/XML
directory so it does not have the umask problem -> non-fatal, which is why I
didn't reopen that bug.  Due to bugs in rpm (for example bug 89500), the "XML"
dir in it may be still left behind after erasing the package because erase
transactions are not properly sorted, but that's something that needs fixing in
rpm.)

Comment 7 Jason Vas Dias 2006-01-17 22:24:25 UTC
OK, change reverted with perl-RPM-Specfile-1.19-2 : RPM/* -> RPM .