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

Fix Compose Gradle plugin Issue with daemon in Issue 3933 #3951

Merged

Conversation

dima-avdeev-jb
Copy link
Contributor

@dima-avdeev-jb dima-avdeev-jb commented Nov 20, 2023

Fix for Issue #3933

Applyed workaround described in Gradle Issue: gradle/gradle#27099
We just need to call all functions (getters in our case) in our interface based on BuildServiceParameters at least once

@dima-avdeev-jb dima-avdeev-jb changed the base branch from master to release/1.5.11 November 24, 2023 06:51
@dima-avdeev-jb dima-avdeev-jb force-pushed the dima.avdeev/Fix-gradle-daemon-problem-in-issue-3933 branch from aebb563 to 0424b5e Compare November 24, 2023 06:54
@dima-avdeev-jb dima-avdeev-jb marked this pull request as ready for review November 24, 2023 07:00
@dima-avdeev-jb dima-avdeev-jb removed the request for review from terrakok November 24, 2023 07:06
@dima-avdeev-jb dima-avdeev-jb force-pushed the dima.avdeev/Fix-gradle-daemon-problem-in-issue-3933 branch from d386564 to 4ed1b24 Compare November 24, 2023 09:18
@dima-avdeev-jb dima-avdeev-jb changed the base branch from release/1.5.11 to master November 24, 2023 09:18
@dima-avdeev-jb
Copy link
Contributor Author

dima-avdeev-jb commented Nov 24, 2023

First we will merge it to master and after that, cherry pick to release/1.5.11

@dima-avdeev-jb
Copy link
Contributor Author

We decided not cherry pick to 1.5.11

@dima-avdeev-jb dima-avdeev-jb merged commit f4272bf into master Nov 24, 2023
@dima-avdeev-jb dima-avdeev-jb deleted the dima.avdeev/Fix-gradle-daemon-problem-in-issue-3933 branch November 24, 2023 09:58
@dmitryskorbovenko
Copy link

@dima-avdeev-jb So we should wait for 1.6.0 to make our build work? could you please share details regarding not cherry picking to 1.5.11. Thanks

@dima-avdeev-jb
Copy link
Contributor Author

dima-avdeev-jb commented Nov 28, 2023

@dmitryskorbovenko Yes, you should wait 1.6.0 or use dev builds from here: https://github.com/JetBrains/compose-multiplatform/releases

We decided to not cherry pick because of unsufficient amout of time to test new build artifacts. We was need to publish 1.5.11 with another critical bug fixes.
Sorry for that...

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.

None yet

3 participants