-
Notifications
You must be signed in to change notification settings - Fork 24
Update XCode to 26.2, MacOS SDK to 26.2, iOS SDK to 26.2 #822
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| distributionBase=GRADLE_USER_HOME | ||
| distributionPath=wrapper/dists | ||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.9-all.zip | ||
| distributionUrl=https\://services.gradle.org/distributions/gradle-9.1.0-all.zip | ||
| zipStoreBase=GRADLE_USER_HOME | ||
| zipStorePath=wrapper/dists |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,6 @@ buildscript { | |
| mavenCentral() | ||
| } | ||
| dependencies { | ||
| classpath 'io.spring.gradle:dependency-management-plugin:1.1.7' | ||
| classpath 'org.springframework.boot:spring-boot-gradle-plugin:3.5.5' | ||
| } | ||
| } | ||
|
|
@@ -29,25 +28,17 @@ ext { | |
| } | ||
|
|
||
| configurations { | ||
| // A configuration meant for consumers that need the API of this component | ||
| implementation { | ||
| canBeResolved = true | ||
| } | ||
| all*.exclude group: 'commons-logging', module: 'commons-logging' | ||
| } | ||
|
|
||
| repositories { | ||
| mavenCentral() | ||
| } | ||
|
|
||
| dependencyManagement { | ||
| imports { | ||
| mavenBom 'com.google.cloud:spring-cloud-gcp-dependencies:7.4.0' | ||
| mavenBom 'org.junit:junit-bom:5.13.4' | ||
| } | ||
| } | ||
|
|
||
| dependencies { | ||
| implementation(platform("com.google.cloud:spring-cloud-gcp-dependencies:7.4.0")) | ||
| testImplementation(platform("org.junit:junit-bom:5.13.4")) | ||
|
|
||
| implementation('org.springframework.boot:spring-boot-starter-web') { | ||
| exclude module: 'spring-boot-starter-tomcat' | ||
| } | ||
|
|
@@ -175,9 +166,9 @@ if (project.hasProperty('publishToGar')) { | |
| publications { | ||
| extender(MavenPublication) { | ||
| artifact bootJar | ||
| groupId 'com.defold.extender' | ||
| artifactId 'server' | ||
| version project.ext.extenderVersion | ||
| groupId = 'com.defold.extender' | ||
| artifactId = 'server' | ||
| version = project.ext.extenderVersion | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. syntax changes for new gradle |
||
| } | ||
| } | ||
| } | ||
|
|
@@ -224,4 +215,4 @@ jacocoTestReport { | |
| xml.required.set(true) | ||
| html.required.set(true) | ||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,3 +5,10 @@ XCODE_16_VERSION=16.2 | |
| XCODE_16_CLANG_VERSION=16.0.0 | ||
| MACOS_15_VERSION=15.2 | ||
| IOS_18_VERSION=18.2 | ||
| SWIFT_6_2_VERSION=6.2 | ||
| IOS_26_VERSION_MIN=15.0 | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. min version changed, so we need it in a separate variable now |
||
| MACOS_26_VERSION_MIN=11.5 | ||
| XCODE_26_VERSION=26.2 | ||
| XCODE_26_CLANG_VERSION=17.0.0 | ||
| MACOS_26_VERSION=26.2 | ||
| IOS_26_VERSION=26.2 | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -20,9 +20,11 @@ jar { | |
|
|
||
|
|
||
| configurations { | ||
| // A configuration meant for consumers that need the API of this component | ||
| implementation { | ||
| // Resolvable configuration for shading contents into the fat jar | ||
| implementationResolved { | ||
| canBeResolved = true | ||
| canBeConsumed = false | ||
| extendsFrom implementation | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. here we iterate over artifacts, so it's reimplemented gradle 9 way |
||
| } | ||
| all*.exclude group: 'com.android.tools', module: 'repository' | ||
| all*.exclude group: 'com.android.tools', module: 'sdklib' | ||
|
|
@@ -35,13 +37,13 @@ task mainJar(type: Jar) { | |
| attributes 'Main-Class': 'com.defold.manifestmergetool.ManifestMergeTool' | ||
| } | ||
| from { | ||
| configurations.implementation.collect { it.isDirectory() ? it : zipTree(it) } | ||
| configurations.implementationResolved.collect { it.isDirectory() ? it : zipTree(it) } | ||
| } { | ||
| exclude "META-INF/*.SF" | ||
| exclude "META-INF/*.DSA" | ||
| exclude "META-INF/*.RSA" | ||
| } | ||
| duplicatesStrategy DuplicatesStrategy.WARN | ||
| duplicatesStrategy = DuplicatesStrategy.WARN | ||
| with jar | ||
| doLast { | ||
| copy { | ||
|
|
@@ -60,17 +62,17 @@ if (project.hasProperty('publishToGar')) { | |
| publications { | ||
| manifestMergeTool(MavenPublication) { | ||
| artifact mainJar | ||
| groupId 'com.defold.extender' | ||
| artifactId 'manifestmergetool' | ||
| version project.ext.manifestMergetoolVersion | ||
| groupId = 'com.defold.extender' | ||
| artifactId = 'manifestmergetool' | ||
| version = project.ext.manifestMergetoolVersion | ||
| } | ||
| } | ||
| } | ||
| } | ||
|
|
||
| repositories { | ||
| mavenCentral() | ||
| maven { url "https://maven.google.com" } | ||
| maven { url = uri("https://maven.google.com") } | ||
| } | ||
|
|
||
| dependencies { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't iterate over implementation artifacts directly, so we don't need it here (it's not supported in new Gradle, so it's easier to just remove it)