Set osx architectures

Update to new github permissions

.github/workflows/build-all.yml

@@ -15,6 +15,8 @@ on:
 jobs:
   create-release:
     runs-on: ubuntu-latest
+    permissions:
+      contents: write
     outputs:
       release_tag_name: ${{ env.release_tag_name }}
       build_meta: ${{ env.build_meta }}
@@ -50,6 +52,8 @@ jobs:
   build-linux-amd64:
     runs-on: ubuntu-latest
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       - name: Set up QEMU
@@ -75,6 +79,8 @@ jobs:
   build-linux-arm64:
     runs-on: ubuntu-latest
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       - name: Set up QEMU
@@ -100,6 +106,8 @@ jobs:
   build-linux-arm:
     runs-on: ubuntu-latest
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       - name: Set up QEMU
@@ -123,8 +131,10 @@ jobs:
 
 
   build-windows-amd64:
-    runs-on: windows-2019
+    runs-on: windows-2022
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       -
@@ -141,8 +151,10 @@ jobs:
 
 
   build-windows-i386:
-    runs-on: windows-2019
+    runs-on: windows-2022
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       -
@@ -159,8 +171,10 @@ jobs:
           
   
   build-windows-arm64:
-    runs-on: windows-2019
+    runs-on: windows-2022
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       -
@@ -179,6 +193,8 @@ jobs:
   build-macosx-amd64:
     runs-on: macos-12
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       -
@@ -244,6 +260,8 @@ jobs:
   build-macosx-arm64:
     runs-on: macos-12
     needs: create-release
+    permissions:
+      contents: write
     steps:
       - uses: actions/checkout@v3
       -

.github/workflows/build-windows-amd64.yml

@@ -14,7 +14,7 @@ on:
 
 jobs:
   build-windows-amd64:
-    runs-on: windows-2019
+    runs-on: windows-2022
     steps:
       - uses: actions/checkout@v3
       -

.github/workflows/build-windows-arm64.yml

@@ -14,7 +14,7 @@ on:
 
 jobs:
   build-windows-arm64:
-    runs-on: windows-2019
+    runs-on: windows-2022
     steps:
       - uses: actions/checkout@v3
       -

.github/workflows/build-windows-i386.yml

@@ -14,7 +14,7 @@ on:
 
 jobs:
   build-windows-i386:
-    runs-on: windows-2019
+    runs-on: windows-2022
     steps:
       - uses: actions/checkout@v3
       -

DockerfileLinux

@@ -22,7 +22,6 @@ COPY natives /natives
 
 #Copy cmake patching script
 COPY scripts/patch_cmake.py .
-COPY patch/CMakeLists.txt.patch .
 
 #Copy and launch run script
 COPY scripts/run_linux.sh .

DockerfileLinuxARMPrebuild

@@ -19,6 +19,9 @@ WORKDIR /builder
 #Copy existing sources, if any
 COPY jcef /jcef
 
+#Copy cmake patching script
+COPY scripts/patch_cmake.py .
+
 #Copy and launch run script
 COPY scripts/run_linux_prebuild.sh .
 RUN chmod +x run_linux_prebuild.sh

DockerfileWindows

@@ -4,7 +4,6 @@ WORKDIR C:/
 
 #Copy cmake patching script
 COPY scripts/patch_cmake.py .
-COPY patch/CMakeLists.txt.patch .
 
 #Copy and launch run script
 COPY scripts/run_windows.bat .

README.md

@@ -19,7 +19,7 @@
   </tr>
   <tr>
     <td><b>Java</b></td>
-    <td>OpenJDK 11</td>
+    <td>OpenJDK 17</td>
     <td>Oracle JDK 8</td>
     <td>Microsoft JDK 11</td>
     <td>Temurin JDK 8</td>
@@ -27,8 +27,8 @@
   <tr>
     <td><b>Compiler</b></td>
     <td>GCC 10</td>
