ArC - extensions should be able to specify artifacts that should be excluded from discovery

[mkouba]

See also https://groups.google.com/g/quarkus-dev/c/5DGnNbZ9zdo.

We should introduce a new build item, e.g. ExcludeDependencyBuildItem (with groupdId, artifactId and optional classifier) and include these build items in the processing here: BeanArchiveProcessor#isApplicationArchiveExcluded().

[quarkus-bot]

/cc @manovotn

[masini]

Hi Martin, I introduced the

public final class ExcludeDependencyBuildItem extends MultiBuildItem {
    private final String groupId;
    private final String artifactId;
    private final String classifier;

    public ExcludeDependencyBuildItem(String groupId, String artifactId) {
        this(groupId, artifactId, null);
    }
...
...

which is used inside private boolean isApplicationArchiveExcluded(ArcConfig config, ApplicationArchive archive) and then I tested inside the SmallRyeReactiveMessagingMqttProcessor and it works.

Do you think can be a good idea to create a PR ?

I modified two modules:

• quarkus-core-deployment
• quarkus-arc-deployment

[mkouba]

Do you think can be a good idea to create a PR ?

Sure thing. Go ahead! :-)

[masini]

Created the PR #15781, let me know

[mkouba]

The PR was merged.

[masini]

Thank you Martin. Il giorno gio 25 mar 2021 alle ore 08:46 Martin Kouba < ***@***.***> ha scritto:

…

The PR was merged. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#15713 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAPXERVCK6M5SUFJSGSCAA3TFLS4DANCNFSM4ZF7DKOQ> .

-- **************************************** http://www.lucamasini.net http://twitter.com/lmasini http://www.linkedin.com/pub/luca-masini/7/10/2b9 ****************************************