Bug 593348 - Review Request: maven-site-plugin - Maven Site Plugin
Summary: Review Request: maven-site-plugin - Maven Site Plugin
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Jeff Johnston
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-05-18 15:17 UTC by Alexander Kurtakov
Modified: 2010-05-18 21:10 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-05-18 21:10:59 UTC
Type: ---
Embargoed:
jjohnstn: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)

Description Alexander Kurtakov 2010-05-18 15:17:15 UTC
Spec URL: http://akurtakov.fedorapeople.org/maven-site-plugin.spec
SRPM URL: http://akurtakov.fedorapeople.org/maven-site-plugin-2.1-1.fc13.src.rpm
Description: The Maven Site Plugin is a plugin that generates a site for the current project.

Comment 1 Alexander Kurtakov 2010-05-18 15:18:40 UTC
Successful koji build in dist-f14-maven221 tag:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2194265

Comment 2 Jeff Johnston 2010-05-18 18:23:59 UTC
Needs the standard maven requires.  I believe that java and package-utils will get pulled in by the requires of jetty but I don't think that applies to post and postun.

Requires: java
Requires: jpackage-utils
Requires(post): jpackage-utils
Requires(postun): jpackage-utils

bash $ rpmlint maven-site-plugin.spec
maven-site-plugin.spec: W: invalid-url Source0: maven-site-plugin-2.1.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

bash $ rpmlint maven-site-plugin-javadoc-2.1-1.fc14.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

bash $ rpmlint maven-site-plugin-2.1-1.fc14.noarch.rpm 
maven-site-plugin.noarch: W: no-documentation
maven-site-plugin.noarch: W: non-conffile-in-etc /etc/maven/fragments/maven-site-plugin
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

Comment 3 Alexander Kurtakov 2010-05-18 20:15:13 UTC
(In reply to comment #2)
> Needs the standard maven requires.  I believe that java and package-utils will
> get pulled in by the requires of jetty but I don't think that applies to post
> and postun.
> 
> Requires: java
> Requires: jpackage-utils
> Requires(post): jpackage-utils
> Requires(postun): jpackage-utils
> 
Fixed.

> bash $ rpmlint maven-site-plugin.spec
> maven-site-plugin.spec: W: invalid-url Source0: maven-site-plugin-2.1.tar.xz
> 0 packages and 1 specfiles checked; 0 errors, 1 warnings.
> 
> bash $ rpmlint maven-site-plugin-javadoc-2.1-1.fc14.noarch.rpm 
> 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
> 
> bash $ rpmlint maven-site-plugin-2.1-1.fc14.noarch.rpm 
> maven-site-plugin.noarch: W: no-documentation
> maven-site-plugin.noarch: W: non-conffile-in-etc
> /etc/maven/fragments/maven-site-plugin
> 1 packages and 0 specfiles checked; 0 errors, 2 warnings.    

New sources:
Spec URL: http://akurtakov.fedorapeople.org/maven-site-plugin.spec
SRPM URL:
http://akurtakov.fedorapeople.org/maven-site-plugin-2.1-2.fc13.src.rpm

Comment 4 Jeff Johnston 2010-05-18 20:32:28 UTC
Approved.

Comment 5 Alexander Kurtakov 2010-05-18 20:40:02 UTC
New Package CVS Request
=======================
Package Name: maven-site-plugin
Short Description: Maven Site Plugin
Owners: akurtakov
Branches: 
InitialCC:

Comment 6 Dennis Gilmore 2010-05-18 20:45:47 UTC
CVS Done

Comment 7 Alexander Kurtakov 2010-05-18 21:10:59 UTC
Built in koji.
See http://koji.fedoraproject.org/koji/taskinfo?taskID=2195118


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