-    <td>VS 2019</td>
+    <td>VS 2022</td>
-    <td>VS 2019</td>
+    <td>VS 2022</td>
     <td>Xcode 13</td>
   </tr>
   <tr>

compile_macosx.sh

@@ -45,6 +45,10 @@ else
     cd jcef
 fi
 
+#CMake patching
+python3 ../scripts/patch_cmake.py cmake/DownloadCEF.cmake
+python3 ../scripts/patch_cmake.py CMakeLists.txt
+
 # Create and enter the `jcef_build` directory.
 # The `jcef_build` directory name is required by other JCEF tooling
 # and should not be changed.
@@ -55,15 +59,16 @@ cd jcef_build
 
 # MacOS: Generate amd64/arm64 Makefiles.
 if [ ${TARGETARCH} == 'amd64' ]; then
-    cmake -G "Ninja" -DPROJECT_ARCH="x86_64" -DCMAKE_BUILD_TYPE=${BUILD_TYPE} ..
+    cmake -G "Ninja" -DPROJECT_ARCH="x86_64" -DCMAKE_OSX_ARCHITECTURES="x86_64" -DCMAKE_BUILD_TYPE=${BUILD_TYPE} ..
 else
-    cmake -G "Ninja" -DPROJECT_ARCH="arm64" -DCMAKE_BUILD_TYPE=${BUILD_TYPE} ..
+    cmake -G "Ninja" -DPROJECT_ARCH="arm64" -DCMAKE_OSX_ARCHITECTURES="arm64" -DCMAKE_BUILD_TYPE=${BUILD_TYPE} ..
 fi
 # Build native part using ninja.
 ninja -j4
 
 #Generate distribution
 cd ../tools
+sed -i "" 's/--ignore-source-errors//g' make_docs.sh
 chmod +x make_distrib.sh
 ./make_distrib.sh macosx64
 cd ..

macosx_codesign.sh

@@ -36,6 +36,8 @@ codesign --force --options runtime --entitlements "$ENTITLEMENTS_HELPER" --sign
 bash macosx_notarize.sh "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/jcef Helper (Plugin).app" "$2" $3 org.jcef.jcef.helper.plugin $4 $5 $6
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_HELPER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/jcef Helper (Renderer).app"
 bash macosx_notarize.sh "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/jcef Helper (Renderer).app" "$2" $3 org.jcef.jcef.helper.renderer $4 $5 $6
+codesign --force --options runtime --entitlements "$ENTITLEMENTS_HELPER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/jcef Helper (Alerts).app"
+bash macosx_notarize.sh "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/jcef Helper (Alerts).app" "$2" $3 org.jcef.jcef.helper.alerts $4 $5 $6
 
 #Sign libraries and framework
 echo "Signing libraries and framework..."
@@ -43,6 +45,7 @@ codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/$FRAMEWORK_NAME/Libraries/libGLESv2.dylib"
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/$FRAMEWORK_NAME/Libraries/libvk_swiftshader.dylib"
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/$FRAMEWORK_NAME"
+touch "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/$FRAMEWORK_NAME/Contents/CodeResources" # Small trick to not let the validate action fail
 bash macosx_notarize.sh "$APP_DIR/$APP_NAME/$FRAMEWORKS_DIR/$FRAMEWORK_NAME" "$2" $3 org.cef.framework $4 $5 $6
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/Contents/Java/libjcef.dylib"
 bash macosx_codesign_zip.sh "$APP_DIR/$APP_NAME/Contents/Java/gluegen-rt-natives-macosx-universal.jar" "natives/macosx-universal/libgluegen_rt.dylib" "$2"
