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

Fix build issues: 1 clean non-exists target dirs; 2 remove duplicated plugin #4456

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

res-life
Copy link
Collaborator

@res-life res-life commented Jan 5, 2022

This fixes #4445
1 clean non-exists target dirs;
2 fix warnning a warnning duplicated plugin

Signed-off-by: Chong Gao res_life@163.com

… plugin

Signed-off-by: Chong Gao <res_life@163.com>
@res-life
Copy link
Collaborator Author

res-life commented Jan 5, 2022

Use "rm -rf" instead of "rm -r".
"rf" option can handle the following paths without "sort", no need to remove the deeply subdir first.

target/a
target/a/target
target/a/target/b
target/a/target/b/target
target/a/target/b/target/c
target/a/target/b/target/c/target
target/a/target/b/target/c/target/bb
target/a/target/b/target/c/target/bb/target

BTW: Remove a dupilcated plugin to remove warning message:

$ mvn clean package -DskipTests
[INFO] Scanning for projects...
[WARNING] 
[WARNING] Some problems were encountered while building the effective model for com.nvidia:rapids-4-spark-sql_2.12:jar:22.02.0-SNAPSHOT
[WARNING] 'build.plugins.plugin.(groupId:artifactId)' must be unique but found duplicate declaration of plugin org.codehaus.mojo:build-helper-maven-plugin @ line 386, column 21
[WARNING] 
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING] 
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING] 
[INFO] ------------------------------------------------------------------------

@res-life
Copy link
Collaborator Author

res-life commented Jan 5, 2022

build

@sameerz sameerz added the build Related to CI / CD or cleanly building label Jan 5, 2022
@sameerz sameerz added this to the Dec 13 - Jan 7 milestone Jan 5, 2022
@res-life res-life merged commit c4194bb into NVIDIA:branch-22.02 Jan 6, 2022
@res-life res-life deleted the fix-clean-issue branch January 6, 2022 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] mvn clean prints an error message on a clean dir
3 participants