Skip to content

Commit 28c78d5

Browse files
committed
Consolidate VERSION variables in unsafe scripts
- Replace CUSTOM_VERSION and ORIGINAL_VERSION with single VERSION - Remove redundant version update sed commands - Cleaner, simpler script logic
1 parent 1941990 commit 28c78d5

2 files changed

Lines changed: 14 additions & 33 deletions

File tree

.github/unsafe/build-custom-armeria.sh

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,29 +3,25 @@ set -e
33

44
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
55
WORK_DIR="$(mktemp -d)"
6-
CUSTOM_VERSION="1.33.4-SNAPSHOT"
7-
ORIGINAL_VERSION="1.33.3"
6+
VERSION="1.33.3"
87

98
echo "Building armeria-core with UNSAFE disabled..."
109

1110
cd "${WORK_DIR}"
1211

1312
# Clone Armeria repository
14-
git clone --depth 1 --branch "armeria-${ORIGINAL_VERSION}" https://github.com/line/armeria.git
13+
git clone --depth 1 --branch "armeria-${VERSION}" https://github.com/line/armeria.git
1514

1615
cd armeria
1716

1817
# Apply patch to remove unsafe access
1918
git apply "${SCRIPT_DIR}/unsafe.patch"
2019

21-
# Update version to snapshot version
22-
sed -i "s/version=${ORIGINAL_VERSION}/version=${CUSTOM_VERSION}/" gradle.properties
23-
2420
# Build the core module
2521
./gradlew :core:jar -x javadoc -x :docs-client:nodeSetup -x :docs-client:npmSetup -x :docs-client:npmInstall -x :docs-client:eslint -x :docs-client:lint -x :docs-client:buildWeb -x :docs-client:copyWeb
2622

2723
# Find the built JAR
28-
CORE_JAR="core/build/libs/armeria-${CUSTOM_VERSION}.jar"
24+
CORE_JAR="core/build/libs/armeria-${VERSION}.jar"
2925
if [ ! -f "$CORE_JAR" ]; then
3026
echo "Error: Could not find armeria JAR at $CORE_JAR"
3127
exit 1
@@ -36,14 +32,7 @@ mvn install:install-file -q \
3632
-Dfile="$CORE_JAR" \
3733
-DgroupId=com.linecorp.armeria \
3834
-DartifactId=armeria \
39-
-Dversion="${CUSTOM_VERSION}" \
35+
-Dversion="${VERSION}" \
4036
-Dpackaging=jar
4137

42-
# Update dependencies
43-
DEPS="${SCRIPT_DIR}/../../dependencyManagement/build.gradle.kts"
44-
45-
# Update only the armeria core artifact version
46-
sed -i '/com\.linecorp\.armeria:armeria:/d' "${DEPS}"
47-
sed -i '/^val DEPENDENCIES = listOf(/a\ "com.linecorp.armeria:armeria:'"${CUSTOM_VERSION}"'",' "${DEPS}"
48-
49-
echo "armeria ${CUSTOM_VERSION} built and installed (UNSAFE disabled)."
38+
echo "armeria ${VERSION} built and installed (UNSAFE disabled)."

.github/unsafe/build-custom-protobuf.sh

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -3,20 +3,19 @@ set -e
33

44
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
55
WORK_DIR="$(mktemp -d)"
6-
CUSTOM_VERSION="4.32.2-SNAPSHOT"
7-
ORIGINAL_VERSION="4.32.1"
6+
VERSION="4.32.1"
87

98
echo "Building protobuf-java with UNSAFE disabled..."
109

1110
cd "${WORK_DIR}"
1211

1312
# Download original artifact and extract
14-
curl -sL -o "protobuf-java-${ORIGINAL_VERSION}.jar" "https://repo1.maven.org/maven2/com/google/protobuf/protobuf-java/${ORIGINAL_VERSION}/protobuf-java-${ORIGINAL_VERSION}.jar"
13+
curl -sL -o "protobuf-java-${VERSION}.jar" "https://repo1.maven.org/maven2/com/google/protobuf/protobuf-java/${VERSION}/protobuf-java-${VERSION}.jar"
1514

1615
# Extract JAR
1716
mkdir classes
1817
cd classes
19-
jar -xf "../protobuf-java-${ORIGINAL_VERSION}.jar"
18+
jar -xf "../protobuf-java-${VERSION}.jar"
2019
cd ..
2120

2221
# Clone protobuf repository
@@ -26,24 +25,17 @@ git clone --depth 1 --branch "v32.1" https://github.com/protocolbuffers/protobuf
2625
sed -i 's/private static final sun\.misc\.Unsafe UNSAFE = getUnsafe();/private static final sun.misc.Unsafe UNSAFE = null;/' protobuf/java/core/src/main/java/com/google/protobuf/UnsafeUtil.java
2726

2827
# Compile modified classes
29-
javac -cp protobuf-java-${ORIGINAL_VERSION}.jar -d classes protobuf/java/core/src/main/java/com/google/protobuf/UnsafeUtil.java
28+
javac -cp protobuf-java-${VERSION}.jar -d classes protobuf/java/core/src/main/java/com/google/protobuf/UnsafeUtil.java
3029

31-
# Update version and create new JAR
32-
sed -i "s/Implementation-Version: ${ORIGINAL_VERSION}/Implementation-Version: ${CUSTOM_VERSION}/" classes/META-INF/MANIFEST.MF
33-
find classes/META-INF -name "pom.properties" -exec sed -i "s/version=${ORIGINAL_VERSION}/version=${CUSTOM_VERSION}/" {} \;
34-
jar -cf "protobuf-java-${CUSTOM_VERSION}.jar" -C classes .
30+
# Create new JAR with modified classes
31+
jar -cf "protobuf-java-${VERSION}.jar" -C classes .
3532

3633
# Install to Maven local repository
3734
mvn install:install-file -q \
38-
-Dfile="protobuf-java-${CUSTOM_VERSION}.jar" \
35+
-Dfile="protobuf-java-${VERSION}.jar" \
3936
-DgroupId=com.google.protobuf \
4037
-DartifactId=protobuf-java \
41-
-Dversion="${CUSTOM_VERSION}" \
38+
-Dversion="${VERSION}" \
4239
-Dpackaging=jar
4340

44-
# Update dependencies
45-
DEPS="${SCRIPT_DIR}/../../dependencyManagement/build.gradle.kts"
46-
sed -i '/com\.google\.protobuf:protobuf-java/d' "${DEPS}"
47-
sed -i '/^val DEPENDENCIES = listOf(/a\ "com.google.protobuf:protobuf-java:'"${CUSTOM_VERSION}"'",' "${DEPS}"
48-
49-
echo "protobuf-java ${CUSTOM_VERSION} built and installed (UNSAFE disabled)."
41+
echo "protobuf-java ${VERSION} built and installed (UNSAFE disabled)."

0 commit comments

Comments
 (0)