Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-1.fc12.src.rpm Description: jay/yydebug supports tracing and animation for a Java-based parser generated by jay. An implementation of yyDebug is passed as an additional argument to yyparse() to trace a Java-based parser generated by jay with option -t set. yyDebugAdapter produces one-line messages, by default to standard output. The messages are designed to be filtered by a program such as grep. yyAnim provides an animation of the parsing process Required by JRuby. Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=1959840 rpmlint -i rpmbuild/RPMS/i386/yydebug-1.1.0-1.fc12.i386.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint -i rpmbuild/SRPMS/yydebug-1.1.0-1.fc12.src.rpm yydebug.src:87: W: libdir-macro-in-noarch-package (main package) 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Ignoring this last warning as when package is noarch (eg if using java-openjdk) the libdir file will not be included, and when the package is architecture specific (eg when using gcj) it will be. See spec file for conditionals.
I'm taking this one.
Please drop gcj_support. THere is no javadoc subpackage which should be present.
Why does gcj support need to be dropped when the guidelines indicate that it should always be included if possible?
Updated, gcj removed, no javadocs provided by upstream so no subpackage is necessary New spec / srpm: Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-2.fc11.src.rpm See my comment here for more info on the gcj bits: https://bugzilla.redhat.com/show_bug.cgi?id=561451#c5
Review: OK: rpmlint must be run on every package. Output: yydebug.src: W: spelling-error %description -l en_US yyparse -> parsec, sparse, parse yydebug.src: W: spelling-error %description -l en_US yyAnim -> cyanide, animus, animal False positives. OK: The package must be named according to the Package Naming Guidelines . OK: The spec file name must match the base package %{name}, in the format %{name}.spec unless your package has an exemption. OK: The package must meet the Packaging Guidelines . NOTOK: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines . Where did you get the License info from. I haven't found anything on the topic on the site and in the sources NOTOK: The License field in the package spec file must match the actual license. See prievious. OK: If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc. OK: The spec file must be written in American English. OK: The spec file for the package MUST be legible. OK: The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use md5sum for this task. OK: The package MUST successfully compile and build into binary rpms on at least one primary architecture. OK: All build dependencies must be listed in BuildRequires, except for any that are listed in the exceptions section of the Packaging Guidelines ; inclusion of those as BuildRequires is optional. Apply common sense. OK: Packages must NOT bundle copies of system libraries. OK: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. OK: A Fedora package must not list a file more than once in the spec file's %files listings. OK: Permissions on files must be set properly. Executables should be set with executable permissions, for example. Every %files section must include a %defattr(...) line. OK: Each package must consistently use macros. OK: The package must contain code, or permissable content. OK: Large documentation files must go in a -doc subpackage. OK: If a package includes something as %doc, it must not affect the runtime of the application. OK: Packages must not own files or directories already owned by other packages. OK: All filenames in rpm packages must be valid UTF-8. Licensing should be cleared - you have to put info in the spec file where you get the license from. About javadocs I would understand if you don't want to do it but adding javadoc generation should be just a few lines to the makefile and it's really good to have.
New Spec/SRPM w/ license source Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-3.fc11.src.rpm License is set to BSD since the 'jay' project, which 'yydebug' is a subproject under is licensed under BSD. See the jay.1 man page and the src/ subdir in the project repo: http://svn.codehaus.org/jruby/trunk/jay/ http://www.cs.rit.edu/~ats/projects/lp/doc/jay/doc-files/src.zip AFAIK (based on looking at the imports/dependencies) jruby only depends on yydebug and not the jay superproject, thus I only packaged yydebug. As far as javadocs, I'm more that willing to put them in, I'm not just that familiar w/ the javadoc system. If you want to send me a makefile patch I can include it, update the spec, and regenerate the srpm. Thanks for the review.
Created attachment 411522 [details] Patch adding javadoc generation
This patch should be applied after your current one.
New Spec/SRPM w/ javadocs Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-4.fc11.src.rpm New koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2165345
%changelog entries are missing your name. Otherwise the package looks good. Please make sure that you fix the %changelog before committing. This package is APPROVED.
Ah yes, my mistake. New version of the spec / srpm, with my name added to the changelog (the only change): New Spec/SRPM w/ javadocs Spec URL: http://mo.morsi.org/files/jruby/yydebug.spec SRPM URL: http://mo.morsi.org/files/jruby/yydebug-1.1.0-5.fc11.src.rpm New Package CVS Request ======================= Package Name: yydebug Short Description: Supports tracing and animation for a Java-based parser generated by jay. Owners: mmorsi Branches: F-12 F-13 InitialCC:
CVS done (by process-cvs-requests.py).
yydebug-1.1.0-5.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/yydebug-1.1.0-5.fc12
yydebug-1.1.0-5.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/yydebug-1.1.0-5.fc13
yydebug-1.1.0-5.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
yydebug-1.1.0-5.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.