Support Native Image for Edge

.github/workflows/junit.yml

@@ -38,7 +38,7 @@ jobs:
            done
 
       - name: Publish Unit Test Results
-        uses: EnricoMi/publish-unit-test-result-action@afb2984f4d89672b2f9d9c13ae23d53779671984 # v2.19.0
+        uses: EnricoMi/publish-unit-test-result-action@3a74b2957438d0b6e2e61d67b05318aa25c9e6c6 # v2.20.0
         id: test-results
         with:
           commit: ${{ github.event.workflow_run.head_sha }}

.github/workflows/maven.yml

@@ -62,7 +62,7 @@ jobs:
     - name: Set up Maven
       uses: stCarolas/setup-maven@v5
       with:
-        maven-version: 3.9.9
+        maven-version: 3.9.10
     - name: Build
       env:
         GTK_XCFLAGS: '-Wno-deprecated-declarations'
@@ -72,7 +72,7 @@ jobs:
         --threads 1C
         -DforkCount=1
         '-Dnative=${{ matrix.config.native }}'
-        -Papi-check
+        -Papi-check -Pjavadoc
         '-Dtycho.baseline.replace=none'
         --fail-at-end
         -DskipNativeTests=false

Jenkinsfile

@@ -44,7 +44,7 @@ def runOnNativeBuildAgent(String platform, Closure body) {
 
 /** Returns the download URL of the JDK against whose C headers (in the 'include/' folder) and native libraries the SWT natives are compiled.*/
 def getNativeJdkUrl(String os, String arch) { // To update the used JDK version update the URL template below
-	if('win32'.equals(os) && 'aarch64'.equals(arch)) {
+	if ('win32'.equals(os) && 'aarch64'.equals(arch)) {
 		// Temporary workaround until there are official Temurin GA releases for Windows on ARM that can be consumed through JustJ
 		dir("${WORKSPACE}/repackage-win32.aarch64-jdk") {
 			sh """
@@ -71,7 +71,7 @@ def getNativeJdkUrl(String os, String arch) { // To update the used JDK version
 }
 
 def getLatestGitTag() {
-	return sh(script: 'git describe --abbrev=0 --tags --match v[0-9][0-9][0-9][0-9]*', returnStdout: true).strip()
+	return sh(script: 'git describe --abbrev=0 --tags --match v[0-9][0-9][0-9][0-9]*', returnStdout: true).trim()
 }
 
 def getSWTVersions() { // must be called from the repository root
@@ -86,10 +86,10 @@ def Set NATIVES_CHANGED = []
 
 pipeline {
 	options {
-		skipDefaultCheckout() // Specialiced checkout is performed below
+		skipDefaultCheckout() // Specialized checkout is performed below
 		timestamps()
 		timeout(time: 180, unit: 'MINUTES')
-		buildDiscarder(logRotator(numToKeepStr:'5'))
+		buildDiscarder(logRotator(numToKeepStr: 'master'.equals(env.BRANCH_NAME) ? '20' : '5', artifactNumToKeepStr: 'master'.equals(env.BRANCH_NAME) ? '3' : '1' ))
 		disableConcurrentBuilds(abortPrevious: true)
 	}
 	agent {
@@ -116,32 +116,31 @@ pipeline {
 					script {
 						def authorMail = sh(script: 'git log -1 --pretty=format:"%ce" HEAD', returnStdout: true)
 						echo 'HEAD commit author: ' + authorMail
-						if ('[email protected]'.equals(authorMail) && !params.forceNativeBuilds) {
-							// Prevent endless build-loops due to self triggering because of a previous automated build of SWT-natives and the associated updates.
+						def buildBotMail = '[email protected]'
+						if (buildBotMail.equals(authorMail) && !params.forceNativeBuilds) {
+							// Prevent endless build-loops due to self triggering because of a previous automated build of natives and the associated updates.
 							currentBuild.result = 'ABORTED'
 							error('Abort build only triggered by automated SWT-natives update.')
 						}
-					}
-					sh '''
+						sh """
+						java -version
 						git version
 						git lfs version
+						git config --global user.email '${buildBotMail}'
+						git config --global user.name 'Eclipse Platform Bot'
 						git config --unset core.hooksPath # Jenkins disables hooks by default as security feature, but we need the hooks for LFS
 						git lfs update # Install Git LFS hooks in repository, which has been skipped due to the initially nulled hookspath
 						git lfs pull
 						git fetch --all --tags --quiet
 						git remote set-url --push origin [email protected]:eclipse-platform/eclipse.platform.swt.git
-					'''
+						"""
+					}
 				}
 			}
 		}
 		stage('Check if SWT-binaries build is needed') {
 			steps {
 				dir('eclipse.platform.swt') {
-					sh'''
-						java -version
-						git config --global user.email '[email protected]'
-						git config --global user.name 'Eclipse Releng Bot'
-					'''
 					script {
 						def allWS = ['cocoa', 'gtk', 'win32']
 						def libraryFilePattern = [ 'cocoa' : 'libswt-*.jnilib', 'gtk' : 'libswt-*.so', 'win32' : 'swt-*.dll' ]
@@ -157,9 +156,9 @@ pipeline {
 							'''
 							def sourceFoldersProps = readProperties(file: 'nativeSourceFolders.properties')
 							def sources = sourceFoldersProps.collectEntries{ k, src -> [ k, src.split(',').collect{ f -> '\'' + f + '\''}.join(' ') ] }
-							for(ws in allWS) {
+							for (ws in allWS) {
 								def diff = sh(script: "git diff HEAD ${swtTag} ${sources.src_common} ${sources['src_' + ws]}", returnStdout: true)
-								if (!diff.strip().isEmpty()) {
+								if (!diff.trim().isEmpty()) {
 									NATIVES_CHANGED += ws
 								}
 							}
@@ -175,7 +174,7 @@ pipeline {
 									sed -i -e "s/rev=${versions.rev}/rev=${versions.new_rev}/g" \
 										'bundles/org.eclipse.swt/Eclipse SWT/common/library/make_common.mak'
 								"""
-							for(ws in allWS) {
+							for (ws in allWS) {
 								if (NATIVES_CHANGED.contains(ws)) {
 								sh """
 									# Delete native binaries to be replaced by subsequent binaries build
@@ -239,7 +238,7 @@ pipeline {
 							script {
 								def (ws, os, arch) = env.PLATFORM.split('\\.')
 								dir("jdk-download-${ws}.${arch}") {
-									// Fetch the JDK, which provides the C header-files and shared native libraries, against which the natives are build.
+									// Fetch the JDK, which provides the C header files and shared native libraries against which the native code is built.
 									sh "curl ${getNativeJdkUrl(os, arch)} | tar -xzf - include/ lib/"
 									stash name:"jdk.resources.${ws}.${arch}", includes: "include/,lib/"
 									deleteDir()
@@ -350,8 +349,8 @@ pipeline {
 					dir('eclipse.platform.swt') {
 						sh '''
 							mvn clean verify \
-								--batch-mode --threads 1C -V -U -e -DforkCount=0 \
-								-Papi-check \
+								--batch-mode --threads 1C -V -U -e \
+								-Pbree-libs -Papi-check -Pjavadoc \
 								-Dcompare-version-with-baselines.skip=false \
 								-Dorg.eclipse.swt.tests.junit.disable.test_isLocal=true \
 								-Dmaven.test.failure.ignore=true -Dmaven.test.error.ignore=true
@@ -382,11 +381,11 @@ pipeline {
 				sshagent(['github-bot-ssh']) {
 					dir('eclipse.platform.swt') {
 						sh """
-							# Check for the master-branch as late as possible to have as much of the same behaviour as possible
+							# Check for the master-branch as late as possible to have as much of the same behavior as possible
 							if [[ '${BRANCH_NAME}' == master ]] || [[ '${BRANCH_NAME}' =~ R[0-9]+_[0-9]+(_[0-9]+)?_maintenance ]]; then
 								if [[ ${params.skipCommit} != true ]]; then
 
-									# Don't rebase and just fail in case another commit has been pushed to the master/maintanance branch in the meantime
+									# Don't rebase and just fail in case another commit has been pushed to the master/maintenance branch in the meantime
 									git push origin HEAD:refs/heads/${BRANCH_NAME}
 									git push origin refs/tags/${getLatestGitTag()}
 

binaries/.settings/org.eclipse.jdt.core.prefs

@@ -46,7 +46,7 @@ org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=warning
 org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=warning
 org.eclipse.jdt.core.compiler.problem.invalidJavadoc=error
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
-org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=disabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=enabled
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=disabled
 org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=public
 org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore