Skip to content

experiment with new group id#219

Closed
cdietrich wants to merge 12 commits into
masterfrom
cd-gradle-exp-xtend
Closed

experiment with new group id#219
cdietrich wants to merge 12 commits into
masterfrom
cd-gradle-exp-xtend

Conversation

@cdietrich

Copy link
Copy Markdown
Member

No description provided.

@cdietrich cdietrich marked this pull request as draft June 24, 2025 19:18

val ComparableVersion currentXtextVersion = new ComparableVersion(version);
val ComparableVersion targetVersion = new ComparableVersion("2.40.0")
if (currentXtextVersion >= targetVersion) {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@oehme while we are parallel investigating with relocations on maven side, wdyt about this approach

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmmm no idea why xtextVersion.getVersion crashes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The xtextVersion variable is determined from the dependencies, so you can't determine the dependencies from the Xtext version. Chicken-egg problem :)

//if (currentXtextVersion >= targetVersion) {
val m1 = module('''org.eclipse.xtend:org.eclipse.xtend.core:>=2.40.0'''.toString());
val m2 = module('''org.eclipse.xtext:org.eclipse.xtend.core'''.toString());
//substitute(m1).using(m2)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

problem seems to be we need a target version for the substitute here too

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ofc wont work cause it runs too early

@cdietrich

Copy link
Copy Markdown
Member Author
  • What went wrong:
    Could not determine the dependencies of task ':generateXtext'.

Could not resolve all task dependencies for configuration ':xtextToolingMain'.
Could not find org.eclipse.xtext:org.eclipse.xtext:.
Required by:
project :
Could not find org.eclipse.xtext:org.eclipse.xtext.smap:.
Required by:
project :
Could not find org.eclipse.xtext:org.eclipse.xtext.xbase:.
Required by:
project :
Could not find org.eclipse.xtext:org.eclipse.xtext.java:.
Required by:
project :
Could not find org.eclipse.xtext:xtext-dev-bom:.
Required by:
project :
Could not find org.eclipse.xtend:org.eclipse.xtend.core:.
Required by:
project

@cdietrich

Copy link
Copy Markdown
Member Author

have no clue why it harms the test even if it is not going into my replacer

Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
java-version: ${{ matrix.java }}
- name: Build
uses: eskatos/gradle-command-action@v1
uses: gradle/gradle-build-action@v2

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i assume the other wf needs change too

with:
arguments: build
dependencies-cache-enabled: true
cache-read-only: ${{ github.ref != 'refs/heads/master' }}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no idea which config we want here

@cdietrich cdietrich closed this Aug 20, 2025
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.

2 participants