Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 355046

Summary: tycho-versions-plugin should ignore leading/trailing whitespaces in pom
Product: z_Archived Reporter: Matthias Gradl <matthias.gradl>
Component: TychoAssignee: Tobias Oberlies <t-oberlies>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Proposal to ignore whitespace that surrounds values in GAV t-oberlies: iplog+

Description Matthias Gradl CLA 2011-08-18 05:33:12 EDT
Build Identifier: 

When updating the specified version in a multimodule Tycho project using tycho-versions-plugin, it is checked whether the module should be updated. The update is only applied if the module is linked to the reactor using the parent specification. 

Now, if there is whitespace surrounding any of the GAV values specifying the parent no match will be found and the version change is not applied.

As whitespaces are illegal characters in Maven groupId or artifactId anyway, but quite in common use to format an XML file, we should only used the trimmed values from the pom, like standard Maven also does.

Reproducible: Always
Comment 1 Matthias Gradl CLA 2011-08-18 09:41:22 EDT
Created attachment 201718 [details]
Proposal to ignore whitespace that surrounds values in GAV
Comment 2 Tobias Oberlies CLA 2011-08-18 10:50:07 EDT
Fixed with d42e4b1. Thank you for the patch!