RFR: 8249777: build.gradle: project.version should not contain time stamps [v2]
Kevin Rushforth
kcr at openjdk.java.net
Thu Jul 23 13:45:33 UTC 2020
> The addMavenPublication method sets the `project.version` to `$MAVEN_VERSION`, where project is base, graphics,
> controls, etc. When doing an ordinary "developer" build, this property contains the time stamp . This is causing
> problems with the NetBeans gradle plugin not being able to determine the dependencies between the projects (modules).
> We haven't noticed this bug before now, because we suppress the creation of the individual project jar files, but if we
> were creating them, we would have the problem that each time we ran gradle, it would generate a new
> `project-$version.jar` file with a different time stamp each time the build was run. The setting of
> `project.version=$MAVEN_VERSION` is only used by the maven "publish" tasks, which isn't done for developer builds nor
> for most productions builds. The proposed solution is to add a new boolean `MAVEN_PUBLISH` flag, which will default to
> `false`. This flag will qualify the setting of the `MAVEN_VERSION` property and the configuration of the maven
> publishing tasks. After this change, it will be necessary to run gradle with `-PMAVEN_PUBLISH=true` in order to run the
> maven publishing tasks.
Kevin Rushforth has updated the pull request incrementally with one additional commit since the last revision:
Remove unised mavenPublish property from graphics project
-------------
Changes:
- all: https://git.openjdk.java.net/jfx/pull/270/files
- new: https://git.openjdk.java.net/jfx/pull/270/files/592ad650..1c077ec5
Webrevs:
- full: https://webrevs.openjdk.java.net/jfx/270/webrev.01
- incr: https://webrevs.openjdk.java.net/jfx/270/webrev.00-01
Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
Patch: https://git.openjdk.java.net/jfx/pull/270.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/270/head:pull/270
PR: https://git.openjdk.java.net/jfx/pull/270
More information about the openjfx-dev
mailing list