Skip to content

MPOM-255 enforce setting property "project.build.outputTimestamp"#30

Closed
kwin wants to merge 1 commit intoapache:masterfrom
kwin:feature/enforce-setting-timestamp-property
Closed

MPOM-255 enforce setting property "project.build.outputTimestamp"#30
kwin wants to merge 1 commit intoapache:masterfrom
kwin:feature/enforce-setting-timestamp-property

Conversation

@kwin
Copy link
Member

@kwin kwin commented Jan 3, 2021

No description provided.

@kwin kwin force-pushed the feature/enforce-setting-timestamp-property branch from 0d5e3fa to 45d63d2 Compare January 3, 2021 12:56
@michael-o michael-o self-requested a review January 3, 2021 13:03
@michael-o
Copy link
Member

Althouth this looks technically correct, I do think that this will not work. You have moved the property to a comment and added an enforcer rule, but with the next release the property will be added again and inherited.

@kwin
Copy link
Member Author

kwin commented Jan 4, 2021

IMHO maven-release-plugin will only update an existing property but never add it initially.

@asfgit asfgit closed this in 908aae1 Jan 4, 2021
@kwin
Copy link
Member Author

kwin commented Jan 4, 2021

Seems we need to disable the enforcer check for the build of asf parent itself

@michael-o
Copy link
Member

What would be your proposal then?

@kwin
Copy link
Member Author

kwin commented Jan 4, 2021

Activate the rule only in a profile which is activated only in case a marker file with name asf-parent.marker is missing. That would require adding such a file (could be empty) to the maven-apache-parent repository.
The only other option is to deal with explicitly given profiles or system properties in case asf-parent is built.

I tried it out in kwin@aff2ed0 and it seems to work. Should I raise a follow-up PR?

@michael-o
Copy link
Member

I think we can make the file even hidden because this is a technical detail. Please raise a followup PR for this.

@kwin
Copy link
Member Author

kwin commented Jan 5, 2021

I raised a follow-up PR in #33. @michael-o Please have a look.

@jira-importer
Copy link

Resolve #481

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.

3 participants