[Bug 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
https://bugzilla.redhat.com/show_bug.cgi?id=738746 Vít Ondruch vondr...@redhat.com changed: What|Removed |Added CC||vondr...@redhat.com --- Comment #11 from Vít Ondruch vondr...@redhat.com --- Hi Slavek, Since Fotios seems to be busy, I updated his package to the latest coffee-script-source version to keep going forward: SPEC: http://people.redhat.com/vondruch/rubygem-coffee-script-source.spec SRPM: http://people.redhat.com/vondruch/rubygem-coffee-script-source-1.3.3-1.fc18.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4243247 Please note that the test suite is not possible to Run. The rubygem-coffee-script is testing this package. In ideal world, this package should not be needed, since we would have independent coffee-script package, build from source. But that would be hard. Since there is exception for javascript bundling, I would prefer to not dive into this mess. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
https://bugzilla.redhat.com/show_bug.cgi?id=738746 Bohuslav Slavek Kabrda bkab...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #12 from Bohuslav Slavek Kabrda bkab...@redhat.com --- In ideal world, this package should not be needed, since we would have independent coffee-script package, build from source. But that would be hard. Since there is exception for javascript bundling, I would prefer to not dive into this mess. Agreed. Not much to review here :) The only thing that comes to my mind is that it might be nice to ask upstream to include a license file in the package, but this is not a blocker. This package is APPROVED. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
https://bugzilla.redhat.com/show_bug.cgi?id=738746 Vít Ondruch vondr...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #13 from Vít Ondruch vondr...@redhat.com --- @Slavek: Thank you for the review. I'll consider it :) https://github.com/jashkenas/coffee-script/issues/2443 @Fotios: I am going to request the git repository. Please ask for co-maintaining or I am completely fine to hand the package over to you if you like. Just let me know. New Package SCM Request === Package Name: rubygem-coffee-script-source Short Description: The CoffeeScript Compiler Owners: vondruch Branches: InitialCC: -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #10 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-26 04:58:38 EDT --- Hi Fotios, any progress on this? We will need this for new rails in F18, so it would be great to have it ASAP. Thanks. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #8 from Fotios Lindiakos fot...@redhat.com 2012-02-29 10:44:31 EST --- Ok, followed your suggestions and here is a new package (note: SPEC is the same location because it's controlled via git) SPEC: https://raw.github.com/fotioslindiakos/rubygem-all/master/SPECS/rubygem-coffee-script-source.spec SRPM: https://github.com/fotioslindiakos/rubygem-all/raw/master/SRPMS/rubygem-coffee-script-source-1.2.0-2.fc16.src.rpm KOJI: http://koji.fedoraproject.org/koji/taskinfo?taskID=3840346 As for tests, there are tests upstream under the coffee-script package (which includes this source package and a js runtime), but they can't be run without additional dependencies. IMHO, since this source package, installing a javascript runtime just for running the tests would defeat the purpose of having a package with no deps. But this is my first package, so if that is a common thing to do, let me know, I'd be happy to take care of it. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #9 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-03-01 01:30:18 EST --- (In reply to comment #8) Ok, followed your suggestions and here is a new package (note: SPEC is the same location because it's controlled via git) SPEC: https://raw.github.com/fotioslindiakos/rubygem-all/master/SPECS/rubygem-coffee-script-source.spec SRPM: https://github.com/fotioslindiakos/rubygem-all/raw/master/SRPMS/rubygem-coffee-script-source-1.2.0-2.fc16.src.rpm KOJI: http://koji.fedoraproject.org/koji/taskinfo?taskID=3840346 As for tests, there are tests upstream under the coffee-script package (which includes this source package and a js runtime), but they can't be run without additional dependencies. IMHO, since this source package, installing a javascript runtime just for running the tests would defeat the purpose of having a package with no deps. But this is my first package, so if that is a common thing to do, let me know, I'd be happy to take care of it. Well, my idea is, that you only run tests in Build time, not runtime. That way, you may have many BR:, but runtime requires will stay the same. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #6 from Fotios Lindiakos fot...@redhat.com 2012-02-28 09:47:58 EST --- Updated the package to build against rawhide and Ruby 1.9.1. Links to updated files are included. I also successfully build the package in koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3827994 SPEC: https://raw.github.com/fotioslindiakos/rubygem-all/master/SPECS/rubygem-coffee-script-source.spec SRPM: https://github.com/fotioslindiakos/rubygem-all/raw/master/SRPMS/rubygem-coffee-script-source-1.2.0-1.fc16.src.rpm -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #7 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-02-29 02:07:02 EST --- - Please bump the release when doing changes (you can use rpmdev-bumpspec) - or if you are updating to newer version, you still need to add a changelog entry, that notes it. - According to [1], you can use the new macros for gem files and directories. The only thing you need to do is define %gem_name, then you will have %gem_instdir, %gem_cache, etc... Also, you should now have BR: rubygems-devel instead of BR: ruby(rubygems) (the runtime dependency however stays on ruby(rubygems) ). - If there are tests available in the upstream, that are not present in the gem itself, it a good custom to run them. You can see rubygem-delorean as an example how to do that (you have to check them out from git separately in the correct version and add them as another source). [1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby#Macros -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #5 from Fotios Lindiakos fot...@redhat.com 2012-02-27 11:55:52 EST --- Sorry, fell onto the back burner. I'm doing it right now, I'll update when I have a new package for the latest version built against the new guidelines -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #4 from Bohuslav Slavek Kabrda bkab...@redhat.com 2012-02-27 02:37:26 EST --- Hi Fotios, any progress on this one? It is a requirement for Rails 3.2 stack, which we would like to get to Rawhide ASAP, so we can have enough time to fix all the things it will break. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 Bohuslav Kabrda bkab...@redhat.com changed: What|Removed |Added CC||bkab...@redhat.com --- Comment #1 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 07:42:52 EST --- I'll take this one. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 Bohuslav Kabrda bkab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|bkab...@redhat.com Flag||fedora-review? -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #2 from Bohuslav Kabrda bkab...@redhat.com 2012-01-30 07:56:36 EST --- - Please update to the latest upstream version (currently 1.2.0). - Consider excluding cached gem. This is only used with gem pristine command, which is irrelevant for RPM gems, as they should be restored by yum reinstall. * You probably haven't noticed, but we are moving to Ruby 1.9.3 in rawhide, which involves some changes to the gem packaging - see [1] for new guidelines draft. Some information can be found on Ruby-sig fedora mailing list [2] (you should subscribe to that one, if you haven't already). * Currently, we only build new packages for Ruby 1.9.3 in its special target f17-ruby (again, all the info is is [2]). * I suggest you update to the latest version, then we can sort out any leftovers and I can also help you with the new-style packaging. The package looks good, so I think it shouldn't be a problem. [1] https://fedoraproject.org/wiki/PackagingDrafts/Ruby [2] http://lists.fedoraproject.org/pipermail/ruby-sig/2012-January/000805.html -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 --- Comment #3 from Fotios Lindiakos fot...@redhat.com 2012-01-30 10:08:30 EST --- Thanks for picking this up! I'll look over those suggestions and update the package to the newest version. -- 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 738746] Review Request: rubygem-coffee-script-source - A package for the coffee-script-source Ruby gem
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=738746 Fotios Lindiakos fot...@redhat.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR), ||738742 -- 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