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

Support for multiple file configs for each applicable attribute #248

Open
Tracked by #221
sgammon opened this issue Jan 14, 2024 · 0 comments
Open
Tracked by #221

Support for multiple file configs for each applicable attribute #248

sgammon opened this issue Jan 14, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@sgammon
Copy link
Owner

sgammon commented Jan 14, 2024

Since GraalVM's original rule set, new support has dropped: now, a user can add multiple configurations in several cases (JNI, reflection, resources), and GraalVM will merge these before applying.

Unfortunately, our existing attributes assume just one file input for each of these. We should add alternative attributes which allow specifying more than one config for each. Later, it might be worth deprecating the older attributes.

Alternatives to consider:
Do we need this if the user can wrap the configurations in a filegroup themselves?

@sgammon sgammon added the enhancement New feature or request label Jan 14, 2024
@sgammon sgammon added this to the 1.0.0 milestone Jan 14, 2024
@sgammon sgammon self-assigned this Jan 14, 2024
@sgammon sgammon moved this to In Progress in GraalVM Rules for Bazel Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

1 participant