hemantk-12 commented on code in PR #5073:
URL: https://github.com/apache/ozone/pull/5073#discussion_r1265651852
##########
hadoop-ozone/dev-support/checks/dependency.sh:
##########
@@ -23,28 +23,43 @@ REPORT_DIR=${OUTPUT_DIR:-"$DIR/../../../target/dependency"}
mkdir -p "$REPORT_DIR"
REPORT_FILE="$REPORT_DIR/summary.txt"
-hadoop-ozone/dist/src/main/license/update-jar-report.sh current.txt
+src_dir=hadoop-ozone/dist/src/main/license
-cp hadoop-ozone/dist/src/main/license/jar-report.txt "$REPORT_DIR"
-cp hadoop-ozone/dist/src/main/license/current.txt "$REPORT_DIR"
+${src_dir}/update-jar-report.sh
-#implementation of sort cli is not exactly the same everywhere. It's better to
sort with the same command locally
-(diff -uw <(sort hadoop-ozone/dist/src/main/license/jar-report.txt) <(sort
hadoop-ozone/dist/src/main/license/current.txt) || true ) > "$REPORT_FILE"
+cp ${src_dir}/jar-report.txt "$REPORT_DIR"/
+cp ${src_dir}/current.txt "$REPORT_DIR"/
+#implementation of sort cli is not exactly the same everywhere. It's better to
sort with the same command locally
+(diff -uw \
+ <(sort ${src_dir}/jar-report.txt) \
+ <(sort ${src_dir}/current.txt) \
+ || true) \
+ > "$REPORT_FILE"
if [ -s "$REPORT_FILE" ]; then
- echo ""
- echo "Jar files are added/removed to/from the binary package."
- echo ""
- echo "Please update the hadoop-ozone/dist/src/main/license/bin/LICENSE.txt
file with the modification"
- echo " AND execute
hadoop-ozone/dist/src/main/license/update-jar-report.sh when you are ready
(after a full build)"
- echo ""
- echo "Generated hadoop-ozone/dist/src/main/license/jar-report.txt file
should be added to your pull-request. It will be used as the base of future
comparison."
- echo ""
- echo "This check may also report positive for PRs if the source is not
up-to-date with the base branch (eg. \`master\`). In this case please merge
the base branch into your source branch."
- echo ""
- echo "Changed jars:"
- echo ""
- cat $REPORT_FILE
- exit 1
+ cat <<-EOF
+Jar files under share/ozone/lib in the build have changed.
+
+Please update:
+
+ * ${src_dir}/bin/LICENSE.txt
+ (add new dependencies with the appropriate license, delete any removed
dependencies)
Review Comment:
nit: should it be `(add new dependencies **under** the appropriate license,
...`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
