Open
Bug 1173223
Opened 9 years ago
Updated 3 years ago
Maven warnings when building parser
Categories
(Core :: DOM: HTML Parser, defect, P5)
Core
DOM: HTML Parser
Tracking
()
UNCONFIRMED
People
(Reporter: hogg.bob.2, Unassigned)
Details
Attachments
(1 file)
917 bytes,
patch
|
hsivonen
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Build ID: 20150511103818 Steps to reproduce: 1. Get latest mercurial trunk: "hg clone https://hg.mozilla.org/projects/htmlparser" 2. Run "mvn clean source:jar javadoc:jar repository:bundle-create" Actual results: There were warnings generated by Apache Maven complaining about unspecified plugin versions: [WARNING] [WARNING] Some problems were encountered while building the effective model for nu.validator.htmlparser:htmlparser:bundle:1.4 [WARNING] 'build.plugins.plugin.version' for org.codehaus.mojo:rpm-maven-plugin is missing. @ line 165, column 15 [WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-compiler-plugin is missing. @ line 79, column 15 [WARNING] 'build.plugins.plugin.version' for org.apache.maven.plugins:maven-surefire-plugin is missing. @ line 140, column 15 [WARNING] [WARNING] It is highly recommended to fix these problems because they threaten the stability of your build. [WARNING] [WARNING] For this reason, future Maven versions might no longer support building such malformed projects. [WARNING] The bundle was still built successfully though. Expected results: There should not have been any warnings of this form.
Reporter | ||
Comment 1•9 years ago
|
||
Added a patch to fix the Maven warnings
Attachment #8617704 -
Flags: review+
Reporter | ||
Updated•9 years ago
|
Attachment #8617704 -
Flags: review+ → review?(hsivonen)
Reporter | ||
Updated•9 years ago
|
Severity: normal → trivial
OS: Unspecified → All
Hardware: Unspecified → All
Comment 2•9 years ago
|
||
Comment on attachment 8617704 [details] [diff] [review] Patch to add version numbers to Maven plugins used It's been a long time since I've looked at Maven, so this is basically a rubber-stamp from me.
Attachment #8617704 -
Flags: review?(hsivonen) → review+
Reporter | ||
Comment 3•9 years ago
|
||
(In reply to Henri Sivonen (:hsivonen) from comment #2) > Comment on attachment 8617704 [details] [diff] [review] > Patch to add version numbers to Maven plugins used > > It's been a long time since I've looked at Maven, so this is basically a > rubber-stamp from me. Thank you! Although I now see that I made a mistake: the POM file said I was actually supposed to submit this patch to Validator.nu's Bugzilla. Should I copy this bug over? My apologies!
Comment 4•3 years ago
|
||
Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.
If you have reason to believe this is wrong, please write a comment and ni :jstutte.
Severity: trivial → S4
Priority: -- → P5
You need to log in
before you can comment on or make changes to this bug.
Description
•