[Bug 674060] Review Request: rubygem-deltacloud-core - Deltacloud REST API
bugzilla at redhat.com
bugzilla at redhat.com
Fri Feb 4 11:02:34 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=674060
--- Comment #4 from Michal Fojtik <mfojtik at redhat.com> 2011-02-04 06:02:33 EST ---
(In reply to comment #3)
> * Check
> - pushd ./%{geminstdir} is not stepping into the correct directory.
> The right one is: pushd %{buildroot}%{geminstdir}
AFAIK it's stepping to buildroot, but I replaced ./ with that macro just for
sure. Thanks for noticing that.
> * Shipping external project
> - This gem bundles external project in support folder. On Fedora shipping
> external project in the same package should (must) be avoided, see:
Ouch, sorry I built that gem in support directory so there was some leftovers.
It should be fixed now.
> https://fedoraproject.org/wiki/Packaging/Guidelines#Bundling_of_multiple_projects
>
> * Missing dependencies
> - Since this is Sinatra application, there have to be specified
> appropriate dependencies:
>
> Require: rubygem(sinatra)
> BuildRequire: rubygem(sinatra)
Require: rubygem(sinatra) was already here but it was missing in BuildRequire.
Should be fixed now.
> - Missing rubygem-net-ssh dependency. Not sure if it is runtime
> or just development dependency
That's right, thanks for corrections.
>
> - There are missing plenty of others, such as rack. Please test using mock.
Should be fixed now. Note that other dependencies like 'rubygem-aws' are not
'runtime' dependencies but they are optional. Eg. for using EC2 or other cloud
providers.
> * rpmlint output:
> - Is this package ahead of upstream?
>
> $ rpmlint rubygem-deltacloud-core.spec
> rubygem-deltacloud-core.spec: W: invalid-url Source0:
> http://gems.rubyforge.org/gems/deltacloud-core-0.2.0.gem
> HTTP Error 404: Not Found
> 0 packages and 1 specfiles checked; 0 errors, 1 warnings.
Yes, I'll need to wait with importing until this package will be available in
upstream (which should happen in next few days).
>
> I am not going forward with this review, since I cannot build on my environment
> due to missing dependencies.
======================= v0.2.0-2 ======================
Spec URL: http://mifo.sk/RPMS/deltacloud-core.spec
SRPM URL: http://mifo.sk/RPMS/deltacloud-core-0.2.0-2.fc14.src.rpm
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2761635
--
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