You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2022/06/29 13:31:53 UTC

[commons-crypto] branch master updated (c7c5cba -> 7b163dc)

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

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


    from c7c5cba  Try alternate fix
     new a00ad68  Need Java8 for now
     new 7b163dc  Need Java8 for now

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .github/workflows/codeql-analysis.yml | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)


[commons-crypto] 01/02: Need Java8 for now

Posted by se...@apache.org.
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

commit a00ad68006acf8b1bf81db183cc87fdbc5868d87
Author: Sebb <se...@apache.org>
AuthorDate: Wed Jun 29 14:29:53 2022 +0100

    Need Java8 for now
---
 .github/workflows/codeql-analysis.yml | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index 71aa5bc..f740025 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -53,12 +53,13 @@ jobs:
 
     # Override Java default
     # Java 11 complains about illegal access; drop this override when sorted
-#    - name: Set up JDK ${{ matrix.java }}
-#      if: ${{ matrix.language == 'java' }}
-#      uses: actions/setup-java@v3
-#      with:
-#        distribution: 'temurin'
-#        java-version: ${{ matrix.java }}
+    # Also complains: "Corrupted channel by directly writing to native stream in forked JVM 1"
+   - name: Set up JDK ${{ matrix.java }}
+     if: ${{ matrix.language == 'java' }}
+     uses: actions/setup-java@v3
+     with:
+       distribution: 'temurin'
+       java-version: ${{ matrix.java }}
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
@@ -83,7 +84,7 @@ jobs:
     #    uses a compiled language
 
     - name: Build with Maven
-      run: mvn -V package  --no-transfer-progress -Drat.skip -DargLine=--add-opens=java.base/sun.nio.ch=ALL-UNNAMED
+      run: mvn -V package  --no-transfer-progress -Drat.skip -Danimal.sniffer.skip -DargLine=--add-opens=java.base/sun.nio.ch=ALL-UNNAMED
 #      make bootstrap
 #      make release
 


[commons-crypto] 02/02: Need Java8 for now

Posted by se...@apache.org.
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

commit 7b163dcd5d20c556a7fb5df5871a7b3a4bce6890
Author: Sebb <se...@apache.org>
AuthorDate: Wed Jun 29 14:31:46 2022 +0100

    Need Java8 for now
---
 .github/workflows/codeql-analysis.yml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/codeql-analysis.yml b/.github/workflows/codeql-analysis.yml
index f740025..70833d5 100644
--- a/.github/workflows/codeql-analysis.yml
+++ b/.github/workflows/codeql-analysis.yml
@@ -84,7 +84,8 @@ jobs:
     #    uses a compiled language
 
     - name: Build with Maven
-      run: mvn -V package  --no-transfer-progress -Drat.skip -Danimal.sniffer.skip -DargLine=--add-opens=java.base/sun.nio.ch=ALL-UNNAMED
+      # -DargLine=--add-opens=java.base/sun.nio.ch=ALL-UNNAMED (not with Java 8)
+      run: mvn -V package  --no-transfer-progress -Drat.skip -Danimal.sniffer.skip
 #      make bootstrap
 #      make release