[Bug 227087] Review Request: modello-1.0-0.a8.4jpp - Modello Data Model toolkit

bugzilla at redhat.com bugzilla at redhat.com
Mon Mar 5 15:55:56 UTC 2007

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: modello-1.0-0.a8.4jpp - Modello Data Model toolkit


tbento at redhat.com changed:

           What    |Removed                     |Added
         AssignedTo|tbento at redhat.com           |pcheung at redhat.com

------- Additional Comments From tbento at redhat.com  2007-03-05 10:55 EST -------
X license field matches the actual license.
I can't find the license for this project, could you please point me to it?


X verify source and patches (md5sum matches upstream, know what the patches do)
 - please put clear instructions on how to generate the the source drop, the
   svn export command doesn't work

The svn export command works for me.

X rpmlint on <this package>.srpm gives no output
W: modello non-standard-group Development/Java - this is ok
W: modello unversioned-explicit-provides modello-maven-plugin -please fix
W: modello mixed-use-of-spaces-and-tabs (spaces: line 9, tab: line 36) -please fix

When I fixed the mixed-use-of-spaces-and-tabs warning, the
unversioned-explicit-provides warning disappeared.  Perhaps you could double
check this?
X specfile is legible
 - should have %define _with_gcj_support 1 at the top of the spec file, please
get rid of %define _with_gcj_support 0 and %define gcj_support 0, and the
%define gcj_support .... doesn't seems like it can be split up into multiple

When I do this, I am no longer able to build:
  java-gcj-compat-devel is needed by modello-1.0-0.1.a8.4jpp.1.i38

X make sure lines are <= 80 characters
There are a couple of lines that are > 80, please fix those if possible.

I fixed most of them.  The ones that I didn't, were causing build errors.

Here are the links to the the source rpm and spec file:



Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

More information about the package-review mailing list