Hello,

Vít Ondruch wrote, at 12/14/2011 10:33 PM +9:00:
Hi,

I have uploaded updated version of Ruby 1.9.3 packages into my testing 
repository.
They are probably very close to the shape of Ruby I'd like to see in future 
Fedoras.

First of all, I appreciate your work on ruby 193 packaging. Then I leave some 
quick
comments now.

The main changes are:

- Install RubyGems library outside of Ruby directory structure into 
/usr/share/rubygems folder.
  This should allows us to share RubyGems among different Ruby implementations.

- Installed gems are now divided into different directories. Gems installed by 
regular user goes into his/her home directory,
  gems installed by root goes to /usr/local/ directory, while the gems 
installed by RPM will go into /usr directory.

I don't this behavior is right. If gems installed by regular users goes into 
under their home
directory, so should be on root because root is one of the users and we should 
prevent root's
installing gems under root's home directory.


- "gem install" now installs gems into share/gems directory while their binary 
extensions goes into lib{64}/gems
  directory. This (in theory) allows to install binary gems side by side 
without conflicts.
- RubyGems has now its own -devel subpackage, which will be need in the future 
for build of RPM packages.
- Enhanced macros.ruby and macros.rubygems.
- All tests are green now (credits goes to bkabrda).

The detailed changelog you can find at GitHub repo [1].

Some packaging issue.
- Please check directory ownership issue.
  - For example, the directory %{ruby_libarchdir}/digest/ or so are not owned by
    any packages.
  - The directory %_libdir/pkgconfig should not be owned by ruby-devel.
- The dependency of -tcltk subpackage on -libs package should be %_isa specific.
- Maybe ri directory should be moved to %_libdir/ri for now?
- It seems that %{ruby_libdir}/rbconfig should belong to rubygems, not 
ruby-libs.
- build.log just shows:
--------------------------------------------------
compiling main.c
compiling dmydln.c
compiling dmyencoding.c
compiling version.c
--------------------------------------------------
  or so, It is hard to check from this log if Fedora specific compilation flags 
are
  passed correctly or not. Please make build.log more verbose so that we can
  see what commands are actually executed during build.
- Isn't COPY="cp -p" needed also on %install? Also
  "cp %{SOURCE1} %{buildroot}%{rubygems_dir}/rubygems/defaults"
  in %spec file should be replaced by "cp -p".
- Consider to move mkmf.rb to ruby-devel:
--------------------------------------------------
<mock-chroot>[root@localhost /]# rpm -q ruby
ruby-1.9.3.0-2.fc17.i686
<mock-chroot>[root@localhost /]# rpm -q ruby-devel
package ruby-devel is not installed
<mock-chroot>[root@localhost /]# ruby -e "require 'mkmf'"
mkmf.rb can't find header files for ruby at /usr/share/include/ruby.h
--------------------------------------------------
- rubygems rpm contains "bigdecimal-1.1.0.gemspec  io-console-0.3.gemspec
  json-1.5.4.gemspec minitest-2.5.1.gemspec"??
  - At least these components should have its own subpackage rpms.
  - Also, for example the latest minitest is 2.9.1. rubygems (or 
rubygem-minitest
    built from ruby.... too complicated) should use latest minitest
  (and also please re-check components bundled in ruby - I hope
   ruby upstream will again unbundle these components ... this is
   again too complicated ..)
- include/ruby/ contains origuruma.h, however origuruma is separately
  packaged on Fedora.
  http://koji.fedoraproject.org/koji/packageinfo?packageID=5432
  Can ruby use system-widely provided origuruma?
  If not, what prevents it?

If you have previously installed Ruby
1.9.3 from my testing repository, you should be able to simply do:
# yum update ruby
However, please note that due to directory structure changes, you might need 
reinstall all your gems.
And as always, any feedback is appreciated.

Vit


[1] https://github.com/voxik/ruby.spec

Regards,
Mamoru


_______________________________________________
ruby-sig mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/ruby-sig

Reply via email to