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.

Sign up for GitHub

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

Jump to bottom

Add VssDefinition plugin for KSP usage #45

Closed
Chrylo opened this issue Nov 24, 2023 · 0 comments · Fixed by #71
Closed

Add VssDefinition plugin for KSP usage #45

Chrylo opened this issue Nov 24, 2023 · 0 comments · Fixed by #71
Labels
enhancement New feature or request
Milestone
0.1.4

Comments

@Chrylo
Copy link
Contributor

Chrylo commented Nov 24, 2023

Is your feature request related to a problem? Please describe.
See #44.

"The current loadAssets workaround will result into a ksp error if
only tasks like kspDebug are executed. The reason is that the build
folder does not have the information from the assets folder (from
the tests or apps build command) yet."

This can lead to an unexpected behavior for multiple clean builds like:
./gradlew clean test

Describe the solution you'd like
Since KSP has a limited access to assets and resources provided by the app developer this problem has to be solved on gradle level.

  • Create a plugin which has to be integrated by the user when using our KSP library. The plugin should copy every compatible .yaml file (in the future also .json etc.) to the build directory so that the KSP processor has always access to it.
  • Update documentation
  • ./gradlew clean kspDebugKotlin should generate all models

Describe alternatives you've considered

  • Let the user copy paste Gradle code and document it (Not as convenient)
This was referenced Nov 24, 2023
Merged
Merged
Chrylo added a commit to boschglobal/kuksa-android-sdk that referenced this issue Jan 31, 2024
@Chrylo
feature: Add VSS Processor Plugin
aa3d2d2 
The plugin is a composite project which means a higher separation to
the other modules and common buildSrc plugins can't be used here. So
plugins like publish / version had to be copied. The publish script
had to be adapted for publishing a plugin

close eclipse-kuksa#45

Signed-Off-By: Mark Hüsers <mark.huesers@etas.com>
@wba2hi wba2hi linked a pull request Feb 2, 2024 that will close this issue
Add VSS Processor Plugin #71
Merged
This was referenced Feb 7, 2024
Merged
Closed
@Chrylo Chrylo mentioned this issue Feb 23, 2024
Open
@Chrylo Chrylo added this to the 0.2 milestone Feb 27, 2024
@Chrylo Chrylo added the enhancement New feature or request label Feb 27, 2024
@Chrylo Chrylo mentioned this issue Feb 27, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.1.4
Development

Successfully merging a pull request may close this issue.

Add VSS Processor Plugin boschglobal/kuksa-android-sdk
1 participant
@Chrylo