Skip to content
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

gradlew: no "--source 11" #13404

Merged
merged 1 commit into from
May 24, 2024
Merged

gradlew: no "--source 11" #13404

merged 1 commit into from
May 24, 2024

Conversation

dsmiley
Copy link
Contributor

@dsmiley dsmiley commented May 23, 2024

  • avoid WrapperDownloader if have the JAR
  • don't specify --source More specific than needed, and some JDK/configs may complain about an incompatibility with --release.

From apache/solr#2419

* avoid WrapperDownloader if have the JAR
* don't specify --source
More specific than needed, and some JDK/configs may complain about an incompatibility with --release.

From apache/solr#2419
@dsmiley dsmiley requested a review from uschindler May 23, 2024 01:46
@dweiss
Copy link
Contributor

dweiss commented May 23, 2024

LGTM, although "some JDK/configs may complain about an incompatibility with --release" is intriguing - what are these JDK distributions, exactly?

@dsmiley
Copy link
Contributor Author

dsmiley commented May 23, 2024

Admittedly the --source issue was specific to a custom JVM variant where I work. Maybe --release alone would work; I didn't check. Any way, I'd rather remove needless specificity.

@dsmiley dsmiley merged commit 90e07f6 into apache:main May 24, 2024
3 checks passed
@dsmiley dsmiley deleted the gradlew_noSource11 branch May 24, 2024 22:20
dsmiley added a commit that referenced this pull request May 24, 2024
* avoid WrapperDownloader if have the JAR
* don't specify --source
More specific than needed, and some JDK/configs may complain about an incompatibility with --release.

From apache/solr#2419
@dsmiley dsmiley added this to the 9.11.0 milestone May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants