Bug 500248 - Pre-Review Request: plexus-cli - Command Line Interface facilitator for Plexus
Summary: Pre-Review Request: plexus-cli - Command Line Interface facilitator for Plexus
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:00 UTC by Fernando Nasser
Modified: 2009-09-14 09:00 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-21 18:50:24 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Fernando Nasser 2009-05-11 22:00:44 UTC
Package needed for Maven 2.0.8

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

SRPM URL: http://people.fedoraproject.org/~fnasser/plexus-cli-1.2-5.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:24:45 UTC
- there are 3 sources without URLs
- I can't reproduce the source tarball as I get:

svn: URL 'http://svn.codehaus.org/plexus/plexus-tools/tags/plexus-cli-1.2' doesn't exist

- the license should be "ASL 2.0"
- I don't see any license in the sources nor on their website; how do we know it's ASL 2.0?

Comment 2 Fernando Nasser 2009-05-18 02:02:29 UTC
Sources moved upstream, fixed URL.

Fixed license.  Assuming the text below applies to the whole plexus project:

http://plexus.codehaus.org/get-involved.html

But the truth is, in this particular subproject that header is not to be found in any file.  Sort of weird.

Anyway, the new spec file is in place and the SRPM is now:

SRPM URL:
http://people.fedoraproject.org/~fnasser/plexus-cli-1.2-6.jpp6.src.rpm


Thanks for catching the SVN URL problem.

Comment 3 Andrew Overholt 2009-05-19 13:19:34 UTC
Spot, can we assume the text regarding licensing being ASL 2.0 at:

http://plexus.codehaus.org/get-involved.html

applies to the whole project?  The files in this package (569 lines in total among 4 files) have no license headers and there is no license text in SVN.

Fernando:  Thanks for fixing the steps to reproduce the source tarball.  I'm not sure of the license field being "ASL 2.0 and Plexus" -- shouldn't it just be "ASL 2.0"?

Comment 4 Tom "spot" Callaway 2009-05-19 13:26:46 UTC
(In reply to comment #3)
> Spot, can we assume the text regarding licensing being ASL 2.0 at:
> 
> http://plexus.codehaus.org/get-involved.html
> 
> applies to the whole project?  The files in this package (569 lines in total
> among 4 files) have no license headers and there is no license text in SVN.

Given how divergent the licensing has been in the various plexus bits, I don't think we can safely make that assumption. We need upstream to clarify things, preferrably by fixing their file attribution.

Comment 5 Jens Petersen 2009-05-29 01:27:12 UTC
So is someone going to ask about it on the plexus mailing-lists?

Comment 6 Jens Petersen 2009-05-29 01:38:33 UTC
I poked around http://jira.codehaus.org a little but was not smart enough to work out where this should be filed.

Comment 7 Jens Petersen 2009-05-29 03:02:27 UTC
Well I filed http://jira.codehaus.org/browse/PLX-418 under unknown component.

Comment 8 Andrew Overholt 2009-08-12 15:19:13 UTC
http://jira.codehaus.org/browse/PLX-418 has been fixed (thanks, Paul Gier!) in trunk.  Spot, are we good to go ahead here?

Comment 9 Tom "spot" Callaway 2009-08-12 15:48:56 UTC
As with 500252, I'd like to see a fixed SRPM with the proper license tag before letting this go forward.

Comment 10 Alexander Kurtakov 2009-08-17 19:28:20 UTC
(In reply to comment #9)
> As with 500252, I'd like to see a fixed SRPM with the proper license tag before
> letting this go forward.  
Spot,
Do we need to put the license headers as a patch or we can just add a comment pointing to the jira issue for clarification?

Comment 11 Tom "spot" Callaway 2009-08-17 19:34:43 UTC
Just make a new SRPM where the source files are properly licensed, please.

Comment 12 Andrew Overholt 2009-08-20 21:11:18 UTC
Full review request (not just a pre-review):  bug #518550.

Comment 13 Andrew Overholt 2009-08-21 18:50:24 UTC
Built:

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

Comment 14 Alexander Kurtakov 2009-09-14 09:00:16 UTC
Drop review flag. It was reviewed in #518550


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