[Bug 1015778] Review Request: rubygem-ruby-openid - A library for consuming and serving OpenID identities

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1015778



--- Comment #8 from Ken Dreyer <ktdreyer@xxxxxxxxxxxx> ---
(As a correction to comment #6: my license clarification request is at
https://github.com/openid/ruby-openid/issues/64, not 60.)

(In reply to Mamoru TASAKA from comment #7)
>     * Note that the executable permission on 
>       /examples/rails_openid/script/rails,
>       /examples/discover adds the dependency "/usr/bin/env" to -doc
>       subpackage. You may also want to suppress this dependency
>       (by also removing executable permission on these files, or
>        to add filtering macro again (which is contrary to what
>        I said before, however for this case it seems okay) )

For the /usr/bin/env dependency in -doc, I switched that to /usr/bin/ruby.
Please let me know if you're ok with this, or if you'd like me to adjust it
further (ie. remove it altogether.)

The PNGs' executable bits is a bug in the rdoc gem. I've submitted
https://github.com/rdoc/rdoc/pull/258 to fix this.

Here's the newest version.

* Sun Oct 27 2013 Ken Dreyer <ktdreyer@xxxxxxxxxxxx> - 2.3.0-3
- Updates for review request (RHBZ #1015778)
- Update obsoletes
- Update license
- Remove /usr/bin/env from -doc auto-requires

Specific changes (in git):
http://fedorapeople.org/cgit/ktdreyer/public_git/rubygem-ruby-openid.git/commit/?id=6b384da08eb75cdf95572af52f90393dbaa256ae

Spec: http://ktdreyer.fedorapeople.org/reviews/rubygem-ruby-openid.spec
SRPM:
http://ktdreyer.fedorapeople.org/reviews/rubygem-ruby-openid-2.3.0-3.fc21.src.rpm

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]