This is an automated email from the ASF dual-hosted git repository.
wusheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/skywalking-graalvm-distro.git
The following commit(s) were added to refs/heads/main by this push:
new bcf9531 Fix CI: install classified precompiler JAR before
javadoc/test steps
bcf9531 is described below
commit bcf953165651596acca59066be4a210b2560652d
Author: Wu Sheng <[email protected]>
AuthorDate: Thu Feb 19 22:37:26 2026 +0800
Fix CI: install classified precompiler JAR before javadoc/test steps
The precompiler produces a classified JAR (precompiler-*-generated.jar) at
the package phase. The javadoc and test steps need this artifact resolved
but ran before it was available. Change compile target to mvn install so
the artifact is in the local repo for subsequent steps. Also skip tests in
compile and build-distro since tests run as a dedicated step.
Co-Authored-By: Claude Opus 4.6 <[email protected]>
---
.github/workflows/ci.yml | 2 +-
Makefile | 11 +++++++----
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 5c558d1..13bc159 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -56,7 +56,7 @@ jobs:
- name: Build SkyWalking submodule
run: make build-skywalking
- - name: Style check + Compile
+ - name: Style check + Compile + Install
run: make compile
- name: Javadoc
diff --git a/Makefile b/Makefile
index 3d8068a..01d7424 100644
--- a/Makefile
+++ b/Makefile
@@ -35,9 +35,12 @@ init-submodules:
build-skywalking: init-submodules
cd skywalking && ../mvnw flatten:flatten install -DskipTests
-Dmaven.javadoc.skip=true -Dcheckstyle.skip=true -Dgpg.skip=true
-# Compile only (no tests)
+# Compile + install to local repo (no tests).
+# Install is needed because the precompiler produces a classified JAR
(precompiler-*-generated.jar)
+# at the package phase. Subsequent standalone goals (javadoc:javadoc, test)
resolve this artifact
+# from the local Maven repo, so it must be installed first.
compile:
- $(MVN) clean compile -DskipTests $(MVN_ARGS)
+ $(MVN) clean install -DskipTests $(MVN_ARGS)
# Run tests (includes compile)
test:
@@ -47,9 +50,9 @@ test:
javadoc:
$(MVN) javadoc:javadoc -DskipTests $(MVN_ARGS)
-# Build the distro modules (compile + test + package + assembly)
+# Build the distro modules (package + assembly, no tests)
build-distro:
- $(MVN) clean package $(MVN_ARGS)
+ $(MVN) clean package -DskipTests $(MVN_ARGS)
# Show the distribution directory
dist: build-distro
