Skip to content

Commit 8cbfe5f

Browse files
committed
maven-dependency-plugin: ignoring dependencies of generated sources in arrow-vector
1 parent dc833bb commit 8cbfe5f

5 files changed

Lines changed: 76 additions & 51 deletions

File tree

java/format/pom.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@
2626
<flatc.download.skip>false</flatc.download.skip>
2727
<flatc.executable>${project.build.directory}/flatc-${os.detected.classifier}-${fbs.version}.exe</flatc.executable>
2828
<flatc.generated.files>${project.build.directory}/generated-sources/flatc</flatc.generated.files>
29-
<maven-compiler-plugin.version>3.3</maven-compiler-plugin.version>
30-
<maven-dependency-plugin.version>2.10</maven-dependency-plugin.version>
3129
<os-maven-plugin.version>1.5.0.Final</os-maven-plugin.version>
3230
</properties>
3331

@@ -52,7 +50,6 @@
5250
<plugin> <!-- download the flatbuffer compiler -->
5351
<groupId>org.apache.maven.plugins</groupId>
5452
<artifactId>maven-dependency-plugin</artifactId>
55-
<version>${maven-dependency-plugin.version}</version>
5653
<executions>
5754
<execution>
5855
<id>copy-flatc</id>

java/memory/pom.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,12 @@
4444

4545

4646
<build>
47+
<plugins>
48+
<plugin>
49+
<groupId>org.apache.maven.plugins</groupId>
50+
<artifactId>maven-dependency-plugin</artifactId>
51+
</plugin>
52+
</plugins>
4753
</build>
4854

4955

java/pom.xml

Lines changed: 47 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -273,56 +273,60 @@
273273
</configuration>
274274
</plugin>
275275

276-
<plugin>
277-
<groupId>org.apache.maven.plugins</groupId>
278-
<artifactId>maven-checkstyle-plugin</artifactId>
279-
<version>2.17</version>
280-
<dependencies>
281-
<dependency>
282-
<groupId>com.puppycrawl.tools</groupId>
283-
<artifactId>checkstyle</artifactId>
284-
<version>6.19</version>
285-
</dependency>
286-
<dependency>
287-
<groupId>com.google.guava</groupId>
288-
<artifactId>guava</artifactId>
289-
<version>${dep.guava.version}</version>
290-
</dependency>
291-
<dependency>
292-
<groupId>org.slf4j</groupId>
293-
<artifactId>jcl-over-slf4j</artifactId>
294-
<version>1.7.5</version>
295-
</dependency>
296-
</dependencies>
297-
<executions>
298-
<execution>
299-
<id>validate</id>
300-
<phase>validate</phase>
301-
<goals>
302-
<goal>check</goal>
303-
</goals>
304-
</execution>
305-
</executions>
306-
<configuration>
307-
<configLocation>google_checks.xml</configLocation>
308-
<encoding>UTF-8</encoding>
309-
<consoleOutput>true</consoleOutput>
310-
<failsOnError>${checkstyle.failOnViolation}</failsOnError>
311-
<failOnViolation>${checkstyle.failOnViolation}</failOnViolation>
312-
<violationSeverity>warning</violationSeverity>
313-
<format>xml</format>
314-
<format>html</format>
315-
<outputFile>${project.build.directory}/test/checkstyle-errors.xml</outputFile>
316-
<linkXRef>false</linkXRef>
317-
</configuration>
318-
</plugin>
276+
<plugin>
277+
<groupId>org.apache.maven.plugins</groupId>
278+
<artifactId>maven-checkstyle-plugin</artifactId>
279+
<version>2.17</version>
280+
<dependencies>
281+
<dependency>
282+
<groupId>com.puppycrawl.tools</groupId>
283+
<artifactId>checkstyle</artifactId>
284+
<version>6.19</version>
285+
</dependency>
286+
<dependency>
287+
<groupId>com.google.guava</groupId>
288+
<artifactId>guava</artifactId>
289+
<version>${dep.guava.version}</version>
290+
</dependency>
291+
<dependency>
292+
<groupId>org.slf4j</groupId>
293+
<artifactId>jcl-over-slf4j</artifactId>
294+
<version>1.7.5</version>
295+
</dependency>
296+
</dependencies>
297+
<executions>
298+
<execution>
299+
<id>validate</id>
300+
<phase>validate</phase>
301+
<goals>
302+
<goal>check</goal>
303+
</goals>
304+
</execution>
305+
</executions>
306+
<configuration>
307+
<configLocation>google_checks.xml</configLocation>
308+
<encoding>UTF-8</encoding>
309+
<consoleOutput>true</consoleOutput>
310+
<failsOnError>${checkstyle.failOnViolation}</failsOnError>
311+
<failOnViolation>${checkstyle.failOnViolation}</failOnViolation>
312+
<violationSeverity>warning</violationSeverity>
313+
<format>xml</format>
314+
<format>html</format>
315+
<outputFile>${project.build.directory}/test/checkstyle-errors.xml</outputFile>
316+
<linkXRef>false</linkXRef>
317+
</configuration>
318+
</plugin>
319+
</plugins>
319320

321+
<pluginManagement>
322+
<plugins>
320323
<plugin>
321324
<groupId>org.apache.maven.plugins</groupId>
322325
<artifactId>maven-dependency-plugin</artifactId>
323326
<version>3.0.1</version>
324327
<executions>
325328
<execution>
329+
<id>analyze</id>
326330
<goals>
327331
<goal>analyze-only</goal>
328332
</goals>
@@ -333,11 +337,6 @@
333337
</execution>
334338
</executions>
335339
</plugin>
336-
337-
</plugins>
338-
<pluginManagement>
339-
340-
<plugins>
341340
<plugin>
342341
<groupId>org.apache.rat</groupId>
343342
<artifactId>apache-rat-plugin</artifactId>

java/tools/pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,10 @@
7979
</execution>
8080
</executions>
8181
</plugin>
82+
<plugin>
83+
<groupId>org.apache.maven.plugins</groupId>
84+
<artifactId>maven-dependency-plugin</artifactId>
85+
</plugin>
8286
</plugins>
8387
</build>
8488

java/vector/pom.xml

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,11 @@
5353
<artifactId>hppc</artifactId>
5454
<version>0.7.2</version>
5555
</dependency>
56+
<dependency>
57+
<groupId>org.apache.commons</groupId>
58+
<artifactId>commons-lang3</artifactId>
59+
<version>3.6</version>
60+
</dependency>
5661
<dependency>
5762
<groupId>commons-codec</groupId>
5863
<artifactId>commons-codec</artifactId>
@@ -150,6 +155,20 @@
150155
</execution>
151156
</executions>
152157
</plugin>
158+
<plugin>
159+
<groupId>org.apache.maven.plugins</groupId>
160+
<artifactId>maven-dependency-plugin</artifactId>
161+
<executions>
162+
<execution>
163+
<id>analyze</id>
164+
<configuration>
165+
<ignoredUnusedDeclaredDependencies>
166+
<ignoredUnusedDeclaredDependency>org.apache.commons:commons-lang3</ignoredUnusedDeclaredDependency>
167+
</ignoredUnusedDeclaredDependencies>
168+
</configuration>
169+
</execution>
170+
</executions>
171+
</plugin>
153172
</plugins>
154173
<pluginManagement>
155174
<plugins>

0 commit comments

Comments
 (0)