Skip to content

Conversation

asodja
Copy link
Member

@asodja asodja commented Oct 9, 2025

Bug

So if you use fetch API to query models per project, i.e. by first querying GradleBuild and then querying some model for a project, you get a failure like:

Don't know how to build models for GradleProject{path=':'}

Fix:

We were missing Object originalTarget = unpackModelTarget(target); in FetchAwareBuildControllerAdapter.java

This is part of #35212

@asodja asodja self-assigned this Oct 9, 2025
@asodja asodja added the a:bug label Oct 9, 2025
@asodja asodja added this to the 9.3.0 RC1 milestone Oct 9, 2025
@asodja

This comment has been minimized.

@bot-gradle

This comment has been minimized.

@bot-gradle

This comment has been minimized.

@asodja asodja force-pushed the asodja/fix-fetch-api branch from 45b295a to d054d65 Compare October 10, 2025 09:53
@asodja

This comment has been minimized.

@bot-gradle

This comment has been minimized.

@asodja

This comment has been minimized.

@bot-gradle

This comment has been minimized.

@bot-gradle
Copy link
Collaborator

The following builds have been cancelled:

@bot-gradle
Copy link
Collaborator

The following builds have passed:

@asodja asodja marked this pull request as ready for review October 10, 2025 11:47
@asodja asodja requested a review from a team as a code owner October 10, 2025 11:47
@asodja asodja requested review from a team and eskatos October 10, 2025 11:47
@asodja asodja marked this pull request as draft October 10, 2025 11:47
@asodja asodja force-pushed the asodja/fix-fetch-api branch from d054d65 to 10ba326 Compare October 10, 2025 12:18
@asodja
Copy link
Member Author

asodja commented Oct 10, 2025

@bot-gradle test this

@asodja asodja marked this pull request as ready for review October 10, 2025 12:18
@bot-gradle

This comment has been minimized.

@bot-gradle
Copy link
Collaborator

The following builds have passed:

Copy link
Member

@reinsch82 reinsch82 left a comment

Choose a reason for hiding this comment

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

LGTM

@asodja asodja added this pull request to the merge queue Oct 10, 2025
Merged via the queue into master with commit 0f4fcb5 Oct 10, 2025
33 checks passed
@asodja asodja deleted the asodja/fix-fetch-api branch October 10, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants