Summary: | dev-ruby/ruby-rdig-0.3.5: new ebuild | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Wim Muskee <wimmuskee> |
Component: | New packages | Assignee: | Default Assignee for New Packages <maintainer-wanted> |
Status: | RESOLVED WONTFIX | ||
Severity: | enhancement | CC: | ruby |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | x86 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
new ebuild for dev-ruby/ruby-rdig-0.3.5
new ebuild (from source) for dev-ruby/rdig patch for bin/rdig file improved ebuild (from source) for dev-ruby/rdig improved patch for bin/rdig file loader file for rdig |
Description
Wim Muskee
2008-12-21 23:54:31 UTC
Created attachment 176115 [details]
new ebuild for dev-ruby/ruby-rdig-0.3.5
Thanks for submitting the ebuild. The ruby herd is currently still understaffed, so I'm not sure how quickly we can process it. A few notes: - since the name of the package is just rdig, it might be better to call the package like that if there is no obvious collision with another package; - is this primarily used as a Ruby library or is it primarily used for indexing; in the latter case it might be better suited as app-text/rdig; - if upstream provides a non-gem release it might be much better to not use the gem (see http://blog.flameeyes.eu/2008/12/12/id-rather-keep-myself-away-from-gems for some reasoning and bug #252257 as an example of where a working FEATURES=test is quite helpful). Created attachment 176545 [details]
new ebuild (from source) for dev-ruby/rdig
Installs from source instead of gem. Did not use the install.rb file, but instead copied everything manually. The bin/rdig file was patched (see bin-rdig.patch) to be run from commandline without rubygems.
Unfortunately $LOAD_PATH has to be loaded in each ruby script calling rdig. This is not necessary in the gem ebuild version.
Created attachment 176546 [details, diff]
patch for bin/rdig file
Created attachment 176610 [details]
improved ebuild (from source) for dev-ruby/rdig
An added loader file (files/rdig.rb) loads the appdir into $LOAD_PATH, making the call in each ruby script needing rdig not necessary anymore. a3li ;)
Created attachment 176612 [details, diff]
improved patch for bin/rdig file
removed $LOAD_PATH line, this is done by the new loader file (files/rdig.rb)
Created attachment 176614 [details]
loader file for rdig
Never mind looking for a maintainer for this package. The last commit on the source was three years ago, and I'm not using it anymore. |