[Bug 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #1 from Vít Ondruch 2011-01-07 09:56:16 EST --- Buildroot cleanup added: Spec URL: http://people.redhat.com/vondruch/rubygem-arel.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-arel-2.0.6-2.fc14.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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #2 from Vít Ondruch 2011-01-07 10:35:55 EST --- All documentation moved into subpackage Spec URL: http://people.redhat.com/vondruch/rubygem-arel.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-arel-2.0.6-3.fc14.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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Mohammed Morsi changed: What|Removed |Added Flag|fedora-cvs? |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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Mohammed Morsi changed: What|Removed |Added Status|NEW |ASSIGNED CC||mmo...@redhat.com AssignedTo|nob...@fedoraproject.org|mmo...@redhat.com Flag||fedora-cvs? --- Comment #3 from Mohammed Morsi 2011-01-18 20:14:31 EST --- OK. A few comments. * rpmlint output looks fine save 'no-%build-section', could you add one * It turns out only rubygem-arel 2.0.2 is compatible w/ rails 3.0.3. Anything above that will cause errors in activerecord. Would you consider submitting arel 2.0.2 instead? Other than that, 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #4 from Mohammed Morsi 2011-01-18 21:09:30 EST --- Koji build looks good http://koji.fedoraproject.org/koji/taskinfo?taskID=2730275 -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #5 from Vít Ondruch 2011-01-24 09:26:07 EST --- (In reply to comment #3) > * It turns out only rubygem-arel 2.0.2 is compatible w/ rails 3.0.3. Anything > above that will cause errors in activerecord. Would you consider submitting > arel 2.0.2 instead? How did you tested please? This is list of gems I have on my system: [vondruch@dhcp-25-1 activerecord ((v3.0.3) *)]$ gem list *** LOCAL GEMS *** abstract (1.0.0) actionmailer (3.0.3) actionpack (3.0.3) activemodel (3.0.3) activerecord (3.0.3) activeresource (3.0.3) activesupport (3.0.3) addressable (2.2.3) archive-tar-minitar (0.5.2) arel (2.0.7) builder (2.1.2) bundler (1.0.9) columnize (0.3.1) erubis (2.6.6) faker (0.9.4) gem2rpm (0.6.0) horo (1.0.3) i18n (0.5.0) json (1.4.3) linecache (0.43) mail (2.2.14) memcache-client (1.8.5) mime-types (1.16) mocha (0.9.8) nokogiri (1.4.3.1) polyglot (0.3.1) rack (1.2.1) rack-mount (0.6.13) rack-test (0.5.7) rails (3.0.3) railties (3.0.3) rake (0.8.7) rbench (0.2.3) rdoc (3.4) ruby-debug (0.10.4) ruby-debug-base (0.10.4) ruby-prof (0.9.2) ruby_core_source (0.1.4) sqlite3 (1.3.3) sqlite3-ruby (1.3.3) system_timer (1.0) text-format (1.0.0) text-hyphen (1.0.0) thor (0.14.6) treetop (1.4.9) tzinfo (0.3.24) yajl-ruby (0.7.9) and running: [vondruch@dhcp-25-1 activerecord ((v3.0.3) *)]$ rake test_sqlite3 produces no error: Finished in 402.826755 seconds. 2380 tests, 7383 assertions, 0 failures, 0 errors -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #6 from Mohammed Morsi 2011-01-24 16:59:12 EST --- Created attachment 475056 --> https://bugzilla.redhat.com/attachment.cgi?id=475056 error building activerecord 3.0.3 against arel 2.0.6 $ gem list *** LOCAL GEMS *** activemodel (3.0.3) activesupport (3.0.3) arel (2.0.6) builder (2.1.2) hoe (2.6.2) i18n (0.4.2) json (1.4.3) minitest (1.6.0) mocha (0.9.8) rake (0.8.7) rubyforge (2.0.4) sqlite3-ruby (1.2.4) tzinfo (0.3.24) All installed via yum from Fedora and the locally built copies. When I build the rubygem-activerecord package with the following spec http://mo.morsi.org/files/rpms/rubygem-activerecord.spec Changing the arel dependency to 2.0.6, I get the attached errors in the activerecord spec suite -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #7 from Vít Ondruch 2011-01-28 11:19:35 EST --- I can't help myself but I see no failure for AR on my machine: Finished in 308.784244 seconds. 2374 tests, 7372 assertions, 0 failures, 0 errors + exit 0 Processing files: rubygem-activerecord-3.0.3-1.fc14.noarch Provides: rubygem(activerecord) = 3.0.3 I have the same gems as you except hoe, which is already updated in repository. Well actually also except Arel, because I have already update to Arel 2.0.7: Spec URL: http://people.redhat.com/vondruch/rubygem-arel.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-arel-2.0.7-1.fc14.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2747780 I hope I will find time to test the AR build in mock on my computer at Monday. Anyway I believe we should continue with Arel 2.0.7 and solve problem with AR later. -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #8 from Vít Ondruch 2011-01-31 06:28:07 EST --- I have tested activerecord in mock today and the result is: 1) I can build activerecord with Arel-2.0.2 or 2.0.7 equally well when I use sqlite3-ruby 1.3.3 2) I can't build activerecord with Arel-2.0.2 neither with 2.0.7 while using rubygem-sqlite3-ruby 1.2.4, which is the default in rawhide. Going to ping sqlite3 maintainers for update. However I am not sure what impact it will have on other packgages ... -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #9 from Vít Ondruch 2011-01-31 06:30:02 EST --- This is the bugtrace: /usr/lib/ruby/gems/1.8/gems/sqlite3-ruby-1.2.4/lib/sqlite3/driver/native/driver.rb:76: [BUG] Segmentation fault ruby 1.8.7 (2010-12-23 patchlevel 330) [x86_64-linux] rake aborted! Command failed with status (): [/usr/bin/ruby -w -I"lib:test:test/connecti...] /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:994:in `sh' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1009:in `call' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1009:in `sh' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1093:in `sh' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1028:in `ruby' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1093:in `ruby' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake/testtask.rb:116:in `define' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb::in `verbose' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake/testtask.rb:101:in `define' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:635:in `call' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:635:in `execute' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:630:in `each' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:630:in `execute' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:596:in `invoke_with_call_chain' /usr/lib/ruby/1.8/monitor.rb:242:in `synchronize' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:589:in `invoke_with_call_chain' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:582:in `invoke' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2050:in `invoke_task' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2028:in `top_level' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2028:in `each' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2028:in `top_level' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in `standard_exception_handling' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2022:in `top_level' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2000:in `run' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:2067:in `standard_exception_handling' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/lib/rake.rb:1997:in `run' /usr/lib/ruby/gems/1.8/gems/rake-0.8.7/bin/rake:31 /usr/bin/rake:19:in `load' /usr/bin/rake:19 -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #10 from Mohammed Morsi 2011-01-31 16:33:45 EST --- Created attachment 476270 --> https://bugzilla.redhat.com/attachment.cgi?id=476270 activerecord build in mock against arel 2.0.2 and 2.0.7 I'm not quite sure how we are running into different results using mock. Attached are the commands I ran and their output to build activerecord in via mock. Once again it works with arel 2.0.2 but not 2.0.7. Before I ran this I removed the specific version of the arel dependency in the activerecord rpm so as to be able to build against both 2.0.2 and 2.0.7. In both cases I'm running the test suite against the stock sqlite3-ruby gem in Fedora (1.2.4) Does anything look different between our approaches? -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #11 from Mohammed Morsi 2011-01-31 16:38:29 EST --- Comment on attachment 476270 --> https://bugzilla.redhat.com/attachment.cgi?id=476270 activerecord build in mock against arel 2.0.2 and 2.0.7 Please ignore this line in the attachment: "mock -r fedora-14-x86_64 --copyin rubygem-activemodel-3.0.3-2.fc14.src.rpm /tmp/^C" Notice I canceled it before it was run (the ^C). The activerecord srpm is copied in a little later afterwards. -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #12 from Vít Ondruch 2011-02-02 03:16:49 EST --- If nothing else, you are using older mock. I have mock 1.1.8 on my system while you have mock 1.1.5. Also, I would say that you should try to build preferably against rawhide instead of f14, because that is the version we are targeting. There is also unclear if we are on the same version of Sqlite for example. -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #13 from Vít Ondruch 2011-02-02 03:59:28 EST --- My Sqlite versions: [vondruch@dhcp-25-1 fedora-scm]$ mock shell INFO: mock.py version 1.1.8 starting... State Changed: init plugins INFO: selinux enabled State Changed: start State Changed: lock buildroot mock-chroot> rpm -q sqlite sqlite-3.6.23.1-1.fc14.x86_64 mock-chroot> exit State Changed: unlock buildroot [vondruch@dhcp-25-1 fedora-scm]$ mock -r fedora-devel shell INFO: mock.py version 1.1.8 starting... State Changed: init plugins INFO: selinux enabled State Changed: start State Changed: lock buildroot mock-chroot> rpm -q sqlite sqlite-3.7.4-1.fc15.x86_64 -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Vít Ondruch changed: What|Removed |Added Depends on||674587 -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Mohammed Morsi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #14 from Mohammed Morsi 2011-02-02 13:56:29 EST --- OK thank you greatly for this rpm and your assistance. I was able to get it working on mock on F14 with your latest SRPM. I also verified the updated sqlite3 gem is needed to build activerecord (will review that next). * package passes the review guidelines * package builds fine on koji against dist-rawhide * rpmlint is fine for the most part, complains about a missing %build section, please add (even if its empty) before you push Other than that everything looks great. 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Vít Ondruch changed: What|Removed |Added Flag||fedora-cvs? --- Comment #15 from Vít Ondruch 2011-02-03 05:30:31 EST --- Thank you for your review! New Package SCM Request === Package Name: rubygem-arel Short Description: Arel is a Relational Algebra for Ruby Owners: vondruch Branches: InitialCC: -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 --- Comment #16 from Kevin Fenzi 2011-02-03 15:28:44 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Vít Ondruch changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-02-05 03:21:39 -- 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 667954] Review Request: rubygem-arel - Arel is a Relational Algebra for Ruby
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=667954 Bug 667954 depends on bug 674587, which changed state. Bug 674587 Summary: Review Request: rubygem-sqlite3 - Allows Ruby scripts to interface with a SQLite3 database https://bugzilla.redhat.com/show_bug.cgi?id=674587 What|Old Value |New Value Resolution||RAWHIDE Status|ASSIGNED|CLOSED -- 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