Skip to content

fix: Configuration Cache Support #12

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

Conversation

wba2hi
Copy link
Contributor

@wba2hi wba2hi commented Feb 14, 2025

Issue can be reproduced by executing ./gradlew clean build --configuration-cache. Depending on if the bugfix was applied, the build should be failing or succeed.

Instead of using the project.file method to create a file (which automatically appends the absolute path to the project) we use the java.nio.File to create the corresponding file, while manually appending the absolute path to the file. The absolute path to the directory is read from the project and provided to the VssModelGenerator.

First build with failing configuration cache (pre-fix):
https://github.com/eclipse-velocitas/vehicle-app-java-sdk/actions/runs/13324918831/job/37216039585
Second build with successful configuration cache (post-fix):
https://github.com/eclipse-velocitas/vehicle-app-java-sdk/actions/runs/13325161423/job/37216749317

@wba2hi wba2hi force-pushed the configuration-cache-support branch from 5fa1db7 to 15a99fa Compare February 14, 2025 07:57
@wba2hi wba2hi changed the title chore: Enable Configuration Cache in Github Actions fix: Configuration Cache Support Feb 14, 2025
@wba2hi wba2hi marked this pull request as ready for review February 14, 2025 12:07
@wba2hi
Copy link
Contributor Author

wba2hi commented Feb 20, 2025

Closes #10

@wba2hi wba2hi force-pushed the configuration-cache-support branch from 74ea29d to a18464a Compare February 24, 2025 07:06
Copy link

@lukasmittag lukasmittag left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@BjoernAtBosch BjoernAtBosch left a comment

Choose a reason for hiding this comment

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

Due to @lukasmittag's review

@BjoernAtBosch BjoernAtBosch merged commit f71b1ca into eclipse-velocitas:main Mar 4, 2025
4 checks passed
@wba2hi wba2hi deleted the configuration-cache-support branch March 4, 2025 14:25
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.

3 participants