Spec URL: http://timon.perm.ru/download/perl-Text-Trac.spec SRPM URL: http://timon.perm.ru/download/perl-Text-Trac-0.13-1.fc9.src.rpm Description: Text::Trac parses text with Trac WikiFormatting and convert it to html format. more info: http://search.cpan.org/dist/Text-Trac/
*** Bug 464140 has been marked as a duplicate of this bug. ***
OK source files match upstream OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK build root is correct. OK license field (same as Perl) matches the actual license. OK license is open source-compatible. License text not included upstream. OK latest version is being packaged. OK BuildRequires are proper. OK %clean is present. OK package builds in mock (Rawhide/x86_64). OK debuginfo package isn't need. OK rpmlint is silent. OK final provides and requires look sane. OK no shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK no duplicates in %files. OK file permissions are appropriate. OK no scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK no headers. OK no pkgconfig files. OK no libtool .la droppings.
Created attachment 348669 [details] perl-Text-Trac-0.15-1.fc12.src.rpm
Created attachment 348670 [details] perl-Text-Trac.spec
new version perl-Text-Trac-0.15
Correct me if I'm wrong. But you wasn't still sponsored, so you don't have commit access into cvs?
yup. I need a sponsor.
Hi Timon, 1] please bump everytime you provide new version of package release number and post links to new version in your reply 2] %defattr(0644,root,root,0755) do you have any reason why not to use %defatter(-,root,root,-) ? https://fedoraproject.org/wiki/Packaging/Guidelines#File_Permissions 3] cp in makefile is without -p switch, write patch that cp in Makefile will be performed with -p switch (sed during prep will be good enough) https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps 4] BuildRequires: perl(Test::Pod::Coverage) BuildRequires: perl(Test::Pod) you have to include this as without it, not all tests are performed and you break this guideline: https://fedoraproject.org/wiki/Packaging:Perl#Perl_Packaging Hi Marcela, official review for new packagers which are not yet sponsored must be done by sponsor. http://fedoraproject.org/wiki/Package_Review_Process#Reviewer I think that you should keep this ticket unassigned until Timon is sponsored. That sponsors will see that this ticket needs attention and is not assigned.
Marcela, please clear fedora.review as there are things which needs to be adressed before this package meets packaging guidelines and as Timon is not yet sponsored so fedora.review+ could be set only by sponsor.
It's been quite some time since comment #8 with no response from the submitter of this ticket. I guess it should be closed soon if nothing further happens.
(In reply to comment #8) > 1] > please bump everytime you provide new version of package release number and > post links to new version in your reply I have some problems with public web server. So I attach spec and SRPM. > 2] > %defattr(0644,root,root,0755) > do you have any reason why not to use %defatter(-,root,root,-) ? > https://fedoraproject.org/wiki/Packaging/Guidelines#File_Permissions fixed > 3] cp in makefile is without -p switch, write patch that cp in Makefile will be > performed with -p switch (sed during prep will be good enough) > https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps This is CPAN Package and Makefile generates using Makefile.PL perl script. I can sed in %build section, but I don't think this is good idea. I looked into other SRPM made from CPAN, but didn't find any resolutions of this problem. > 4] > BuildRequires: perl(Test::Pod::Coverage) > BuildRequires: perl(Test::Pod) fixed
Created attachment 367264 [details] perl-Text-Trac.spec
Created attachment 367267 [details] perl-Text-Trac-0.15-1.fc12.src.rpm
(In reply to comment #8) > please bump everytime you provide new version of package release number and > post links to new version in your reply perl-Text-Trac-0.15-1 https://bugzilla.redhat.com/attachment.cgi?id=367267 - perl-Text-Trac-0.15-1.fc12.src.rpm https://bugzilla.redhat.com/attachment.cgi?id=367264 - perl-Text-Trac.spec
Hi Timon, it is better to have this somewhere on web and put just links to that specfile/src.rpm (In reply to comment #11) > > 3] cp in makefile is without -p switch, write patch that cp in Makefile will be > > performed with -p switch (sed during prep will be good enough) > > https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps > This is CPAN Package and Makefile generates using Makefile.PL perl script. I > can sed in %build section, but I don't think this is good idea. I looked into > other SRPM made from CPAN, but didn't find any resolutions of this problem. Why do you think it is wrong idea? if Makefile is not ok by fedora guidelines, you need to alter it into way that it will be ok with guidelines
Is anything happening with this ticket? I suspect that few people want to review it because you make it difficult to get the files. Bugzilla attachments are not a very good way to do this, which is why we offer hosting for this kind of thing. Our documentation tells you how to request hosting, which suggests to me that you've not read it. http://fedoraproject.org/wiki/Join_the_package_collection_maintainers
No response in ages; closing.