@@ -51,6 +54,7 @@ bash macosx_codesign_zip.sh "$APP_DIR/$APP_NAME/Contents/Java/jogl-all-natives-m
 bash macosx_codesign_zip.sh "$APP_DIR/$APP_NAME/Contents/Java/jogl-all-natives-macosx-universal.jar" "natives/macosx-universal/libjogl_mobile.dylib" "$2"
 bash macosx_codesign_zip.sh "$APP_DIR/$APP_NAME/Contents/Java/jogl-all-natives-macosx-universal.jar" "natives/macosx-universal/libnewt_head.dylib" "$2"
 bash macosx_codesign_zip.sh "$APP_DIR/$APP_NAME/Contents/Java/jogl-all-natives-macosx-universal.jar" "natives/macosx-universal/libjogl_desktop.dylib" "$2"
+codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME/Contents/MacOS/JavaAppLauncher"
 codesign --force --options runtime --entitlements "$ENTITLEMENTS_BROWSER" --sign "$2" --timestamp --verbose "$APP_DIR/$APP_NAME"
 bash macosx_notarize.sh "$APP_DIR/$APP_NAME" "$2" $3 org.jcef.jcef $4 $5 $6
 

patch/CMakeLists.txt.patch

@@ -1,44 +0,0 @@
-# Determine the platform.
-if("${CMAKE_SYSTEM_NAME}" STREQUAL "Darwin")
-  if("${PROJECT_ARCH}" STREQUAL "arm64")
-    set(CEF_PLATFORM "macosarm64")
-  else()
-    set(CEF_PLATFORM "macosx64")
-  endif()
-elseif("${CMAKE_SYSTEM_NAME}" STREQUAL "Linux")
-  if("${PROJECT_ARCH}" STREQUAL "amd64")
-    set(CEF_PLATFORM "linux64")
-  elseif("${PROJECT_ARCH}" STREQUAL "arm64")
-    set(CEF_PLATFORM "linuxarm64")
-  elseif("${PROJECT_ARCH}" STREQUAL "arm/v6")
-    set(CEF_PLATFORM "linuxarm")
-  else()
-    set(CEF_PLATFORM "linux32")
-  endif()
-elseif("${CMAKE_SYSTEM_NAME}" STREQUAL "Windows")
-  #Stolen from Jetbrains jcef repository for the arm64 windows build
-  if(MSVC)
-    include(CheckSymbolExists)
-
-    # MSVC predefines _M_ARM64 for compilations that target ARM64
-    # and _M_AMD64 for compilations that target x86_64.
-    check_symbol_exists("_M_ARM64" "" CEF_PLATFORM_WINARM64)
-    check_symbol_exists("_M_AMD64" "" CEF_PLATFORM_WIN64)
-
-    # We also should set PROJECT_ARCH explicitly because FindCEF.cmake deduces it incorrectly for
-    # cross-compilation cases.
-    if(CEF_PLATFORM_WINARM64)
-      set(CEF_PLATFORM "windowsarm64")
-      set(PROJECT_ARCH "arm64")
-    elseif(CEF_PLATFORM_WIN64)
-      set(CEF_PLATFORM "windows64")
-      set(PROJECT_ARCH "x86_64")
-    else()
-      set(CEF_PLATFORM "windows32")
-      set(PROJECT_ARCH "x86")
-    endif()
-  else()
-    message(FATAL_ERROR "Building JCEF for Windows using non-MSVC compiler is not supported.")
-  endif()
-endif()
-

scripts/patch_cmake.py

@@ -1,35 +1,23 @@
-#Small script to patch CMakeLists.txt files with custom build options
+#Small script to patch our cmake download locations
-#Will replace file contents between two markers ("Determine the platform"
+#Usage: python patch_cmake.py <file>
-#and "Add this project's cmake")
-#Usage: python patch_cmake.py <input> <patch>
 
 import sys
 
 input = sys.argv[1]
-patch = sys.argv[2]
 
-print("Patching "+input+" to accept further build architectures...")
+print("Patching "+input+"...")
 
 f = open(input, "r")
-p = open(patch, "r")
 result = ""
-inpatch = False
 for x in f:
