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

[MNG-7059][MNG-4645] Move Central repo definition out of Maven's core so it can be more easily changed #1139

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

gnodet
Copy link
Contributor

@gnodet gnodet commented Jun 1, 2023

@rmannibucau
Copy link
Contributor

Do we intend to break builds overriding settings? Guess injection logic should be kept until we force an implicit settings.xml or do not define any implicit packaging/lifecycle, no?

@gnodet
Copy link
Contributor Author

gnodet commented Jun 1, 2023

Do we intend to break builds overriding settings? Guess injection logic should be kept until we force an implicit settings.xml or do not define any implicit packaging/lifecycle, no?

That's a really good question. I've added back the injection of maven central in case the settings read have no repositories defined. The most usual use case will be if the global settings have been overridden by the command line.

@gnodet gnodet force-pushed the MNG-4645-extract-central-definition branch from 4f5a7ae to 6396f13 Compare June 1, 2023 21:44
@gnodet
Copy link
Contributor Author

gnodet commented Jun 2, 2023

Kinda weird that the only IT failing is because it's now using the newly accepted syntax, but we're testing a failure: https://github.com/apache/maven-integration-testing/blob/941353757073754cedd91f49c3d1d9c216adda0d/core-it-suite/src/test/resources/mng-3748/settings.xml ...

@cstamas
Copy link
Member

cstamas commented Jun 15, 2023

Does this means maven4 settings.xml will not be parsable by Maven3? So projects will have to "take a stance", and be "maven4 only" or 'maven3 only"?

@gnodet
Copy link
Contributor Author

gnodet commented Jun 15, 2023

Does this means maven4 settings.xml will not be parsable by Maven3? So projects will have to "take a stance", and be "maven4 only" or 'maven3 only"?

That's the settings.xml which will come with the maven 4 distribution and located in [MVN_HOME]/conf/settings.xml, so I don't think this is an issue. If people want to modify their own user settings and leverage new features, then yes, but as a workaround, it's always possible to use a profile activated by default instead.

@gnodet gnodet force-pushed the MNG-4645-extract-central-definition branch from d13c497 to 2956e9d Compare June 15, 2023 18:12
@gnodet gnodet changed the title [MNG-4645] Move Central repo definition out of Maven's core so it can be more easily changed [MNG-7059][MNG-4645] Move Central repo definition out of Maven's core so it can be more easily changed Jun 19, 2023
@gnodet gnodet force-pushed the MNG-4645-extract-central-definition branch from 2956e9d to 84e546c Compare June 19, 2023 12:15
@gnodet gnodet merged commit 229b31b into apache:master Jun 19, 2023
18 checks passed
@gnodet gnodet deleted the MNG-4645-extract-central-definition branch June 20, 2023 06:31
This pull request was closed.
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