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

Provide a way to specify exclusions for a dependency #2598

Closed
tadayosi opened this issue Aug 26, 2021 · 4 comments
Closed

Provide a way to specify exclusions for a dependency #2598

tadayosi opened this issue Aug 26, 2021 · 4 comments

Comments

@tadayosi
Copy link
Member

Related #2594.
At this moment, there is no way to exclude some sub-dependencies from a dependency with Dependencies trait. There should be a case when you want to use some lib but it doesn't work well with the rest of the libs without excluding some dependents.

@squakez
Copy link
Contributor

squakez commented Aug 31, 2021

Maybe we can add a exclusions parameter in the trait which looks for the dependencies and remove them from the dependency list.

@lburgazzoli
Copy link
Contributor

We need to be really careful about exposing such advanced feature as we did it already in the past but it ended up being hard to maintain, the ux was very bad and breaking the build was quite easy so IMHO, better to solve the issue in our catalog (especially with libraries like jolokia where the user is not even aware of what we are doing)

An alternative option could be to have the user provide a bom with an explicit dependency management section in which he can tweak every depedency however.

@tadayosi
Copy link
Member Author

I agree with @lburgazzoli. The solution wouldn't have to be easy. At least if there is a way to resolve it for the users who encounter such an issue, that's fine. I like the idea of allowing the users to provide a custom bom as an advanced option.

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale due to 90 days of inactivity.
It will be closed if no further activity occurs within 15 days.
If you think that’s incorrect or the issue should never stale, please simply write any comment.
Thanks for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants