Spec URL: http://stahnma.fedorapeople.org/reviews/rhnpush.spec SRPM URL: http://stahnma.fedorapeople.org/reviews/rhnpush-0.3.1-2.fc10.src.rpm Description: rhnpush uploads package headers to the Red Hat Network servers into specified channels and allows for several other channel management operations relevant to controlling what packages are available per channel.
FAIL source files match upstream The source should be written as whole path. 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). http://koji.fedoraproject.org/koji/taskinfo?taskID=1124650 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. Shouldn't be test executed as a part of build process? (btw you have ugly man page of rhnpush the synopsis part.)
Spec URL: http://stahnma.fedorapeople.org/reviews/rhnpush.spec SRPM URL: http://stahnma.fedorapeople.org/reviews/rhnpush-0.3.1-3.fc10.src.rpm * I have updated the package slightly. I added a source description, as per http://fedoraproject.org/wiki/Packaging/SourceURL . * I tried to play with %check and tests, but I couldn't them to work. It seems like I am missing a harness setup of some type. I will ping the sw-dev list and see if I can find out anything. Either way, that shouldn't be a blocker for package review. * Man page: It's a little different than many, but certainly not wrong. See (man vim), it looks similar. I think it's because the man page is generated using doc2book on the SGML file. I'd rather just write it in groff, but I am not upstream.
Spec URL: http://stahnma.fedorapeople.org/reviews/rhnpush.spec SRPM URL: http://stahnma.fedorapeople.org/reviews/rhnpush-0.3.1-4.fc10.src.rpm * Found a better Source URL.
But if I do it then I have 7f847b5b6066ced228e7b7868524cdcf rhnpush-0.4.2.tar.gz. So we have two problems instead one. You should package the latest version and here are different sources :) The other issues were only comments, nothing serious.
If I am reading this correctly, I should package the released Tarball and not the generated tarball from Git. Is that correct?
You can leave source code as you have it, but if Fedora is upstream you should upload tarball on proper place on https://fedorahosted.org/releases/ Also there is condition that you should package the latest release.
Michael if you do not mind (and I know you will not :) ) I will take ownership of this package. I really want to thanx you for all the job you are doing. I applied your changes to our git repo. I packed latest version. And make sure the tar.gz landed in fedorahosted.org/releases/ *Test - the test are long time abandoned and definitelly do not work. We keep it there if anybody want to fix it as it is better then just deleting it forever. UPDATED SPEC: http://miroslav.suchy.cz/fedora/rhnpush/rhnpush.spec SRPM: http://miroslav.suchy.cz/fedora/rhnpush/rhnpush-0.4.3-1.src.rpm
OK source files match upstream 3b53e8a569b7e486634482216410669d 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 (GPLv2) 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). http://koji.fedoraproject.org/koji/taskinfo?taskID=1142201 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. What about BR? Is there reason for using %{_bindir}/msgfmt instead of gettext and %{_bindir}/docbook2man instead of docbook-utils?
Hmm, actually there is no reason. And yes, package dependence should be prefered rather then file dependence. UPDATED SPEC: http://miroslav.suchy.cz/fedora/rhnpush/rhnpush.spec SRPM: http://miroslav.suchy.cz/fedora/rhnpush/rhnpush-0.4.4-1.src.rpm
ACCEPTED
New Package CVS Request ======================= Package Name: rhnpush Short Description: Package uploader for the RHN Satellite/Spacewalk Server Owners: msuchy Branches: F-10, EL-4, EL-5 InitialCC: stahnma
cvs done.