Bug 589585 - Review Request: maven-javadoc-plugin - Maven Javadoc plugin
Review Request: maven-javadoc-plugin - Maven Javadoc plugin
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeff Johnston
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-06 09:40 EDT by Mary Ellen Foster
Modified: 2010-05-11 11:32 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-05-11 11:32:00 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
jjohnstn: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Mary Ellen Foster 2010-05-06 09:40:38 EDT
Spec URL: http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin.spec
SRPM URL: http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin-2.4-1.fc12.src.rpm

Description:
The Maven Javadoc Plugin is a plugin that uses the javadoc tool for
generating javadocs for the specified project.

Notes about this package:
- It uses version 2.4, which is not the latest, so that it can be built with the current maven2-2.0.8; it will be updated to the newest version once the newer maven is built
- Javadoc generation for the plugin is disabled for the initial version because that currently also won't work until the rest of maven is updated -- this is the role of the "bootstrap" flag
Comment 1 Mary Ellen Foster 2010-05-06 09:41:38 EDT
Additional note: this builds successfully in koji against the dist-f14-maven221 target:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2167910
Comment 2 Jeff Johnston 2010-05-06 15:36:17 EDT
Please provide URL of where the tarball was taken from or else information about how the tarball was created.  If it was a snapshot from a repository, please indicate that with either the date or tag used.

Results of rpmlint on build above.  Please fix where appropriate and/or comment as ignored for the messages below.

bash $ rpmlint -i maven-javadoc-plugin-2.4-1.1.fc14.noarch.rpm 
maven-javadoc-plugin.noarch: W: spelling-error %description -l en_US javadocs -> java docs, java-docs, Javanese
The value of this tag appears to be misspelled. Please double-check.

maven-javadoc-plugin.noarch: E: no-changelogname-tag
There is no %changelog tag in your spec file. To insert it, just insert a
'%changelog' in your spec file and rebuild it.

maven-javadoc-plugin.noarch: W: obsolete-not-provided maven2-plugin-javadoc
If a package is obsoleted by a compatible replacement, the obsoleted package
should also be provided in order to not cause unnecessary dependency breakage.
If the obsoleting package is not a compatible replacement for the old one,
leave out the Provides.

maven-javadoc-plugin.noarch: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

maven-javadoc-plugin.noarch: W: non-conffile-in-etc /etc/maven/fragments/maven-javadoc-plugin
A non-executable file in your package is being installed in /etc, but is not a
configuration file. All non-executable files in /etc should be configuration
files. Mark the file as %config in the spec file.

1 packages and 0 specfiles checked; 1 errors, 4 warnings.
Comment 3 Peter Lemenkov 2010-05-07 05:43:13 EDT
*-javadoc package must require jpackage-utils (owner of %{_javadocdir}), which is a common packaging issue among java-related packages.
Comment 4 Mary Ellen Foster 2010-05-07 05:57:40 EDT
I've fixed the issues identified in the above comments (the missing changelog was an artifact of the build, which wasn't quite the final version).

- Added jpackage requirements
- Updated requirements of the (disabled) jpackage sub-package
- Added a comment showing where the tarball comes from (it's a subversion checkout, so no URL is possible)
- Added a Provides: maven2-plugin-javadoc
- The warning about /etc/maven/fragments/maven-javadoc-plugin not being a conf-file is spurious


http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin.spec
http://www.macs.hw.ac.uk/~mef3/maven221/maven-javadoc-plugin-2.4-2.fc12.src.rpm
Comment 5 Mary Ellen Foster 2010-05-07 06:14:31 EDT
p.s. -- Here's a new koji scratch build:
    http://koji.fedoraproject.org/koji/taskinfo?taskID=2171528
Comment 6 Jeff Johnston 2010-05-07 12:19:33 EDT
Looks good.  I am marking this as reviewed.
Comment 7 Mary Ellen Foster 2010-05-07 12:33:55 EDT
New Package CVS Request
=======================
Package Name: maven-javadoc-plugin
Short Description: Maven Javadoc plugin
Owners: mef
Branches: 
InitialCC:
Comment 8 Kevin Fenzi 2010-05-08 22:10:41 EDT
CVS done (by process-cvs-requests.py).

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