Check MAVEN_PARAMS for -f flag in metadata fetch 06/14106/3
authorEric Ball <eball@linuxfoundation.org>
Wed, 9 Jan 2019 17:07:16 +0000 (09:07 -0800)
committerEric Ball <eball@linuxfoundation.org>
Wed, 9 Jan 2019 22:54:35 +0000 (14:54 -0800)
maven-fetch-metadata.sh was not respecting the "-f" (for file path)
flag in MAVEN_PARAMS, causing lf-maven-merge jobs that utilize this
flag to fail.

Issue: RELENG-1630
Change-Id: I23803fd2e4a46022d85dc2befab1438853c7c3ca
Signed-off-by: Eric Ball <eball@linuxfoundation.org>
releasenotes/notes/metadata-pom-path-d67d24874f3e1f43.yaml [new file with mode: 0644]
shell/maven-fetch-metadata.sh

diff --git a/releasenotes/notes/metadata-pom-path-d67d24874f3e1f43.yaml b/releasenotes/notes/metadata-pom-path-d67d24874f3e1f43.yaml
new file mode 100644 (file)
index 0000000..04ebaed
--- /dev/null
@@ -0,0 +1,7 @@
+---
+fixes:
+  - |
+    maven-fetch-metadata.sh was not respecting the "-f" (for file path)
+    flag in MAVEN_PARAMS, causing lf-maven-merge jobs that utilize this
+    flag to fail. It will now set a path based on this flag if it is present,
+    or default to the current working directory.
index 26b2c96..4696451 100644 (file)
 
 # Uses wget to fetch a project's maven-metadata.xml files from a Maven repository.
 
+# Check for "-f" maven param, indicating a change in pom location.
+pom_path="pom.xml"
+file_path=$(echo $MAVEN_PARAMS | grep -E "\-f \S+" | awk '{ print $2 }')
+if [ ! -z $file_path ]; then
+    pom_path="$file_path/pom.xml"
+fi
+
 # Ensure we fail the job if any steps fail.
 set -xeu -o pipefail
 
@@ -20,7 +27,7 @@ project=$(xmlstarlet sel \
       -v "/x:project/x:groupId" \
     --elif "/x:project/x:parent/x:groupId" \
       -v "/x:project/x:parent/x:groupId" \
-    --else -o "" pom.xml)
+    --else -o "" $pom_path)
 project_path="${project//.//}"
 
 mkdir -p "$WORKSPACE/m2repo/$project_path"