Skip to content

Relative path configurable#162

Merged
asfgit merged 3 commits intoapache:masterfrom
ahgittin:relative-path-configurable
Dec 20, 2019
Merged

Relative path configurable#162
asfgit merged 3 commits intoapache:masterfrom
ahgittin:relative-path-configurable

Conversation

@ahgittin
Copy link
Copy Markdown
Contributor

extends #160 offering a way for downstream projects to customize relative path if it causes issues

geomacy and others added 3 commits December 19, 2019 23:24
surprisingly (to me) this _is_ permitted in maven; according to the docs it only restricts:

> Properties are not supported in the following XPath elements:
> `/project(/parent)?/(groupId/artifactId/version)`
@ahgittin
Copy link
Copy Markdown
Contributor Author

thanks @tbouron for the suggestion in #160 , implemented here

<brooklyn.version>1.0.0-SNAPSHOT</brooklyn.version><!-- BROOKLYN_VERSION -->
<build.version>${revision}</build.version>
<build.name>Apache Brooklyn</build.name>
<brooklyn.ui.relativePath.to.brooklyn.server.parent>../brooklyn-server/parent/</brooklyn.ui.relativePath.to.brooklyn.server.parent>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The name is a bit too much for my taste @ahgittin, I would use a simpler parent.relative.path.
But happy to go ahead and merge if it works

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

assuming this is set higher up in a complex project hierarchy it needs to be namespaced appropriately. someone else might have a different parent.relative.path. agree it's a little bit silly long but at least it's totally unambiguous. :)

@asfgit asfgit merged commit e8dec48 into apache:master Dec 20, 2019
asfgit pushed a commit that referenced this pull request Dec 20, 2019
@ahgittin
Copy link
Copy Markdown
Contributor Author

Seems maven doesn't actually respect a variable in relative path. I must have been taking some cached or downloaded item when i tested, as when we change the version it breaks.

Also irritating, seems maven won't look in the local reactor model so we need either relative path to work or the upstream item to have been built.

Revert this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants