You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by vy...@apache.org on 2023/01/17 19:13:34 UTC

[logging-log4j2] branch release-2.x updated: Fix CI

This is an automated email from the ASF dual-hosted git repository.

vy pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 19feee6617 Fix CI
19feee6617 is described below

commit 19feee661705e53870be44db3df8376add7cc912
Author: Volkan Yazıcı <vo...@yazi.ci>
AuthorDate: Tue Jan 17 20:14:35 2023 +0100

    Fix CI
---
 .github/workflows/build.yml | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 337b7ba7dc..598ec3935a 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -76,7 +76,11 @@ jobs:
         if: runner.os == 'macOS'
         run: env | grep '^JAVA'
 
-      - name: Maven "verify"
+      # Ideally, `verify` goal should be called here.
+      # Though `log4j-osgi` depends on `log4j-samples-configuration`!?!
+      # This causes `site` goal (used in the next step) to fail without an `install` first.
+      # Hence, we temporarily(🤞) use `install` instead of `verify` here.
+      - name: Maven "install"
         timeout-minutes: 60
         shell: bash
         run: |
@@ -85,8 +89,9 @@ jobs:
             -DtrimStackTrace=false \
             -Dsurefire.rerunFailingTestsCount=2 \
             -Dlog4j2.junit.fileCleanerSleepPeriodMillis=1000 \
+            -DinstallAtEnd \
             --global-toolchains ".github/workflows/maven-toolchains.xml" \
-            verify
+            install
 
       - name: Maven "site"
         timeout-minutes: 10