Bug 593348

Summary: Review Request: maven-site-plugin - Maven Site Plugin
Product: [Fedora] Fedora Reporter: Alexander Kurtakov <akurtako>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED RAWHIDE QA Contact: Jeff Johnston <jjohnstn>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, jjohnstn, notting
Target Milestone: ---Flags: jjohnstn: fedora-review+
dennis: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-05-18 21:10:59 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

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