-  if x.startswith("# Determine the platform"):
+  result += x.replace("https://cache-redirector.jetbrains.com/intellij-jbr/", "https://cef-builds.spotifycdn.com/") \
-    inpatch = True
+      .replace("_minimal", "") \
-    for y in p:
+      .replace(".zip", ".tar.bz2") \
-      result += y
+      .replace(".checksum", ".sha1") \
-  elif x.startswith("# Add this project's cmake"):
+      .replace("SHA256", "SHA1") \
-    inpatch = False
+      .replace("x86_64", "amd64")
-  if inpatch == False:
-    #Patch minimum cmake version to not break our builds on linux
-    if x.startswith("cmake_minimum_required"):
-      result += "cmake_minimum_required(VERSION 3.13)\n"
-    else:
-      result += x
 
 f.close()
-p.close()
 
 f = open(input, "w")
 f.write(result)

scripts/run_linux.sh

@@ -9,6 +9,9 @@ if [ ${TARGETARCH} == 'arm/v6' ]; then
     export JAVA_HOME=/usr/lib/jvm/java-11-openjdk-armel
 fi
 
+export PATH=$PATH:/usr/lib/jvm/openjdk-17/bin
+export JAVA_HOME=/usr/lib/jvm/openjdk-17
+
 # Print some debug info
 echo "-------------------------------------"
 echo "JAVA_HOME: $JAVA_HOME"
@@ -26,10 +29,11 @@ if [ ! -f "/jcef/README.md" ]; then
 else
     echo "Found existing files to build"
     cd /jcef
-fi  
+fi
 
-#CMakeLists patching
+#CMake patching
-python3 /builder/patch_cmake.py CMakeLists.txt /builder/CMakeLists.txt.patch
+python3 /builder/patch_cmake.py cmake/DownloadCEF.cmake
+python3 /builder/patch_cmake.py CMakeLists.txt
 
 # Create and enter the `jcef_build` directory.
 # The `jcef_build` directory name is required by other JCEF tooling

scripts/run_linux_prebuild.sh

@@ -25,6 +25,10 @@ else
     cd /jcef
 fi
 
+#CMake patching
+python3 /builder/patch_cmake.py cmake/DownloadCEF.cmake
+python3 /builder/patch_cmake.py CMakeLists.txt
+
 # Create and enter the `jcef_build` directory.
 # The `jcef_build` directory name is required by other JCEF tooling
 # and should not be changed.

scripts/run_windows.bat

@@ -12,16 +12,16 @@ if exist "jcef\README.md" (echo "Found existing files to build" && cd jcef) ^
 else (echo "Did not find files to build - cloning..." && GOTO :CLONE)
 
 :BUILD
-:: CMakeLists patching 
+:: CMakeLists patching
-python C:/patch_cmake.py CMakeLists.txt C:/CMakeLists.txt.patch
+python C:/patch_cmake.py cmake/DownloadCEF.cmake
 
 :: Prepare build dir
 mkdir jcef_build && cd jcef_build
 
 :: Load vcvars for 32 or 64-bit builds
-if "%TARGETARCH%"=="386" (call "C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Auxiliary\Build\vcvars32.bat")
+if "%TARGETARCH%"=="386" (call "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Auxiliary\Build\vcvars32.bat")
-if "%TARGETARCH%"=="amd64" (call "C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Auxiliary\Build\vcvars64.bat")
+if "%TARGETARCH%"=="amd64" (call "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Auxiliary\Build\vcvars64.bat")
-if "%TARGETARCH%"=="arm64" (call "C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Auxiliary\Build\vcvarsamd64_arm64.bat")
+if "%TARGETARCH%"=="arm64" (call "C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\VC\Auxiliary\Build\vcvarsamd64_arm64.bat")
 
 :: Edit PATH variable on 386 to use 32 bit jdk (cmake findjni does not actually care about JAVA_HOME)
 if "%TARGETARCH%"=="386" (set "PATH=C:/Program Files (x86)/Java/jdk1.8.0_211;%PATH%")