Spec URL: http://tdawson.fedorapeople.org/review/rubygem-safe_yaml.spec SRPM URL: http://tdawson.fedorapeople.org/review/rubygem-safe_yaml-0.9.4-1.fc20.src.rpm Description: The SafeYAML gem provides an alternative implementation of YAML.load suitable for accepting user input in Ruby applications. Unlike Ruby's built-in implementation of YAML.load, SafeYAML's version will not expose apps to arbitrary code execution exploits. Fedora Account System Username: tdawson
rpmlint output: $ rpmlint rubygem-safe_yaml.spec /home/quake/rpmbuild/SRPMS/rubygem-safe_yaml-0.9.4-1.fc20.src.rpm /home/quake/rpmbuild/RPMS/noarch/rubygem-safe_yaml-0.9.4-1.fc20.noarch.rpm /home/quake/rpmbuild/RPMS/noarch/rubygem-safe_yaml-doc-0.9.4-1.fc20.noarch.rpm rubygem-safe_yaml-doc.noarch: W: spelling-error Summary(en_US) yaml -> yam, yams, yawl rubygem-safe_yaml-doc.noarch: W: spelling-error %description -l en_US yaml -> yam, yams, yawl 3 packages and 1 specfiles checked; 0 errors, 2 warnings. (Not much I can do when the spell checker doesn't like yaml)
* I understand that specs cannot be run at the moment, but perhaps would be better to fix the spec file in the way that it could run them when the deps are fulfilled. That probably requires running the specs in the right directory: pushd .%{gem_instdir} rspec -Ilib spec popd and requiring RSpec as BuildRequires: BuildRequires: rubygem(rspec) (both within your 0%{?enable_tests} condition). Otherwise the spec matches the guidelines, rpmlint doesn't complain, package builds, installs and runs fine so I am APPROVING. Please, fix the issue above (or delete the conditions whatsoever) before pushing it and don't forget to bump the release. Thank you.
Spec URL: http://tdawson.fedorapeople.org/review/rubygem-safe_yaml.spec SRPM URL: http://tdawson.fedorapeople.org/review/rubygem-safe_yaml-0.9.4-2.fc19.src.rpm - Updated BuildRequires (in 0%{?enable_tests} condition) -- now has BuildRequires: rubygem(rspec) - Update %check (in 0%{?enable_tests} condition) -- Now does pushd / popd to correct directory for testing Thanks for these comments. You are correct, if I'm going to put that condition in, so we can easily do the tests when the requirements are there, I should get it correct.
New Package SCM Request ======================= Package Name: rubygem-safe_yaml Short Description: Parse YAML safely Owners: tdawson Branches: f18 f19 el6 InitialCC:
Git done (by process-git-requests).
rubygem-safe_yaml-0.9.4-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/rubygem-safe_yaml-0.9.4-2.fc19
rubygem-safe_yaml-0.9.4-2.fc19 has been pushed to the Fedora 19 testing repository.
rubygem-safe_yaml-0.9.4-2.fc19 has been pushed to the Fedora 19 stable repository.