Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JBIDE-17281] Use build/pluginManagement instead of build/plugins #143

Closed
wants to merge 1 commit into from

Conversation

dgolovin
Copy link
Member

No description provided.

<build>
<plugins>
<plugin>
<groupId>org.eclipse.tycho</groupId>
<artifactId>tycho-packaging-plugin</artifactId>
<version>${tychoVersion}</version>
<configuration>
<format>'${BUILD_ALIAS}-v'yyyyMMdd-HHmm</format>
Copy link
Member

Choose a reason for hiding this comment

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

File is now 30 lines longer and contains multiple tycho-packaging-plugin entries. Why is this better?

@mickaelistria
Copy link
Contributor

I share the same concerns as @nickboldt . IMO, that change doesn't make things easier to maintain and does not fix any issue.
Which use-case drove you to write this change?

@maxandersen
Copy link
Member

Now I grok this.

This will mean we can remove stuff like https://github.com/jbosstools/jbosstools-server/blob/master/as/tests/org.jboss.tools.as.test.core/pom.xml#L25 since the version used for the plugins will now be defined in the parent pom.

This is actually a recommended best practice for maven.

@nickboldt
Copy link
Member

@dgolovin
Copy link
Member Author

closed as out of date.
I'll create new one for parent/pom.xml and one per component to clean up not needed ${tychoVersion} usage.

@dgolovin dgolovin closed this Nov 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants