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

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-crypto.git


The following commit(s) were added to refs/heads/master by this push:
     new 8a96ee5d Force JNA to fail if testing JNI and vice-versa
8a96ee5d is described below

commit 8a96ee5da3c739bb8070a725bae7391ebc1147c2
Author: Sebb <s...@apache.org>
AuthorDate: Tue Nov 14 12:29:36 2023 +0000

    Force JNA to fail if testing JNI and vice-versa
    
    [skip ci]
---
 .github/workflows/maven.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
index 50c91fcc..46b7bf84 100644
--- a/.github/workflows/maven.yml
+++ b/.github/workflows/maven.yml
@@ -129,7 +129,7 @@ jobs:
     - name: Check JNI and JNA tests are independent
       # N.B. the default library fails with 'java is loading libcrypto in an 
unsafe way'
       # so we need to define the appropriate library for each test
-      if: ${{ matrix.java == '8' && matrix.os == 'ubuntu-20.04' }}
+      if: ${{ matrix.java == '8' && matrix.os != 'windows-latest' }}
       run: |
-        mvn -V -B -ntp test -Ptestjni -D"jni.library.path=$ENGINESDIR"
-        mvn -V -B -ntp test -Ptestjna -D"jna.library.path=$ENGINESDIR"
+        mvn -V -B -ntp test -Ptestjni -D"jni.library.path=$ENGINESDIR" 
-Dcommons.crypto.OpenSslNativeJna=___
+        mvn -V -B -ntp test -Ptestjna -D"jna.library.path=$ENGINESDIR" 
-Djni.library.name=___

Reply via email to