native tests: upgrade to GraalVM v22

Native-image now reports time and RAM usage.
This allowed me to drop using time command to get an idea these numbers.

Now testing under both JDK11 and JDK17.

Now using setup-graalvm GitHub Action from Graal team.
This action should handle vcvars.bat setup on Windows.

Author: lread

.github/workflows/native-image-test.yml

@@ -8,7 +8,7 @@ jobs:
     strategy:
       matrix:
         os: [ windows-latest, ubuntu-latest, macos-latest ]
-        graal-java: [ java11 ]
+        graal-java: [ 11, 17 ]
         test: [ test-native, test-native-sci ]
         clojure-version: [ '1.10', '1.11' ]
 
@@ -29,23 +29,16 @@ jobs:
         key: $${ runner.os }}-cljdeps-${{ hashFiles('deps.edn') }}
         restore-keys: $${ runner.os }}-cljdeps-
 
-    #
-    # Time command
-    #
-    - name: Install time command (linux)
-      run: |
-        sudo apt-get update
-        sudo apt-get install -y time
-      if: matrix.os == 'ubuntu-latest'
-
     #
     # Install GraalVM
     #
     - name: Install GraalVM
-      uses: DeLaGuardo/setup-graalvm@5.0
+      uses: graalvm/setup-graalvm@v1
       with:
-        graalvm: '21.3.0'
-        java: ${{ matrix.graal-java }}
+        version: '22.0.0.2'
+        java-version: ${{ matrix.graal-java }}
+        components: 'native-image'
+        github-token: ${{ secrets.GITHUB_TOKEN }}
 
     #
     # Install Babashka
@@ -104,15 +97,7 @@ jobs:
       run: clojure -Spath
 
     #
-    # SCI native image tests
+    # native image tests
     #
-    - name: Execute ${{ matrix.test }} native test (windows)
-      run: |
-        # it is overly awkward to get vcvars64.bat loaded into powershell, so we use cmd shell instead
-        call "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliary\Build\vcvars64.bat"
-        bb ${{ matrix.test }}
-      shell: cmd
-      if: matrix.os == 'windows-latest'
-    - name: Execute ${{ matrix.test }} native test (macos, linux)
+    - name: Execute ${{ matrix.test }} native test
       run: bb ${{ matrix.test }} --clojure-version ${{ matrix.clojure-version }}
-      if: matrix.os != 'windows-latest'

doc/02-developer-guide.adoc

@@ -13,15 +13,15 @@ We make use of planck for cljs bootstrap (aka cljs self-hosted) testing.
 Planck is currently not available for Windows.
 
 We test that rewrite-clj operates as expected when natively compile via GraalVM.
-Automated testing is setup using GraalVM v21 JDK11.
+Automated testing is setup using GraalVM v22 JDK11 and JDK17.
 
 == Prerequisites
 * Java JDK 1.8 or above
 * NodeJs v12 or above
 * Clojure v1.10.1.697 or above for `clojure` command
 ** Note that rewrite-clj v1 itself supports Clojure v1.8 and above
 * Babashka v0.3.7 or above
-* GraalVM v21.2.0 JDK 11 (if you want to run GraalVM native image tests)
+* GraalVM v22.0.0.2 JDK 11/17 (if you want to run GraalVM native image tests)
 
 === Windows Notes
 

script/helper/graal.clj

@@ -102,11 +102,5 @@
                                "-cp" (str classpath java.io.File/pathSeparator "classes")
                                (str "-J-Xmx" native-image-xmx)
                                entry-class]
-                              (remove nil?))
-         time-cmd (let [os (env/get-os)]
-                    (case os
-                        :mac ["/usr/bin/time" "-l"]
-                        :unix ["/usr/bin/time" "-v"]
-                        (status/line :warn (str "I don't know how to get run stats (user/real/sys CPU, RAM use, etc) for a command on " os))))]
-
-    (apply shell/command (concat time-cmd native-image-cmd))))
+                              (remove nil?))]
+    (apply shell/command native-image-cmd)))