[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
https://bugzilla.redhat.com/show_bug.cgi?id=728088 --- Comment #30 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
https://bugzilla.redhat.com/show_bug.cgi?id=728088 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
https://bugzilla.redhat.com/show_bug.cgi?id=728088 Troy Dawson changed: What|Removed |Added CC||tdaw...@redhat.com Flags|fedora-cvs+ |fedora-cvs? --- Comment #29 from Troy Dawson --- Package Change Request == Package Name: rubygem-idn New Branches: epel7 Owners: tdawson -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Fedora Update System changed: What|Removed |Added Fixed In Version||rubygem-idn-0.0.2-7.fc16 Resolution|RAWHIDE |ERRATA --- Comment #28 from Fedora Update System 2012-02-28 05:09:18 EST --- rubygem-idn-0.0.2-7.fc16 has been pushed to the Fedora 16 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #27 from Fedora Update System 2012-02-27 10:27:42 EST --- rubygem-idn-0.0.2-7.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/rubygem-idn-0.0.2-7.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #26 from Jon Ciesla 2012-02-27 10:08:11 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Mo Morsi changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #25 from Mo Morsi 2012-02-27 10:01:14 EST --- Package Change Request == Package Name: rubygem-idn New Branches: f16 Owners: mmorsi InitialCC: F16 branch for rubygem-idn -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #24 from Shawn Starr 2012-02-23 17:22:27 EST --- Please do request commit, never can have enough maintainers :) even for older versions -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #23 from Mo Morsi 2012-02-23 15:09:40 EST --- Could this be pushed into Fedora 16 as well. rubygem-webmock will be broken there until it is. I've attached the following SRPM / SPEC and diff to the idn spec adding ruby 1.8 / Fedora 16 support. I've also requested commit rights to the rawhide so that I can request the new branch / do the push if you don't have the cycles. Thanks. http://mo.morsi.org/files/rpms/rubygem-idn-0.0.2-7.fc18.src.rpm http://mo.morsi.org/files/rpms/rubygem-idn.spec http://mo.morsi.org/files/rpms/idn-0.0.2.spec.diff -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2012-01-29 14:48:06 --- Comment #22 from Shawn Starr 2012-01-29 14:48:06 EST --- This is now in -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added Resolution|NEXTRELEASE |RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #21 from Jon Ciesla 2012-01-11 20:35:17 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #20 from Shawn Starr 2012-01-11 19:26:52 EST --- n/m fedora-cvs is still used ;p -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #19 from Shawn Starr 2012-01-11 11:31:24 EST --- Can I get an update from the SCM team on this approval? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #18 from Shawn Starr 2012-01-02 15:07:26 EST --- New Package SCM Request === Package Name: rubygem-idn Short Description: Ruby Bindings for the GNU LibIDN library Owners: spstarr -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #17 from Shawn Starr 2011-12-20 12:40:59 EST --- I will be pushing this today was blocked due to fedora filer issue and being sick ;/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 James Laska changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #16 from James Laska 2011-11-29 08:50:04 EST --- I've set fedora_review+ as requested -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #15 from Mo Morsi 2011-11-28 07:31:52 EST --- For some reason the '+' isn't available as an option for the fedora-review flag. Does anyone know the reason for this? It has worked in the past for me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088
--- Comment #14 from Vít Ondruch 2011-11-22 03:46:46 EST
---
(In reply to comment #12)
> Removed copy %{SOURCE0}. I originally wanted to keep the .gem in the SRPM,
> What
> would happen if the gem upstream was compromised somehow?
There is even no internet connection on Koji during build, so there is no way
how the gem could be accidentally downloaded from some compromised source.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088
--- Comment #13 from Vít Ondruch 2011-11-22 03:44:43 EST
---
(In reply to comment #12)
> Removed copy %{SOURCE0}. I originally wanted to keep the .gem in the SRPM,
> What
> would happen if the gem upstream was compromised somehow?
The %{SROURCE0} is always kept in SRPM. You cannot build the SRPM without the
source gem. Moreover there is use --local flag during gem installation. The
copying of %{SOURCE0} did not have any meaning, did not prevent anything.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088
--- Comment #12 from Shawn Starr 2011-11-21 22:45:29
EST ---
Vit feedback is added:
using:testrb -Ilib test/tc_*.rb
Removed copy %{SOURCE0}. I originally wanted to keep the .gem in the SRPM, What
would happen if the gem upstream was compromised somehow?
Removed -devel package.
use %exclude for files above + .h file.
Please mark package review + so I can submit package into git.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #11 from Mo Morsi 2011-11-10 11:48:41 EST --- OK thanks for the update - looks like you addressed all my feedback - rpmlint / koji builds are still green - as Vit mentioned the cp command and devel packages can probably be removed unless there is good reason to keep those there. I feel it might be useful to keep the cached gem in place though as it's good for reference and in order to keep the files provided by rpm and gem installs fairly consistent. None of this is a big deal either way though - Vit's other feedback is also good but not blockers as the package is now in compliance w/ Fedora guidelines. Feel free to implement his suggestions pre/post push. APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088
Vít Ondruch changed:
What|Removed |Added
AssignedTo|shawn.st...@rogers.com |mmo...@redhat.com
--- Comment #10 from Vít Ondruch 2011-11-07 11:55:06 EST
---
This gem doesn't use RSpec, so this issue is irrelevant in this case. Please
don't worry too much about Ruby 1.9 compatibility. If there will be necessary
some changes, it will be just one of 300 gems, so no big deal.
However, you are currently executing the test suite using Rake. I would suggest
to replace the "rake test -v --trac" with something like "testrb -Ilib
test/tc_*.rb". That allows you to remove the dependency on RDoc and Rake and
the patch, which simplifies the maintenance of the library IMO.
Moreover:
* I would suggest to use %exclude macro instead of deleting files in install
section, i.e:
%files
%exclude %{geminstdir}/ext
%exclude %{geminstdir}/.yardoc
%exclude %{gemname}-%{version}.gem
* I suggest to use:
%exclude %{gemdir}/cache/%{gemname}-%{version}.gem
since the gem cache has no meaning with RPM.
* I suggest to remove the -devel package, because I doubt that it has some
meaning in this case. Will somebody except Ruby users base their work on the
gem?
* I am not exactly sure about the meaning of the "cp %{SOURCE0}
%{gemname}-%{version}.gem" command when you can simple call the gem install
command with the %{SOURCE0} macro.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #9 from Shawn Starr 2011-11-06 02:07:17 EST --- Stripped out non-gem, since we're not doing this any more (as mentioned before by Vit, it makes no sense) Updates can be found here: http://www.sh0n.net/spstarr/fedora/rubygem-idn/rubygem-idn.spec http://www.sh0n.net/spstarr/fedora/rubygem-idn/rubygem-idn-0.0.2-2.fc17.src.rpm Additionally, this does not use RSPec 2.x and as mentioned with rubygem-addressable any help to make these compliant would be greatly appreciated for Ruby 1.9.x and 1.8 compatibility. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 James Laska changed: What|Removed |Added CC||jla...@redhat.com --- Comment #8 from James Laska 2011-11-01 17:00:33 EDT --- Hi Shawn, same as rubygem-addressable ... I'm happy to add review feedback to help this review move forward. Let me know if I can help. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #7 from Shawn Starr 2011-10-26 23:54:31 EDT --- Yes, I've just been swamped with RL stuff, I hope to look at this and the other this weekend. These are pre-req's for OpenNebula 3.x also which will depend on how Ruby SIG will have for backwards compatibility or that will kill this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #6 from Mo Morsi 2011-10-26 14:18:46 EDT --- ping? could you fix the issues described in my review. rubygem-addressable needs this to work, which in return is required by rubygem-webmock, which in return is required by aeolus-conductor. Thank you -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Vít Ondruch changed: What|Removed |Added CC||vondr...@redhat.com --- Comment #5 from Vít Ondruch 2011-10-10 03:56:16 EDT --- (In reply to comment #2) > They also have dropped the non-gem use for rubygems (it's not updated in > packaging guidelines!). I proposed this change, since the non-gem use is bad designed and I can't see any benefit of such packages. However, there was not much done WRT update of packaging guidelines, but I'll try to collect some ideas in [1]. Going to add this proposal right now. (In reply to comment #3) > Sounds good. Good to know that the packaging rubygems for non-gem use > guideline > is going away, feel free to remove the non-gem subpackage from this altogether > if you want (this was the first package that I've seen do this in a very long > time). > > Also how definite is it that Ruby 1.9.3 is going into Fedora 17? > > Thank you. I am highly interested in R 1.9.3 in F17 and I did not heard any strong objections, so it is highly probable that I will push them forward. Of course, you can check Ruby-SIG ML as was mentioned by Shawn. [1] https://fedoraproject.org/wiki/Packaging_talk:Ruby -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #4 from Shawn Starr 2011-10-07 15:54:20 EDT --- The Ruby SIG team would need to be engaged they have a mailing list. Their changes will cause changes in these packages. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #3 from Mo Morsi 2011-10-07 06:47:01 EDT --- Sounds good. Good to know that the packaging rubygems for non-gem use guideline is going away, feel free to remove the non-gem subpackage from this altogether if you want (this was the first package that I've seen do this in a very long time). Also how definite is it that Ruby 1.9.3 is going into Fedora 17? Thank you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 --- Comment #2 from Shawn Starr 2011-10-06 20:30:59 EDT --- I will fix however, Ruby is going to 1.9.3 in Fedora 17 which is going to break a lot of things (compatibility?). I will fix these issues. They also have dropped the non-gem use for rubygems (it's not updated in packaging guidelines!). Thanks for reviewing further updates to this bug pending. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088
Mo Morsi changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||mmo...@redhat.com
Flag||fedora-review?
--- Comment #1 from Mo Morsi 2011-10-06 20:04:13 EDT ---
Taking this one
* rpmlint looks good
* koji build is green:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3407485
* The license refers to ASL and MIT but I was only able to find ASL in the
project, which components are licensed under MIT? The files you pointed out are
all licensed under the ASL
http://www.apache.org/licenses/LICENSE-2.0
* Also the NOTICE file states that
"This software contains code derived from the Net::XMPP for Ruby project by
Yuki Mitsui (http://netxmpp-ruby.jabberstudio.org/)"
This exact url does not seem to be accessible but I found the project's website
which indicates that it is licensed under the LGPL:
http://jabberstudio.org/projects/netxmpp/project/view.php
I'm not sure what they mean their code is "derived" from net-xmpp, but it might
be good to clarify (I believe licensing the project under ASL v2 AND LGPL would
be acceptable)
* rm -rf %{buildroot} is no longer needed at the beginning of the %install
section
* since you are packaging this gem for non-gem use, that non-gem subpackage
needs to own the ruby_sitelib symlinks according to these guidelines:
"- All the toplevel library files of the Gem must be symlinked into
ruby_sitelib.
- The subpackage must own these symbolic links."
Other than these, package looks good
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added OS/Version|All |Linux -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN library
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=728088 Shawn Starr changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|shawn.st...@rogers.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
