Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Move to Maven/Tycho for projects and build infrastructure #501

Closed
wants to merge 84 commits into from

Conversation

LorenzoBettini
Copy link
Contributor

Replaces #500

@cdietrich
Copy link
Member

yes. the same as it would be a monorepo. the only place we use other version is xtend-maven-plugin.
and it should be done with profile so that it can be decided if from sonatype or from upstream branches

@LorenzoBettini
Copy link
Contributor Author

yes. the same as it would be a monorepo. the only place we use other version is xtend-maven-plugin. and it should be done with profile so that it can be decided if from sonatype or from upstream branches

I'm doing some experiments in xtext-core concerning the versioning across Git repositories.

So I introduced xtext-dev-bom-version that defaults to project.version and can be redefined in projects like xtext.logging that has a different version eclipse/xtext-core@e329fc2

Besides that, I always use project.version for referring to xtext artifacts coming from other git repositories: eclipse/xtext-core@31358b5

During building, we use the profile useJenkinsSnapshots so the snapshots are found. But what about without that profile? For example, in Eclipse? Without snapshot repository specification, snapshots are not found. A profile for sonatype snapshots will not work because activeByDefault, as I said in the past, is not really active by default.

Thus, the sonatype snapshot repository is the default (specified outside profiles), and the profile useJenkinsSnapshots explicitly disables that sonatype snapshots repository: eclipse/xtext-core@53f0781

@cdietrich
Copy link
Member

with the repo merge done this is obsolete

@cdietrich cdietrich closed this Apr 3, 2023
@cdietrich cdietrich deleted the lb_2052_maven_tycho branch April 18, 2023 14:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants