Skip to content

Commit

Permalink
[MINOR][INFRA] Suppress warning in check-license
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?
This PR aims to suppress the warning `File exists` in check-license

### Why are the changes needed?

**BEFORE**
```
% dev/check-license
Attempting to fetch rat
RAT checks passed.

% dev/check-license
mkdir: target: File exists
RAT checks passed.
```

**AFTER**
```
% dev/check-license
Attempting to fetch rat
RAT checks passed.

% dev/check-license
RAT checks passed.
```

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Manually do dev/check-license twice.

Closes #30460 from williamhyun/checklicense.

Authored-by: William Hyun <williamhyun3@gmail.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
(cherry picked from commit a459238)
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
  • Loading branch information
williamhyun authored and HyukjinKwon committed Nov 23, 2020
1 parent 1e525c1 commit b70584f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dev/check-license
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ mkdir -p "$FWDIR"/lib
exit 1
}

mkdir target
mkdir -p target
$java_cmd -jar "$rat_jar" -E "$FWDIR"/dev/.rat-excludes -d "$FWDIR" > target/rat-results.txt

if [ $? -ne 0 ]; then
Expand Down

0 comments on commit b70584f

Please sign in to comment.