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

Set a property based on the maven.build.timestamp #61

Closed
JeneJasper opened this issue Dec 26, 2017 · 3 comments
Closed

Set a property based on the maven.build.timestamp #61

JeneJasper opened this issue Dec 26, 2017 · 3 comments
Assignees
Milestone

Comments

@JeneJasper
Copy link
Contributor

The maven.build.timestamp property does not allow for timezone overrides for obvious reasons.

The timestamp-property goal can be used to create one with a local timezone timestamp. But the default implementation uses the current time, which is thus not the same for all modules in a multi module project.

The new timeSource parameter with value build now allows the usage of maven.build.timestamp property instead of the current timestamp.

@stefanseifert
Copy link
Collaborator

fixed by #32

@garretwilson
Copy link

Can someone explain the behavior of these two options, current and build? I'm not understanding what they do. I can guess what "current" means—probably something like Instant.now(). But where would the "build" time come from? If we're in the middle of the build, how could it be any different than "current"?

@garretwilson
Copy link

Oh, looking at the source code, the "build" source simply returns the time the entire build started, thus allowing all modules to have a consistent time. Is that a correct interpretation?

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

No branches or pull requests

3 participants