Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: rubygem-amqp - AMQP client implementation in Ruby/EventMachine
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Summary: Review Request: rubygem-amqp - AMQP client implementation in Ruby/EventMachine Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mkent@magoazul.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://magoazul.com/wip/SPECS/rubygem-amqp.spec SRPM URL: http://magoazul.com/wip/SRPMS/rubygem-amqp-0.6.7-1.fc14.src.rpm Description: An implementation of the AMQP protocol in Ruby/EventMachine for writing clients to the RabbitMQ message broker.
rpmlint /var/tmp/results/rubygem-amqp-* | grep -v '/1.8/doc/' rubygem-amqp.src: W: no-buildroot-tag rubygem-amqp-doc.noarch: W: no-documentation 3 packages and 0 specfiles checked; 0 errors, 8 warnings. (skipping the unexpanded-macro warnings in docs)
Build logs here https://koji.fedoraproject.org/koji/taskinfo?taskID=2062259
No test suite is provided by the author.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |mtasaka@ioa.s.u-tokyo.ac.jp Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #1 from Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp 2010-03-19 15:45:40 EDT --- Will review.
By the way I would appreciate it if you would review my review request (bug 575235)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |182235(FE-Legal)
--- Comment #2 from Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp 2010-03-21 09:36:48 EDT --- spot, would you tell me how I should handle the following license?
http://www.amqp.org/confluence/display/AMQP/AMQP+License
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #3 from Matthew Kent mkent@magoazul.com 2010-03-28 01:31:51 EDT --- ping!(In reply to comment #2)
spot, would you tell me how I should handle the following license?
ping! We waiting on a legal expert?
fwiw Red Hat provides an amqp messaging solution and is listed as a copyright holder: http://www.redhat.com/solutions/specifications/amqp/
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Tom "spot" Callaway tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tcallawa@redhat.com
--- Comment #4 from Tom "spot" Callaway tcallawa@redhat.com 2010-04-19 13:26:44 EDT --- So, that license is for the AMQP Specification, not the implementation. Unfortunately, the license is non-free, but it doesn't necessarily mean that the implementations of the AMQP specification are.
It does mean that we should not be shipping copies of the AMQP Specification in Fedora. Unfortunately, it looks like amqp-0.6.7/protocol/amqp-0.8.xml is either a copy of the specification or is under the AMQP specification license, which is a showstopper, unless it can be removed. (Note: amqp-0.6.7/protocol/amqp-0.8.json is MIT, which is fine.)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(mkent@magoazul.co | |m)
--- Comment #5 from Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp 2010-04-29 13:08:03 EDT --- Matthew, ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Matthew Kent mkent@magoazul.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(mkent@magoazul.co | |m) |
--- Comment #6 from Matthew Kent mkent@magoazul.com 2010-05-04 02:53:57 EDT --- (In reply to comment #5)
Matthew, ping?
Apologies for the slow reply.
I'm assuming I'm okay to just drop the file? It's not required by the code at all.
Spec URL: http://magoazul.com/wip/SPECS/rubygem-amqp.spec SRPM URL: http://magoazul.com/wip/SRPMS/rubygem-amqp-0.6.7-2.fc14.src.rpm
* Mon May 3 2010 Matthew Kent mkent@magoazul.com - 0.6.7-2 - Remove amqp specification with incompatible license
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|182235(FE-Legal) | Flag|fedora-review? |fedora-review+
--- Comment #7 from Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp 2010-05-05 09:16:15 EDT --- Okay, this can be approved. spot, thank you for clarifying license issue.
------------------------------------------------------------- This package (rubygem-amqp) is APPROVED by mtasaka -------------------------------------------------------------
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Matthew Kent mkent@magoazul.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Matthew Kent mkent@magoazul.com 2010-05-06 03:22:11 EDT --- New Package CVS Request ======================= Package Name: rubygem-amqp Short Description: AMQP client implementation in Ruby/EventMachine Owners: mkent Branches: F-11 F-12 F-13 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #9 from Kevin Fenzi kevin@tummy.com 2010-05-06 11:25:18 EDT --- CVS done (by process-cvs-requests.py).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2010-05-10 02:53:11 EDT --- rubygem-amqp-0.6.7-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/rubygem-amqp-0.6.7-2.fc11
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2010-05-10 02:54:15 EDT --- rubygem-amqp-0.6.7-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/rubygem-amqp-0.6.7-2.fc12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2010-05-10 02:55:24 EDT --- rubygem-amqp-0.6.7-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/rubygem-amqp-0.6.7-2.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2010-05-10 12:56:16 EDT --- rubygem-amqp-0.6.7-2.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |rubygem-amqp-0.6.7-2.fc11 Resolution| |ERRATA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2010-05-10 13:07:43 EDT --- rubygem-amqp-0.6.7-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|rubygem-amqp-0.6.7-2.fc11 |rubygem-amqp-0.6.7-2.fc12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|rubygem-amqp-0.6.7-2.fc12 |rubygem-amqp-0.6.7-2.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2010-05-10 19:48:00 EDT --- rubygem-amqp-0.6.7-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
Michael Stahnke mastahnke@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mastahnke@gmail.com Flag|fedora-cvs+ |fedora-cvs?
--- Comment #16 from Michael Stahnke mastahnke@gmail.com 2010-09-10 14:37:33 EDT --- Package Change Request ====================== Package Name: rubygem-amqp New Branches: el5 el6 Owners: stahnma
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=574977
--- Comment #17 from Kevin Fenzi kevin@tummy.com 2010-09-10 14:46:03 EDT --- Git done (by process-git-requests).
package-review@lists.fedoraproject.org