[Bug 727780] Review Request: qtruby - Ruby bindings for QT4

bugzilla at redhat.com bugzilla at redhat.com
Thu Aug 4 13:00:07 UTC 2011


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

--- Comment #3 from Vít Ondruch <vondruch at redhat.com> 2011-08-04 09:00:06 EDT ---
(In reply to comment #2)
> (In reply to comment #1)
> > Name: ? shouldn't we name it according ruby package naming scheme ruby-qt [1]
> it's not in packaging guideline, i think it's better to follow the upstream
> name scheme

For Ruby packages you should follow Ruby packaging guidelines [1]. Of course
they specify also the mentioned "ruby-" prefix [2] which is clearly the case
for QT binding.

> > Summary: ok just s/QT4/Qt 4
> > License: invalid-license LGPLv2.1+
> > Group: ok
> > URL: ok
> > Sources: ok (md5sum d17198b985f954cedd92ab669623de59)
> > BuildRoot: ok
> > Requires/BuildRequires: not ok according to Ruby packaging guideline (no ABI
> > specified) etc.
> fixed

Here [3] is the guideline.

> > Provides/Obsoletes: not ok, non sense to obsolete/provide same package (as it
> > was in kdebindings subpackage already)
> we need it for upgrade process.
> 
> > Description: ok
> > Devel package: ok (but same prov/obs)
> > Macros usage: ok, consistent
> > ldconfig: ok
> > Other Ruby requirements: ok
> > Docs: ok
> > 
> 
> > qtruby.src: W: invalid-license LGPLv2.1+
> fixed
> 
> > qtruby.src: E: specfile-error sh: ruby: command not found
> > qtruby.src: E: specfile-error sh: ruby: command not found
> > 1 packages and 0 specfiles checked; 2 errors, 1 warnings.
> 
> it's rpmlint bug

You probably don't have ruby installed on your machine.

[1] https://fedoraproject.org/wiki/Packaging:Ruby
[2] https://fedoraproject.org/wiki/Packaging:Ruby#Naming_Guidelines
[3] https://fedoraproject.org/wiki/Packaging:Ruby#Ruby_Packaging_Guidelines

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