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: classworlds-1.1-0.a2.2jpp - Classworlds Classloader 
Framework


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227047


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[EMAIL PROTECTED]         |[EMAIL PROTECTED]




------- Additional Comments From [EMAIL PROTECTED]  2007-02-16 11:58 EST -------
Updated spec and SRPM:

http://overholt.ca/fedora/classworlds.spec
http://overholt.ca/fedora/classworlds-1.1-1jpp.1.src.rpm

(In reply to comment #2)
> X * if %{?dist} is used, it should be in that form (note the ? and %
> locations)
>     release tag should have a %{?dist}

Fixed.

> X * rpmlint on <this package>.srpm gives no output
>   - justify warnings if you think they shouldn't be there
>   Perhaps change group for javadoc to "Documentation".. ? I will not block on
> this though

Fixed.

> X * package should own all directories and files
>   /usr/share/java is owned by jpackage-utils and it should be a requirement

Fixed.

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

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to