Bug 998322
| Summary: | javapackages-tools: add_pom_dep_mgmt injects incorrect xml | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Matt Spaulding <mspaulding06> | ||||
| Component: | javapackages-tools | Assignee: | Mikolaj Izdebski <mizdebsk> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | unspecified | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 20 | CC: | java-sig-commits, mizdebsk, msrb, sochotni | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | 3.3.1-1 | Doc Type: | Bug Fix | ||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2013-10-10 14:52:02 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 992306 | ||||||
| Attachments: |
|
||||||
Thanks for the patch, this was indeed a bug. Commited upstream. It might take a while before new realease, but you can workaround this bug by using "pom_xpath_inject" if really needed. (un?)fortunately add_pom_dep_mgmnt is needed very rarely so it wasn't very well tested. This bug appears to have been reported against 'rawhide' during the Fedora 20 development cycle. Changing version to '20'. More information and reason for this action is here: https://fedoraproject.org/wiki/BugZappers/HouseKeeping/Fedora20 Fixed in javapackages-tools-3.3.1-1 I believe that this bug is fixed in javapackages-tools-3.3.1-1, which is available in Fedora 20, so I am closing this bug now. The build containing the fix can be found at Koji: http://koji.fedoraproject.org/koji/buildinfo?buildID=469209 |
Created attachment 787875 [details] Patch against upstream repository to fix add_pom_dep_mgmt macro Description of problem: The add_pom_dep_mgmt macro does not inject the correct xml into the pom file, since it does not include "dependencies" in the xpath when doing the injection. Version-Release number of selected component (if applicable): 2.0.1-1 How reproducible: Always Steps to Reproduce: 1. Use the add_pom_dep_mgmt macro in an RPM spec file. Actual results: RPM build will fail since the pom file is incorrect. The "dependency" node is injected right after the "dependencyManagement" node in the pom file. Expected results: The dependency gets injected inside the "dependencies" node which is under the "dependencyManagement" node.