Bug 227096 - Review Request: plexus-archiver - Plexus Archiver Component
Review Request: plexus-archiver - Plexus Archiver Component
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Permaine Cheung
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-02-02 12:50 EST by Rafael H. Schloming
Modified: 2014-12-01 18:14 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-12 17:52:09 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tbento: fedora‑review+
petersen: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Rafael H. Schloming 2007-02-02 12:50:08 EST
Spec URL: http://people.redhat.com/rafaels/specs/plexus-archiver-1.0-0.a6.1jpp.spec
SRPM URL: ftp://jpackage.hmdc.harvard.edu/JPackage/1.7/generic/SRPMS.free/plexus-archiver-1.0-0.a6.1jpp.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.

Javadoc for plexus-archiver.
Comment 2 Tania Bento 2007-02-16 12:36:32 EST
Just a couple of things:

- Remove "%define secion free".
- Source0 should be a URL.
- The license can be found in a few html pages in the following directory:
target/docs/apidocs/org/codehaus/plexus/archiver
- Remove the vendor tag.
- Remove the distribution tag.
- Some lines are more than 80 characters.
- Should gcj support be added?
Comment 3 Matt Wringe 2007-02-16 15:55:27 EST
(In reply to comment #2)
> Just a couple of things:
> 
> - Remove "%define secion free".
Removed

> - Source0 should be a URL.
Source0 is taken from svn, so there is no download url for it. The svn
instructions are included as comments in the spec

> - The license can be found in a few html pages in the following directory:
> target/docs/apidocs/org/codehaus/plexus/archiver
These are the javadocs and are included in the javadoc subpackage. This project
does not include license specific files.

> - Remove the vendor tag.
Done

> - Remove the distribution tag.
Done

> - Some lines are more than 80 characters.
Fixed

> - Should gcj support be added?
Gcj support added

srpm and spec upload to same location as before 

Comment 4 Tania Bento 2007-02-16 17:21:44 EST
Everything looks good to me.  Approved.
Comment 5 Tania Bento 2007-02-19 09:56:26 EST
Actually, there's one thing that I didn't catch.  Under the "svn" command, you
need to include the "tar" command as well. 
Comment 6 Matt Wringe 2007-03-06 11:12:26 EST
(In reply to comment #5)
> Actually, there's one thing that I didn't catch.  Under the "svn" command, you
> need to include the "tar" command as well. 

done, files changed and uploaded to same location:
https://mwringe.108.redhat.com/files/documents/175/224/plexus-archiver-1.0-0.1.a6.1jpp.1.src.rpm
https://mwringe.108.redhat.com/files/documents/175/225/plexus-archiver.spec
Comment 7 Permaine Cheung 2007-03-06 11:31:12 EST
New Package CVS Request
=======================
Package Name: plexus-archiver
Short Description: Plexus Archiver Component
Owners: pcheung@redhat.com
Branches: 
InitialCC: 
Comment 8 Permaine Cheung 2007-03-12 17:52:09 EDT
Package built into plague. Closing as NEXTRELEASE.

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