Bug 727780

Name: ? shouldn't we name it according ruby package naming scheme ruby-qt [1]
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.
Provides/Obsoletes: not ok, non sense to obsolete/provide same package (as it was in kdebindings subpackage already)
Description: ok
Devel package: ok (but same prov/obs)
Macros usage: ok, consistent
ldconfig: ok
Other Ruby requirements: ok
Docs: ok

rpmlint qtruby-4.7.0-1.src.rpm 
qtruby.src: W: invalid-license LGPLv2.1+
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.

[1] http://fedoraproject.org/wiki/Packaging:Ruby

(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

> 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

> 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

(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

BTW the package does not build in Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3251955

Good spot Vit, thanks,
Than, please add %{?_kde4_macros_api:Requires: kde4-macros(api) = %{_kde4_macros_api} } to the spec file so cmake_kde4 is brought in.

ruby-qt-4.7.0-1.src.rpm , ruby-qt.spec ware uploaded. all issues should be fixed in this new package.

thanks

Does not build "%setup -q -n qt-ruby" is missing - http://koji.fedoraproject.org/koji/taskinfo?taskID=3252077

new packages were uploaded and include the fix for above issue

* The release should include %{dist} macro
  Release: 1%{?dist}

(In reply to comment #9)
> * The release should include %{dist} macro
>   Release: 1%{?dist}

it's now fixed in new package
http://koji.fedoraproject.org/koji/taskinfo?taskID=3252297

thanks

Than

The package looks good. However

* Is it possible to execute the test suite in %check section?
* The examples might be worth of including into -doc subpackage.
* There is "rails_support" available in sources. It might be good idea
  to include it in some -rails subpackage.

(In reply to comment #11)
> The package looks good. However
> 
> * Is it possible to execute the test suite in %check section?
it doesn't work yet, but we can enable it in the future

> * The examples might be worth of including into -doc subpackage.
yes, it's included in ruby-qt-examples

> * There is "rails_support" available in sources. It might be good idea
>   to include it in some -rails subpackage.

it doesn't make sense to split it to subpackage.

Than

Ok,
thanks Than and Vit! I agree with Than regarding rails_support but of course if someone asks, it can be packaged separately.

rpmlint ruby-qt-4.7.0-1.fc15.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

package builds correctly http://koji.fedoraproject.org/koji/taskinfo?taskID=3279572

APPROVED

Thanks!

New Package SCM Request
=======================
Package Name: ruby-qt
Short Description: Ruby bindings for Qt 4
Owners: than rdieter jreznik kkofler ltinkl rnovacek
Branches: f15

Please make the name in the summary and Package Request match.  Also, don't
forget the f16 branch.

New Package SCM Request
=======================
Package Name: ruby-qt
Short Description: Ruby bindings for Qt 4
Owners: than rdieter jreznik kkofler ltinkl rnovacek
Branches: f15 and f16

Complete, but there were hiccups, let me know if anything isn't right.  It
should be fine.

ruby-qt was commited and built. thanks