Skip to content

Commit 304d848

Browse files
committed
ci: more CI testing hwo it's different than local
1 parent b5116d5 commit 304d848

File tree

3 files changed

+39
-36
lines changed

3 files changed

+39
-36
lines changed

.github/workflows/pull_request.yml

Lines changed: 35 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -176,29 +176,29 @@ jobs:
176176
- name: Swift Test
177177
run: "swift test"
178178

179-
build-swift-android:
180-
name: Sample SwiftJavaExtractJNISampleApp (Android) (${{ matrix.os_version }} swift:${{ matrix.swift_version }} jdk:${{matrix.jdk_vendor}} android:${{matrix.sdk_triple}})
181-
runs-on: ubuntu-latest
182-
strategy:
183-
fail-fast: false
184-
matrix:
185-
swift_version: ['nightly-main']
186-
os_version: ['jammy']
187-
jdk_vendor: ['corretto']
188-
sdk_triple: ['x86_64-unknown-linux-android28']
189-
ndk_version: ['r27d']
190-
container:
191-
image: ${{ (contains(matrix.swift_version, 'nightly') && 'swiftlang/swift') || 'swift' }}:${{ matrix.swift_version }}-${{ matrix.os_version }}
192-
steps:
193-
- uses: actions/checkout@v4
194-
- name: Prepare CI Environment
195-
uses: ./.github/actions/prepare_env
196-
- name: Install Swift SDK for Android and build
197-
run: |
198-
apt-get -q update && apt-get -yq install curl
199-
cd Samples/SwiftJavaExtractJNISampleApp
200-
curl -s --retry 3 https://raw.githubusercontent.com/swiftlang/github-workflows/refs/heads/main/.github/workflows/scripts/install-and-build-with-sdk.sh | \
201-
bash -s -- --android --build-command="swift build" --android-sdk-triple="${{ matrix.sdk_triple }}" --android-ndk-version="${{ matrix.ndk_version }}" ${{ matrix.swift_version }}
179+
# build-swift-android:
180+
# name: Sample SwiftJavaExtractJNISampleApp (Android) (${{ matrix.os_version }} swift:${{ matrix.swift_version }} jdk:${{matrix.jdk_vendor}} android:${{matrix.sdk_triple}})
181+
# runs-on: ubuntu-latest
182+
# strategy:
183+
# fail-fast: false
184+
# matrix:
185+
# swift_version: ['nightly-main']
186+
# os_version: ['jammy']
187+
# jdk_vendor: ['corretto']
188+
# sdk_triple: ['x86_64-unknown-linux-android28']
189+
# ndk_version: ['r27d']
190+
# container:
191+
# image: ${{ (contains(matrix.swift_version, 'nightly') && 'swiftlang/swift') || 'swift' }}:${{ matrix.swift_version }}-${{ matrix.os_version }}
192+
# steps:
193+
# - uses: actions/checkout@v4
194+
# - name: Prepare CI Environment
195+
# uses: ./.github/actions/prepare_env
196+
# - name: Install Swift SDK for Android and build
197+
# run: |
198+
# apt-get -q update && apt-get -yq install curl
199+
# cd Samples/SwiftJavaExtractJNISampleApp
200+
# curl -s --retry 3 https://raw.githubusercontent.com/swiftlang/github-workflows/refs/heads/main/.github/workflows/scripts/install-and-build-with-sdk.sh | \
201+
# bash -s -- --android --build-command="swift build" --android-sdk-triple="${{ matrix.sdk_triple }}" --android-ndk-version="${{ matrix.ndk_version }}" ${{ matrix.swift_version }}
202202

203203
verify-samples:
204204
name: Sample ${{ matrix.sample_app }} (${{ matrix.os_version }} swift:${{ matrix.swift_version }} jdk:${{matrix.jdk_vendor}})
@@ -210,12 +210,12 @@ jobs:
210210
os_version: ['jammy']
211211
jdk_vendor: ['corretto']
212212
sample_app: [ # TODO: use a reusable-workflow to generate those names
213-
'JavaDependencySampleApp',
214-
'JavaKitSampleApp',
215-
'JavaProbablyPrime',
216-
'JavaSieve',
217-
'SwiftAndJavaJarSampleLib',
218-
'SwiftJavaExtractFFMSampleApp',
213+
# 'JavaDependencySampleApp',
214+
# 'JavaKitSampleApp',
215+
# 'JavaProbablyPrime',
216+
# 'JavaSieve',
217+
# 'SwiftAndJavaJarSampleLib',
218+
# 'SwiftJavaExtractFFMSampleApp',
219219
'SwiftJavaExtractJNISampleApp',
220220
]
221221
container:
@@ -242,12 +242,12 @@ jobs:
242242
os_version: ['macos']
243243
jdk_vendor: ['corretto']
244244
sample_app: [ # TODO: use a reusable-workflow to generate those names
245-
'JavaDependencySampleApp',
246-
'JavaKitSampleApp',
247-
'JavaProbablyPrime',
248-
'JavaSieve',
249-
'SwiftAndJavaJarSampleLib',
250-
'SwiftJavaExtractFFMSampleApp',
245+
# 'JavaDependencySampleApp',
246+
# 'JavaKitSampleApp',
247+
# 'JavaProbablyPrime',
248+
# 'JavaSieve',
249+
# 'SwiftAndJavaJarSampleLib',
250+
# 'SwiftJavaExtractFFMSampleApp',
251251
'SwiftJavaExtractJNISampleApp',
252252
]
253253
steps:

Samples/SwiftJavaExtractJNISampleApp/ci-validate.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ set -e
66
# GitHub Actions workaround; We seem to be hitting gradle wrapper download issues, and this attempts to workaround them
77
./gradlew -h
88

9+
which gradle
10+
echo $PATH
11+
912
GRADLE_USER_HOME="$(pwd)/.build/plugins/outputs/swiftjavaextractjnisampleapp/MySwiftLibrary/destination/JExtractSwiftPlugin/gradle-user-home"
1013
if [ -d "$HOME/.gradle" ] ; then
1114
echo "COPY $HOME/.gradle to $GRADLE_USER_HOME"

Sources/JExtractSwiftLib/JNI/JNISwift2JavaGenerator+SwiftThunkPrinting.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,7 @@ extension JNISwift2JavaGenerator {
318318
#".init(name: "\#(name)", signature: "\#(signature.mangledName)")"#
319319
}.joined(separator: ",\n")
320320

321-
return #"_JNIMethodIDCache(environment: try! JavaVirtualMachine.shared().environment(), className: "\#(fullClassName)", methods: [\#(methods)])"#
321+
return #"_JNIMethodIDCache(className: "\#(fullClassName)", methods: [\#(methods)])"#
322322
}
323323

324324
private func printEnumGetAsCaseThunk(

0 commit comments

Comments
 (0)