Bug 818414

Summary: Review Request: mruby - lightweight ruby
Product: [Fedora] Fedora Reporter: Rick Elrod <relrod>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: notting, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-04-03 03:41:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Rick Elrod 2012-05-03 03:14:28 UTC
Spec URL: http://codeblock.fedorapeople.org/packages/mruby/mruby.spec
SRPM URL: http://codeblock.fedorapeople.org/packages/mruby/mruby-mruby-1032e9e.tar.gz
Description: This is a lightweight implementation of Ruby created by the author of the original Ruby interpreter ("matz"). https://github.com/mruby/mruby

Comment 1 Rick Elrod 2012-05-03 03:22:19 UTC
Well I messed that one up. :)

Correct SRPM: http://codeblock.fedorapeople.org/packages/mruby/mruby-1.0.0-1.20120502git1032e9e.fc18.src.rpm

Comment 2 Rick Elrod 2012-05-26 05:46:00 UTC
Updating to upstream HEAD.

[ricky@t520 RPMS]$ rpmlint ./x86_64/mruby-1.0.0-1.20120526gitf6d539e.fc18.x86_64.rpm 
mruby.x86_64: W: no-manual-page-for-binary mruby
mruby.x86_64: W: no-manual-page-for-binary mrbc
mruby.x86_64: W: no-manual-page-for-binary mirb
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

[ricky@t520 rpmbuild]$ rpmlint ./SPECS/mruby.spec 
./SPECS/mruby.spec:48: E: hardcoded-library-path in %{buildroot}/usr/lib/libmruby.a
./SPECS/mruby.spec: W: invalid-url Source0: mruby-mruby-f6d539e.tar.gz
0 packages and 1 specfiles checked; 1 errors, 1 warnings.

The first one is because no matter what I try, cmake is insisting on installing a static lib into /usr/lib/, regardless of architecture. But because it's a static lib, it gets rm'd anyway.

The second is because this is pulled from Github, which has no way to easily link to tarballs of commits. Instructions are included in the specfile for obtaining the tar.gz.

New links:

Spec URL: http://codeblock.fedorapeople.org/packages/mruby/mruby.spec
SRPM URL: http://codeblock.fedorapeople.org/packages/mruby/mruby-1.0.0-1.20120526gitf6d539e.fc18.src.rpm

Comment 3 Rick Elrod 2013-04-03 03:41:08 UTC
I'm going to close this out for now.

I might fix up and open a new review at some point.