[Bug 651898] Review Request: rubygem-activemodel - A toolkit for building modeling frameworks

bugzilla at redhat.com bugzilla at redhat.com
Wed Jan 26 08:52:44 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=651898

--- Comment #4 from Vít Ondruch <vondruch at redhat.com> 2011-01-26 03:52:43 EST ---
I am taking over this package (right Jozef?).

So here is updated package:

Spec URL: http://people.redhat.com/vondruch/rubygem-activemodel.spec
SRPM URL:
http://people.redhat.com/vondruch/rubygem-activemodel-3.0.3-1.fc14.src.rpm


(In reply to comment #3)
> rubygem-i18n has now made it into fedora and I've update rubygem-activesupport
> to version 3.0.3 which will be pushed into fedora as soon as it's dependencies
> are.
> 
> For now use the following for this review
> 
> http://mo.morsi.org/files/rpms/rubygem-activesupport-3.0.3-1.fc14.src.rpm
> 
> * please update to latest activemodel upstream release, 3.0.3

DONE

> * rpmlint looks good
> 
> * mark README, LICENSE, CHANGELOG files as %doc

DONE

> * you do not need rm -rf %{buildroot} in %install and %clean sections (should
> be removed)

Removed from install section. Left in %clean section. If you remove it, then
the activemode directory stays in BUILDROOT, which is not nice, although it
doesn't make any problems.

I should remember this and be consistent ...


> * feel free to tar up upstream test suite and run in in a %check section

DONE

> * feel free to move docs into their own subpackage

DONE

I do not provide the Koji build due to missing activesupport dependency.
However if approved, I can already prepare the repository.

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