Bug 500249 - Pre-Review Request: plexus-digest - Plexus Digest / Hashcode Components
Summary: Pre-Review Request: plexus-digest - Plexus Digest / Hashcode Components
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Andrew Overholt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 429551
TreeView+ depends on / blocked
 
Reported: 2009-05-11 22:02 UTC by Fernando Nasser
Modified: 2009-09-14 09:00 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-21 19:11:58 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Fernando Nasser 2009-05-11 22:02:49 UTC
Package needed for Maven 2.0.8

Spec URL: http://people.fedoraproject.org/~fnasser/plexus-digest.spec

SRPM URL: http://people.fedoraproject.org/~fnasser/plexus-digest-1.0-7.jpp6.src.rpm

Description: 
The Plexus project seeks to create end-to-end developer tools for
writing applications. At the core is the container, which can be
embedded or for a full scale application server. There are many
reusable components for hibernate, form processing, jndi, i18n,
velocity, etc. Plexus also includes an application server which
is like a J2EE application server, without all the baggage.

Comment 1 Andrew Overholt 2009-05-14 14:28:02 UTC
- looking at the source file headers, I think the license should be "ASL 2.0"
- my generated source tarball doesn't have a matching md5sum but a recursive diff of the exploded contents shows no differences
- there are 3 source files with no URLs

Comment 3 Andrew Overholt 2009-05-19 13:14:47 UTC
I don't think the license field should contain " and Plexus", do you?

Comment 4 Fernando Nasser 2009-05-20 20:22:08 UTC
I wrote it based on spot's comment:

https://bugzilla.redhat.com/show_bug.cgi?id=500246#c8

DO you think that was not meant for the whole plexus, but just for plexus-classworlds?

Comment 5 Andrew Overholt 2009-05-21 14:51:45 UTC
(In reply to comment #4)
> I wrote it based on spot's comment:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=500246#c8
> 
> DO you think that was not meant for the whole plexus, but just for
> plexus-classworlds?  

Yes, I think it just applies to plexus-classworlds where that LICENSE.txt file exists.  There's nothing similar in this package to warrant a " and Plexus", is there?

Comment 6 Andrew Overholt 2009-08-21 19:11:58 UTC
See full review request:  bug #518634.

Built:

http://koji.fedoraproject.org/koji/taskinfo?taskID=1623893

Comment 7 Alexander Kurtakov 2009-09-14 09:00:50 UTC
Drop review flag. It was reviewed in #518634.


Note You need to log in before you can comment on or make changes to this bug.