[Bug 639615] Review Request: rubygem-rvm - Ruby Version Manager

bugzilla at redhat.com bugzilla at redhat.com
Fri Oct 29 08:46:06 UTC 2010


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=639615

Michal Fojtik <mfojtik at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?(mfojtik at redhat.co |
                   |m)                          |

--- Comment #5 from Michal Fojtik <mfojtik at redhat.com> 2010-10-29 04:46:05 EDT ---
(In reply to comment #4)
> Any word on this, Michal?

Hi, sorry for the delay, but I was terribly busy last days. I fixed all things
you mentioned above also moved README to main package and source.
However, there is also one issue with installing this package and running
rvm-install command, which will need some more patching/work:

/usr/bin/rvm-install:19:in `load':
/usr/lib/ruby/gems/1.8/gems/rvm-1.0.11/bin/rvm-install:3: syntax error,
unexpected tIDENTIFIER, expecting kDO or '{' or '(' (SyntaxError)
/usr/lib/ruby/gems/1.8/gems/rvm-1.0.11/bin/rvm-install:6: syntax error,
unexpected tSTRING_BEG, expecting kDO or '{' or '('
if [[ -z "$rvm_selfcontained" ]]; then
          ^
/usr/lib/ruby/gems/1.8/gems/rvm-1.0.11/bin/rvm-install:6: syntax error,
unexpected ']', expecting $end
if [[ -z "$rvm_selfcontained" ]]; then
                               ^
        from /usr/bin/rvm-install:19

Btw. Thank you for review. I'll push fixed version very soon.


* Fri Oct 29 2010 Michal Fojtik <mfojtik at redhat.com> - 1.0.11-3
- LICENSE and README moved to main package

Spec URL: http://mifo.sk/RPMS/rubygem-rvm.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-rvm-1.0.11-3.fc13.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.



More information about the package-review mailing list