odpi#4664 quote all periods in github action as per actions/toolkit#231

Signed-off-by: Nigel Jones <nigel.l.jones+git@gmail.com>
planetf1 · Feb 25, 2021 · d1a100e · d1a100e
1 parent 8dcd46a
commit d1a100e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.github/workflows/merge.yml b/.github/workflows/merge.yml
@@ -41,9 +41,8 @@ jobs:
       # Build and publish - but only for master
       - name: Build with Maven (Publish snapshots to maven central + docker)
         if: ${{ github.repository == 'odpi/egeria' && github.ref == 'refs/heads/master'}}
-        run: mvn -B -DuseMavenCentral \
-             -Ddocker -Ddocker.taglatest -Ddocker.repo=odpi -Ddocker.registry=docker.io -Ddocker.images=core \
-             clean deploy
+        # See https://github.com/actions/toolkit/issues/231 requires parms using . to be quoted
+        run: 'mvn -B -DuseMavenCentral -Ddocker "-Ddocker.taglatest" "-Ddocker.repo=odpi" "-Ddocker.registry=docker.io" "-Ddocker.images=core" clean deploy'
         # Needed for publishing -- note we push to a staging area, login to oss.sonatype.org to 
         # verify (close) the repository & release
         env:

diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -44,9 +44,8 @@ jobs:
           gpg-passphrase: MAVEN_GPG_PASSPHRASE
       # Normal build (in future may also run reports (site))
       - name: Build with Maven
-        run: mvn -B -DuseMavenCentral \
-             -Ddocker -Ddocker.repo=odpi -Ddocker.registry=docker.io -Ddocker.images=core \
-             clean deploy        
+        # See https://github.com/actions/toolkit/issues/231 requires parms using . to be quoted
+        run: 'mvn -B -DuseMavenCentral -Ddocker "-Ddocker.repo=odpi" "-Ddocker.registry=docker.io" "-Ddocker.images=core" clean deploy'        
         # Needed for publishing -- note we push to a staging area, login to oss.sonatype.org to 
         # verify (close) the repository & release
         env: