Hide Forgot
Spec URL: http://gomix.fedorapeople.org/rubygem-rhc/rubygem-rhc.spec SRPM URL: http://gomix.fedorapeople.org/rubygem-rhc/rubygem-rhc-0.71.2-1.fc15.src.rpm Description: Openshift Express Client Tools OpenShift Express allows you to create and deploy applications to the cloud. The OpenShift Express client is a command line tool that allows you to manage your applications in the cloud.
rpmlint output missing: $ rpmlint -i SPECS/rubygem-rhc.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint -i SRPMS/rubygem-rhc-0.71.2-1.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v * rubygem-rhc.src: I: checking rubygem-rhc.src: I: checking-url https://openshift.redhat.com (timeout 10 seconds) rubygem-rhc.src: I: checking-url http://rubygems.org/downloads/rhc-0.71.2.gem (timeout 10 seconds) 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [lbazan@BAKER-FEDORA ruby]$ rpmlint *.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. I think after a review, the package is good. the spec is complete without failure.
APROVED!
When starting off as a new packager, you should do reviews that list off all the things that you've checked whether they pass or fail. That way other people can see that you know what you're doing or help understand what to look for if you don't detect something. The Review Guidelines here can be used as a partial checklist: http://fedoraproject.org/wiki/Packaging:ReviewGuidelines You can see that I also reference the rubygem guidelines in this review because there's some additional packaging guidelines for rubygems. Good: * Named according to the naming guidelines * Spec file named correctly * Package meets some of the rubygem guidelines: + naming, source, Provides, Requires, BR, empty %prep and %build, pkg is properly noarch + See below for parts that are not being followed. * Package includes license text * Spec file is legible * Source matches upstream * No locales to handle * Not a shared library or GUI * Does not bundle libraries * Not relocatable * Builds in koji * No files listed twice * Permissions on files set properly * Macros used consistently * Code, not content * Package does not own files or directories owned by another package * All filenames are utf-8 * rpmlint has only this: rubygem-rhc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/rhc-0.71.2/ri/RHC/hostexist%3f-c.yaml %3f False positive. Rubygems are internally escaping certain characters for these files. The "%" shows up for that reason, not because of an unexpanded rpm macro. Needswork: * Package does not meet the following rubygem guidelines: - Package needs to own: %{gemdir}/gems/%{gemname}-%{version}/ http://fedoraproject.org/wiki/Packaging:Ruby#Ruby_Gems * The use of %dir and the wildcard * in the %files section seems suspect. %dir is used to tell rpm to only include the following entry as a single directory, not recursively traverse it. However, these entries seem to be used to list files, not directories so %dir should be removed here. * Additionally, the rpm does not appear to have any files in %{geminstdir}/lib and %{geminstdir}/conf so both of those entries should be removed. * Maybe a better URL: https://openshift.redhat.com/app/express * I cannot find the GPLv2 as a license anywhere in this package. I do find the MIT license though. So the license needs to be changed to MIT. * From looking at this package it seems that the things in /usr/lib/ruby/gems/1.8/bin/ or /usr/lib/ruby/gems/1.8/gems/rhc-0.71.2/bin/ are intended to be run by end users. They need to have a presence in %{_bindir} for the user to invoke then. I don't know if the standard for rubygems is to move the files there or to have a small shell script that invokes the program in the %{gemdir} * Should everything in /usr/lib/ruby/gems/1.8/doc/rhc-0.71.2 be marked doc? If a file is depended on at runtime, it cannot be marked %doc because the person installing the package may choose to exclude them from being installed. The yaml files, for instance, don't seem to be there for end users to read... will removing them when installing the package break something?
spec: http://gomix.fedorapeople.org/rubygem-rhc/rubygem-rhc.spec srpm: http://gomix.fedorapeople.org/rubygem-rhc/rubygem-rhc-0.71.2-2.fc15.src.rpm * Package now owns %{gemdir}/gems/%{gemname}-%{version}/ * Licence fixed to MIT. * Missuse of %dir corrected. * Better URL done. * User binaries moved to the right place. * Should everything in /usr/lib/ruby/gems/1.8/doc/rhc-0.71.2 be marked doc? Yes, none of those files are dependencies for runtime execution and can be safely removed without breaking a thing. yaml files are used by ri ruby online documentation system. rdoc is html formatted versions. rpmlint outputs $ rpmlint -i SPECS/rubygem-rhc.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint -i SRPMS/rubygem-rhc-0.71.2-2.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint -i RPMS/noarch/rubygem-rhc-0.71.2-2.fc15.noarch.rpm rubygem-rhc.noarch: W: unexpanded-macro /usr/lib/ruby/gems/1.8/doc/rhc-0.71.2/ri/RHC/hostexist%3f-c.yaml %3f This package contains a file whose path contains something that looks like an unexpanded macro; this is often the sign of a misspelling. Please check your specfile. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-ctl-app Each executable in standard binary directories should have a man page. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-snapshot Each executable in standard binary directories should have a man page. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-create-app Each executable in standard binary directories should have a man page. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-user-info Each executable in standard binary directories should have a man page. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-create-domain Each executable in standard binary directories should have a man page. rubygem-rhc.noarch: W: no-manual-page-for-binary rhc-tail-files Each executable in standard binary directories should have a man page. 1 packages and 0 specfiles checked; 0 errors, 7 warnings. besides lack of man pages, yaml-rpmlint reporting issue is ignorable. it looks better now, thanks for reviewing
Problems noted before have been taken care of. rpmlint warnings are ignorable. APPROVED
New Package SCM Request ======================= Package Name: rubygem-rhc Short Description: OpenShift Express Client Tools Owners: gomix Branches: f14 f15 el6 InitialCC:
Git done (by process-git-requests).
rubygem-rhc-0.71.2-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/rubygem-rhc-0.71.2-2.fc15
rubygem-rhc-0.71.2-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-rhc-0.71.2-2.fc14
rubygem-rhc-0.71.2-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/rubygem-rhc-0.71.2-2.el6
rubygem-rhc-0.71.2-2.fc14 has been pushed to the Fedora 14 testing repository.
rubygem-rhc-0.71.2-2.fc14 has been pushed to the Fedora 14 stable repository.
rubygem-rhc-0.71.2-2.fc15 has been pushed to the Fedora 15 stable repository.
rubygem-rhc-0.71.2-2.el6 has been pushed to the Fedora EPEL 6 stable repository.
Package Change Request ====================== Package Name: rubygem-rhc New Branches: el5
No owner specified.
Package Change Request ====================== Package Name: rubygem-rhc New Branches: el5 Owners: gomix
Package Change Request ====================== Package Name: rubygem-rhc New Branches: epel7 Owners: gomix tdawson