apache / apache/drill

Build fails because redundant “git.properties” definition from "pom.xml"

Open
#2,966 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
2k
Forks
990
Avg merge
1d 8h
Merged PRs (30d)
5

Description

**Describe the bug**
There are some redundant “git.properties” definition from "pom.xml" which causes build fail.

**To Reproduce**
1. Build the project via:
mvn clean package -DskipTests

2. Build fails with below error message:
java.nio.file.NoSuchFileException: xxx\\distribution\..\git.properties

**Expected behavior**
Build should succeeds.

**Drill version**
1.21.2

**Additional context**
I've checked "pom.xml" and found the root cause of this issue:

```
>
> ${project.build.outputDirectory}
>
>
>
> ${maven.multiModuleProjectDirectory}
>
> git.properties
>
>
>
```

But, "git.properties" also exist on `...`definition, which causes conflict.

```
>
>
>
> **/*.accdb
> **/*.access_log
> **/*.access_log
> **/.asf.yaml
> **/*.autotools
> **/*.avi
> **/*.avro
> **/.buildpath
> **/*.checkstyle
> **/.classpath
> **/client/*build*/**
> **/clientlib/y2038/*.c
> **/clientlib/y2038/*.h
> **/client/tags
> **/cmake_install.cmake
> **/*.cnf
> **/control
> **/*.cproject
> **/*.css
> **/*.csv
> **/*.csvh
> **/*.csvh-test
> **/*.dbf
>
> dependency-reduced-pom.xml
> **/*.drill
> **/.drill.parquet_metadata
> **/*.eps
> ****/git.properties**
> ...
```

So, it's necessary to remove the redundant “git.properties” definition from "`...`" block.

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect the root pom.xml resource and exclusion configuration, then reproduce the failure with `mvn clean package -DskipTests`. Confirm the redundant git.properties handling is resolved and that the build completes without the NoSuchFileException.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.