mirror of
https://github.com/vanitasvitae/Smack.git
synced 2024-10-18 20:25:59 +02:00
1e5d34eacf
Bump Gradle from 6.8.3 to 8.10.2 and increase the minimum required Java version from 8 to 11 (SMACK-953). The switch from Java 8 to 11 caused some Bytecode portability issues regarding NIO Buffers. Java changed with version 9 the return type of some subclasses of Buffer to return the specific Buffer type instead of the Buffer superclass [JDK-4774077]. For example, ByteBuffer.filp() previously returned Buffer, while it does return ByteBuffer now. This sensible change was not reflected by the Android API [1], which means that AnimalSniffer rightfully started to complain that there is no method "ByteBuffer ByteBuffer.flip()" in Android, there is only "Buffer ByteBuffer.flip()", and those are incompatible methods on Java's Bytecode layer. As workaround, this changes return charBuffer.flip().toString(); to ((java.nio.Buffer) charBuffer).flip(); return charBuffer.toString(); to restore the Bytecode portability between Android and Java. Errorprone also got new checks, of which JavaUtilDate and JdkObsolete are wroth mentioning. JavaUtilData basically strongly recommends to use Java's newer time API over java.util.Date. But since Smack was Java 8 until now, j.u.Date is widely used. Similar JdkObsolete mentions obsolete JDK APIs, like data structures like Vector and Stack. But mostly LinkedList, which should usually be replaced by ArrayList. And this is what this commit largely does. JDK-4774077: https://bugs.openjdk.org/browse/JDK-4774077 1: https://issuetracker.google.com/issues/369219141
108 lines
3.3 KiB
YAML
108 lines
3.3 KiB
YAML
name: CI
|
|
|
|
on: [push, pull_request]
|
|
|
|
jobs:
|
|
build:
|
|
name: Build Smack
|
|
|
|
runs-on: ubuntu-24.04
|
|
strategy:
|
|
matrix:
|
|
java:
|
|
- 17
|
|
env:
|
|
PRIMARY_JAVA_VERSION: 17
|
|
|
|
steps:
|
|
- name: Checkout
|
|
uses: actions/checkout@v3
|
|
|
|
- name: Set up JDK ${{ matrix.java }}
|
|
uses: actions/setup-java@v3
|
|
with:
|
|
java-version: ${{ matrix.java }}
|
|
distribution: temurin
|
|
|
|
# Caches
|
|
- name: Cache Maven
|
|
uses: actions/cache@v2
|
|
with:
|
|
path: ~/.m2/repository
|
|
key: maven-${{ hashFiles('**/build.gradle') }}
|
|
restore-keys: |
|
|
maven-
|
|
- name: Cache Gradle
|
|
uses: actions/cache@v2
|
|
with:
|
|
path: ~/.gradle/caches
|
|
key: gradle-caches-${{ hashFiles('**/build.gradle') }}
|
|
restore-keys:
|
|
gradle-caches
|
|
- name: Cache Android SDK
|
|
uses: actions/cache@v2
|
|
with:
|
|
path: |
|
|
~/.android/sdk
|
|
key: android-${{ hashFiles('build.gradle') }}
|
|
restore-keys: |
|
|
android-
|
|
|
|
# Pre-reqs
|
|
- name: Install GraphViz
|
|
run: sudo apt update && sudo apt install graphviz
|
|
- name: Install Android SDK Manager
|
|
uses: android-actions/setup-android@v3
|
|
- name: Install Android SDK
|
|
run: |
|
|
sdkmanager "platforms;android-21"
|
|
|
|
# Workaround
|
|
- name: Create gradle.properties
|
|
run: |
|
|
cat <<-EOF > gradle.properties
|
|
# Workaround for https://github.com/CycloneDX/cyclonedx-gradle-plugin/issues/349
|
|
# suggested at https://docs.gradle.org/current/userguide/upgrading_version_8.html#xml_parsing_now_requires_recent_parsers
|
|
systemProp.javax.xml.parsers.SAXParserFactory=com.sun.org.apache.xerces.internal.jaxp.SAXParserFactoryImpl
|
|
systemProp.javax.xml.transform.TransformerFactory=com.sun.org.apache.xalan.internal.xsltc.trax.TransformerFactoryImpl
|
|
systemProp.javax.xml.parsers.DocumentBuilderFactory=com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderFactoryImpl
|
|
EOF
|
|
|
|
# Testing
|
|
- name: Gradle Check
|
|
run: ./gradlew check --stacktrace
|
|
|
|
# Test local publish
|
|
- name: Gradle publish
|
|
run: ./gradlew publishToMavenLocal --stacktrace
|
|
|
|
# Javadoc
|
|
- name: Javadoc
|
|
if: ${{ matrix.java == env.PRIMARY_JAVA_VERSION }}
|
|
run: ./gradlew javadocAll --stacktrace
|
|
|
|
# Test Coverage Report
|
|
- name: Jacoco Test Coverage
|
|
env:
|
|
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
|
|
if: |
|
|
${{ matrix.java == env.PRIMARY_JAVA_VERSION }} &&
|
|
${{ env.COVERALLS_REPO_TOKEN != '' }}
|
|
run: |
|
|
if [[ -z ${COVERALLS_REPO_TOKEN} ]]; then
|
|
echo WARNING: COVERALLS_REPO_TOKEN is empty
|
|
else
|
|
echo COVERALLS_REPO_TOKEN is not empty
|
|
fi
|
|
./gradlew smack-java8-full:testCodeCoverageReport
|
|
./gradlew smack-java8-full:coveralls
|
|
|
|
# Upload build artifacts
|
|
- name: Upload build artifacts
|
|
uses: actions/upload-artifact@v4
|
|
with:
|
|
name: smack-java-${{ matrix.java }}
|
|
path: |
|
|
smack-*/build/libs/*.jar
|
|
!**/*-test-fixtures.jar
|
|
!**/*-tests.jar
|