Bug 1032186

(Just a note that this RPM requires rubygem-rots, which is not yet available in the F20 and F19 stable repos: https://admin.fedoraproject.org/updates/rubygem-rots . You can do the review in a Rawhide buildroot until rubygem-rots gets pushed to stable for those branches.)

(rubygem-rots is in F20 and F19, so rubygem-rack-openid can build on those versions of Fedora without issue.)

I've updated to the latest upstream version.

* Wed Nov 27 2013 Ken Dreyer <ktdreyer> - 1.4.1-1
- Update to rack-openid 1.4.1
- Ship license file from upstream Git

Spec URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-rack-openid.spec
SRPM URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-rack-openid-1.4.1-1.fc21.src.rpm

F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6230770

Taking.

I would appreciate it if you would review one of my review requests (e.g. bug 1035129 )

Thanks for taking the review.

I just realized that 1.4.1-1 contains a bad copy-and-paste from another gem. This text:

  # Until upstream ships the full text of the ASL 2.0 in a released gem, we will
  # ship the version from Git master.
  install -p -m 0644 %{SOURCE1} %{buildroot}%{gem_instdir}/LICENSE

needs to use %{SOURCE3} instead of %{SOURCE1}, and the comment needs to say "MIT", rather than "ASL 2.0". Whoops! I can fix this after your full review.

Some other comments:

* Unpacking tarball in %prep
----------------------------------------------------
    49  %setup -q -D -T -n %{gem_name}-%{version}
    53  tar -xJf %{SOURCE2}
----------------------------------------------------
  - I guess "%setup -q -D -T -n %{gem_name}-%{version} -a 2" is
    better.

* For Source1
  - It is better to add "set -e"
  - And I would remove rack-openid-$VERSION.zip (not a blocker)
  - Also it is better to suppress this rpmlint:
-----------------------------------------------------
rubygem-rack-openid.src: W: strange-permission rubygem-rack-openid-generate-test-tarball.sh 0775L
-----------------------------------------------------

Please check the comments above, and also your previous comments.

Thanks for the suggestions. I've fixed all of them except the strange-permission error in rpmlint. I did change it from 775 to 755, but rpmlint is still unhappy that the permissions are not 644. To me it makes sense to keep the script executable. 

* Thu Nov 28 2013 Ken Dreyer <ktdreyer> - 1.4.1-2
- Address issues from package review (RHBZ #1032186)
- Correct comment about LICENSE
- Unpack the tests tarball during %%setup
- Use "set -e" in tests tarball generation script
- Clean up zip file in tests tarball generation script

Exact changes in Git: http://fedorapeople.org/cgit/ktdreyer/public_git/rubygem-rack-openid.git/commit/?id=ca565b5b5a4322bcb8df6ae616055feb7cf46db6

Spec URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-rack-openid.spec
SRPM URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-rack-openid-1.4.1-2.fc21.src.rpm

F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6235787

Approving.

----------------------------------------------------
  This package (rubygem-rack-openid) is APPROVED
  by mtasaka
----------------------------------------------------

Thanks Mamoru for the review!

New Package SCM Request
=======================
Package Name: rubygem-rack-openid
Short Description: Provides a more HTTPish API around the ruby-openid library
Owners: ktdreyer
Branches: f19 f20

Git done (by process-git-requests).

rubygem-rack-openid-1.4.1-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/rubygem-rack-openid-1.4.1-2.fc20

rubygem-rack-openid-1.4.1-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/rubygem-rack-openid-1.4.1-2.fc19

rubygem-rack-openid-1.4.1-2.fc20 has been pushed to the Fedora 20 testing repository.

rubygem-rack-openid-1.4.1-2.fc19 has been pushed to the Fedora 19 stable repository.

rubygem-rack-openid-1.4.1-2.fc20 has been pushed to the Fedora 20 stable repository.