Bug 818414

Summary:	Review Request: mruby - lightweight ruby
Product:	[Fedora] Fedora	Reporter:	Rick Elrod <relrod>
Component:	Package Review	Assignee:	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:	rawhide	CC:	notting, package-review
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
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:
Embargoed:

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.