Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: maven-javadoc-plugin - Maven Javadoc plugin
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Summary: Review Request: maven-javadoc-plugin - Maven Javadoc plugin Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mefoster@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
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....
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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #1 from Mary Ellen Foster mefoster@gmail.com 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Jeff Johnston jjohnstn@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jjohnstn@redhat.com AssignedTo|nobody@fedoraproject.org |jjohnstn@redhat.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #2 from Jeff Johnston jjohnstn@redhat.com 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Alexander Kurtakov akurtako@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |akurtako@redhat.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #3 from Peter Lemenkov lemenkov@gmail.com 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #4 from Mary Ellen Foster mefoster@gmail.com 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....
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #5 from Mary Ellen Foster mefoster@gmail.com 2010-05-07 06:14:31 EDT --- p.s. -- Here's a new koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2171528
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Jeff Johnston jjohnstn@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
--- Comment #6 from Jeff Johnston jjohnstn@redhat.com 2010-05-07 12:19:33 EDT --- Looks good. I am marking this as reviewed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Mary Ellen Foster mefoster@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #7 from Mary Ellen Foster mefoster@gmail.com 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:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
--- Comment #8 from Kevin Fenzi kevin@tummy.com 2010-05-08 22:10:41 EDT --- CVS done (by process-cvs-requests.py).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=589585
Mary Ellen Foster mefoster@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
package-review@lists.fedoraproject.org