Skip to content

Commit

Permalink
Rename MAVEN_ARGS to MVN_ARGS to fix windows issues with Maven 3.9.9
Browse files Browse the repository at this point in the history
  • Loading branch information
romain-grecourt committed Jan 17, 2025
1 parent 294bda2 commit bf747f8
Show file tree
Hide file tree
Showing 6 changed files with 23 additions and 23 deletions.
4 changes: 2 additions & 2 deletions .github/actions/common/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,8 +150,8 @@ runs:
GPG_PASSPHRASE: ${{ inputs.GPG_PASSPHRASE }}
GPG_PRIVATE_KEY: ${{ inputs.GPG_PRIVATE_KEY }}
MAVEN_SETTINGS: ${{ inputs.MAVEN_SETTINGS }}
MAVEN_ARGS: |
${{ env.MAVEN_ARGS }}
MVN_ARGS: |
${{ env.MVN_ARGS }}
-Dmaven.repo.local=${{ github.workspace }}/.m2/repository
-Dcache.record=${{ inputs.build-cache == 'read-write' }}
run: ${{ inputs.run }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
#
# Copyright (c) 2023, 2024 Oracle and/or its affiliates.
# Copyright (c) 2023, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand All @@ -23,7 +23,7 @@ on:
env:
JAVA_VERSION: 17
JAVA_DISTRO: oracle
MAVEN_ARGS: |
MVN_ARGS: |
-B -e
-Dmaven.wagon.httpconnectionManager.ttlSeconds=60
-Dmaven.wagon.http.retryHandler.count=3
Expand Down
18 changes: 9 additions & 9 deletions .github/workflows/validate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ env:
JAVA_VERSION: 17
JAVA_DISTRO: oracle
GRAALVM_VERSION: 21.3.3.1
MAVEN_ARGS: |
MVN_ARGS: |
-B -fae -e
-Dmaven.wagon.httpconnectionManager.ttlSeconds=60
-Dmaven.wagon.http.retryHandler.count=3
Expand Down Expand Up @@ -90,8 +90,8 @@ jobs:
artifact-path: cli/impl/target/helidon-cli.zip
run: |
mvn --version
mvn ${MAVEN_ARGS} build-cache:go-offline
mvn ${MAVEN_ARGS} -T8 \
mvn ${MVN_ARGS} build-cache:go-offline
mvn ${MVN_ARGS} -T8 \
-Dorg.slf4j.simpleLogger.showThreadName=true \
-DskipTests \
-P tests \
Expand All @@ -117,7 +117,7 @@ jobs:
build-cache: read-only
test-artifact-name: tests-${{ matrix.moduleSet }}-${{ matrix.os }}
run: |
mvn ${MAVEN_ARGS} \
mvn ${MVN_ARGS} \
-DreactorRule=tests \
-DmoduleSet=${{ matrix.moduleSet }} \
-Dsurefire.reportNameSuffix=${{ matrix.platform }} \
Expand All @@ -134,7 +134,7 @@ jobs:
with:
build-cache: read-only
run: |
mvn ${MAVEN_ARGS} -T 8 \
mvn ${MVN_ARGS} -T 8 \
-DskipTests \
-Pspotbugs,build-cache \
verify
Expand All @@ -150,7 +150,7 @@ jobs:
with:
build-cache: read-only
run: |
mvn ${MAVEN_ARGS} -T 8 \
mvn ${MVN_ARGS} -T 8 \
-DskipTests \
-Pjavadoc,build-cache \
package
Expand All @@ -166,7 +166,7 @@ jobs:
with:
build-cache: read-write
run: |
mvn ${MAVEN_ARGS} \
mvn ${MVN_ARGS} \
-pl ide-support/vscode-extension \
-DskipTests \
-P vscode \
Expand Down Expand Up @@ -196,15 +196,15 @@ jobs:
artifact-path: cli/impl/target/helidon-cli-${{ matrix.platform }}${{ matrix.file-ext }}
run: |
# build the executable
mvn ${MAVEN_ARGS} \
mvn ${MVN_ARGS} \
-pl cli/impl \
-DskipTests \
-Pnative-image,!toolchain \
-Dnative.image.name=helidon-cli-${{ matrix.platform }} \
package
# smoke test
mvn ${MAVEN_ARGS} \
mvn ${MVN_ARGS} \
-pl cli/tests/functional \
-Pnative-image \
-Dnative.image.name=helidon-cli-${{ matrix.platform }} \
Expand Down
4 changes: 2 additions & 2 deletions etc/scripts/checkstyle.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash
#
# Copyright (c) 2018, 2023 Oracle and/or its affiliates.
# Copyright (c) 2018, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -53,7 +53,7 @@ die(){ echo "${1}" ; exit 1 ;}
rm -f "${WS_DIR}"/target/checkstyle-*

# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} checkstyle:checkstyle-aggregate \
mvn ${MVN_ARGS} checkstyle:checkstyle-aggregate \
-f "${WS_DIR}"/pom.xml \
-Dcheckstyle.output.format="plain" \
-Dcheckstyle.output.file="${RESULT_FILE}" \
Expand Down
4 changes: 2 additions & 2 deletions etc/scripts/copyright.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash
#
# Copyright (c) 2018, 2023 Oracle and/or its affiliates.
# Copyright (c) 2018, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -49,7 +49,7 @@ readonly RESULT_FILE
die() { echo "${1}" ; exit 1 ;}

# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} \
mvn ${MVN_ARGS} \
-f ${WS_DIR}/pom.xml \
-Dhelidon.enforcer.output.file="${RESULT_FILE}" \
-Dhelidon.enforcer.rules=copyright \
Expand Down
12 changes: 6 additions & 6 deletions etc/scripts/release.sh
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
#!/bin/bash
#
# Copyright (c) 2018, 2023 Oracle and/or its affiliates.
# Copyright (c) 2018, 2025 Oracle and/or its affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -109,7 +109,7 @@ exec 6>&1 1>&2

current_version() {
# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} -q \
mvn ${MVN_ARGS} -q \
-f "${WS_DIR}"/pom.xml \
-Dexec.executable="echo" \
-Dexec.args="\${project.version}" \
Expand All @@ -133,7 +133,7 @@ update_version(){
fi

# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} "${ARGS[@]}" \
mvn ${MVN_ARGS} "${ARGS[@]}" \
-f "${WS_DIR}"/pom.xml versions:set versions:set-property \
-DgenerateBackupPoms="false" \
-DnewVersion="${version}" \
Expand Down Expand Up @@ -176,7 +176,7 @@ release_build(){
if [ -n "${MAVEN_SETTINGS}" ] ; then
tmpfile=$(mktemp XXXXXXsettings.xml)
echo "${MAVEN_SETTINGS}" > "${tmpfile}"
MAVEN_ARGS="${MAVEN_ARGS} -s ${tmpfile}"
MVN_ARGS="${MVN_ARGS} -s ${tmpfile}"
fi
if [ -n "${GPG_PRIVATE_KEY}" ] ; then
tmpfile=$(mktemp XXXXXX.key)
Expand All @@ -193,7 +193,7 @@ release_build(){

# Perform local deployment
# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} "${ARGS[@]}" \
mvn ${MVN_ARGS} "${ARGS[@]}" \
deploy \
-Prelease \
-DskipTests \
Expand All @@ -202,7 +202,7 @@ release_build(){
# Upload all artifacts to nexus
version=$(release_version)
# shellcheck disable=SC2086
mvn ${MAVEN_ARGS} -N nexus-staging:deploy-staged \
mvn ${MVN_ARGS} -N nexus-staging:deploy-staged \
-DstagingDescription="Helidon Build Tools v${version}"
}

Expand Down

0 comments on commit bf747f8

Please sign in to comment.