Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

classifier is deprecated in favor of archiveClassifier - eg: https://… #3817

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

s50600822
Copy link
Contributor

@s50600822 s50600822 commented Oct 19, 2023

This should fix the issue in update-gradle-wrapper: https://github.com/Netflix/conductor/actions/runs/6870985189/job/18686956908

  Error: ❌ FAILURE: Build failed with an exception.
  
  * Where:
  Build file '/home/runner/work/conductor/conductor/server/build.gradle' line: 62
  
  * What went wrong:
  A problem occurred evaluating project ':conductor-server'.
  > Could not set unknown property 'classifier' for task ':conductor-server:bootJar' of type org.springframework.boot.gradle.tasks.bundling.BootJar.

Pull Request type

  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)

Changes in this PR

One of the necessary updates toward Gradle 8
eg: https://github.com/spring-projects/spring-boot/pull/29611/files
Alternatives considered

See also:

  1. deprecated since 5.1 - spring-boot-gradle-plugin docs use deprecated Gradle API spring-projects/spring-boot#29610
  2. will be removed in 8 - https://docs.gradle.org/current/userguide/upgrading_version_7.html where classifier is listed to be removed (replaced by to archiveClassifier)
The deprecated appendix, archiveName, archivePath, baseName, classifier, destinationDir, extension and version properties of the AbstractArchiveTask task type have been removed. Use the archiveAppendix, archiveFileName , archiveFile, archiveBaseName, archiveClassifier, destinationDirectory, archiveExtension and archiveVersion properties instead.
  1. from gradle src: https://github.com/gradle/gradle/pull/22107/files#diff-1c2c4263cffdda364f17c6f3dee6540bbe13d53a38a2814bcb2723fa7c1b3821

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant