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

chore(build): exclude annotation JARs from jetcd-all "fat" JAR #493

Merged
merged 1 commit into from
Mar 21, 2019

Conversation

vorburger
Copy link
Member

because they are not needed at run-time and only bloat the "fat" JAR

and can cause conflicts when building other projects depending on this:

[INFO] --- duplicate-finder-maven-plugin:1.3.0:check(find-duplicate-classpath-entries) @ jetcd ---
[WARNING] Found duplicate and different classes in [io.etcd:jetcd-all:0.3.0-SNAPSHOT, org.checkerframework:checker-compat-qual:2.5.3]:
[WARNING] org.checkerframework.checker.nullness.compatqual.KeyForDecl
[WARNING] org.checkerframework.checker.nullness.compatqual.KeyForType
[WARNING] org.checkerframework.checker.nullness.compatqual.MonotonicNonNullDecl
[WARNING] org.checkerframework.checker.nullness.compatqual.MonotonicNonNullType
[WARNING] org.checkerframework.checker.nullness.compatqual.NonNullDecl
[WARNING] org.checkerframework.checker.nullness.compatqual.NonNullType
[WARNING] org.checkerframework.checker.nullness.compatqual.NullableDecl
[WARNING] org.checkerframework.checker.nullness.compatqual.NullableType
[WARNING] org.checkerframework.checker.nullness.compatqual.PolyNullDecl
[WARNING] org.checkerframework.checker.nullness.compatqual.PolyNullType
[WARNING] Found duplicate and different classes in [io.etcd:jetcd-all:0.3.0-SNAPSHOT, org.codehaus.mojo:animal-sniffer-annotations:1.14]:
[WARNING] org.codehaus.mojo.animal_sniffer.IgnoreJRERequirement
[WARNING] Found duplicate classes/resources in test classpath.

@vorburger vorburger requested a review from lburgazzoli February 6, 2019 12:25
@vorburger
Copy link
Member Author

@lburgazzoli OK for you?

@vorburger vorburger mentioned this pull request Feb 6, 2019
vorburger added a commit to vorburger/opendaylight-etcd that referenced this pull request Feb 6, 2019
Because I suspected that may be responsible for the problem under Karaf.

Use shaded version of ByteString in the EtcdWatcherSplittingConsumerTest

and fix the duplicate-finder-maven-plugin problem with exclusions
see also etcd-io/jetcd#493
vorburger added a commit to vorburger/opendaylight-etcd that referenced this pull request Feb 6, 2019
vorburger added a commit to vorburger/opendaylight-etcd that referenced this pull request Feb 6, 2019
because they are not needed at run-time and only bloat the "fat" JAR

and can cause conflicts when building other projects depending on this:

[INFO] --- duplicate-finder-maven-plugin:1.3.0:check(find-duplicate-classpath-entries) @ jetcd ---
[WARNING] Found duplicate and different classes in [io.etcd:jetcd-all:0.3.0-SNAPSHOT, org.checkerframework:checker-compat-qual:2.5.3]:
[WARNING]   org.checkerframework.checker.nullness.compatqual.KeyForDecl
[WARNING]   org.checkerframework.checker.nullness.compatqual.KeyForType
[WARNING]   org.checkerframework.checker.nullness.compatqual.MonotonicNonNullDecl
[WARNING]   org.checkerframework.checker.nullness.compatqual.MonotonicNonNullType
[WARNING]   org.checkerframework.checker.nullness.compatqual.NonNullDecl
[WARNING]   org.checkerframework.checker.nullness.compatqual.NonNullType
[WARNING]   org.checkerframework.checker.nullness.compatqual.NullableDecl
[WARNING]   org.checkerframework.checker.nullness.compatqual.NullableType
[WARNING]   org.checkerframework.checker.nullness.compatqual.PolyNullDecl
[WARNING]   org.checkerframework.checker.nullness.compatqual.PolyNullType
[WARNING] Found duplicate and different classes in [io.etcd:jetcd-all:0.3.0-SNAPSHOT, org.codehaus.mojo:animal-sniffer-annotations:1.14]:
[WARNING]   org.codehaus.mojo.animal_sniffer.IgnoreJRERequirement
[WARNING] Found duplicate classes/resources in test classpath.
@vorburger vorburger force-pushed the all-without-annotations branch from 93978d6 to 37b1c1a Compare February 7, 2019 00:21
@lburgazzoli lburgazzoli merged commit f735a86 into etcd-io:master Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants