[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