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

Ktlint jdk dependcy has to be 11 #72336

Closed
1 task done
pagsantos opened this issue Mar 2, 2021 · 7 comments
Closed
1 task done

Ktlint jdk dependcy has to be 11 #72336

pagsantos opened this issue Mar 2, 2021 · 7 comments
Labels
bug Reproducible Homebrew/homebrew-core bug outdated PR was locked due to age

Comments

@pagsantos
Copy link

brew gist-logs <formula> link OR brew config AND brew doctor output

brew config
HOMEBREW_VERSION: 3.0.4
ORIGIN: https://github.com/Homebrew/brew
HEAD: ee52b19
Last commit: 2 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 3fc5699
Core tap last commit: 44 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 12.0 build 1200
Git: 2.30.1 => /opt/homebrew/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.2.2-arm64
CLT: 12.4.0.0.1.1610135815
Xcode: N/A
Rosetta 2: false

  • The brew doctor above contains no "Warning" lines.

What were you trying to do (and why)?

Trying to avoid that formulae from ktlint install openjdk15 when in fact only openjdk11 is supported

What happened (include all command output)?

brew install ktlint
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> Updated Formulae
Updated 1 formula.

==> Downloading https://homebrew.bintray.com/bottles/openjdk-15.0.1.arm64_big_sur.bottle.1.t
Already downloaded: /Users/pedrosantos/Library/Caches/Homebrew/downloads/71a21626f6690a21ee1764092d91146ae286323e9a02a44d9d3c76aa0c0eb27e--openjdk-15.0.1.arm64_big_sur.bottle.1.tar.gz
==> Downloading https://github.com/pinterest/ktlint/releases/download/0.40.0/ktlint
Already downloaded: /Users/pedrosantos/Library/Caches/Homebrew/downloads/8c1626a47dae0aac0ca01fe923390e3580c075d7add9880c40122c0c820066d9--ktlint
==> Installing dependencies for ktlint: openjdk
==> Installing ktlint dependency: openjdk
==> Pouring openjdk-15.0.1.arm64_big_sur.bottle.1.tar.gz
Error: Failed applying an ad-hoc signature to /opt/homebrew/Cellar/openjdk/15.0.1/libexec/openjdk.jdk/Contents/MacOS/libjli.dylib:
/opt/homebrew/Cellar/openjdk/15.0.1/libexec/openjdk.jdk/Contents/MacOS/libjli.dylib: replacing existing signature
/opt/homebrew/Cellar/openjdk/15.0.1/libexec/openjdk.jdk/Contents/MacOS/libjli.dylib: resource fork, Finder information, or similar detritus not allowed

What did you expect to happen?

that install ktlint and openjdk11

Step-by-step reproduction instructions (by running brew commands)

brew install ktlint

@pagsantos pagsantos added the bug Reproducible Homebrew/homebrew-core bug label Mar 2, 2021
@SMillerDev
Copy link
Member

Is it actually broken? Or just not in the upstream support list?

@carlocab
Copy link
Member

carlocab commented Mar 2, 2021

It seems like your problem here is that you're not able to install openjdk, not that ktlint has the wrong dependency. It may well be that it does, but we should probably have proper verification of that.

@pagsantos
Copy link
Author

@carlocab @SMillerDev so according to pinterest/ktlint#1063 the issue is that if you use 15 it's an issue so we need to make the formulae to depend only on JDK11 I opening a PR to fix it

@carlocab
Copy link
Member

carlocab commented Mar 2, 2021

Please install openjdk properly first and then verify that ktlint is not working.

@pagsantos
Copy link
Author

@carlocab according to this pinterest/ktlint#1063 it is runned and tested in my machine and it's an issue

@carlocab
Copy link
Member

carlocab commented Mar 2, 2021

We also ran and tested it in #66267, and it wasn't an issue. So we need to figure out what exactly is going wrong here first before we discuss switching ktlint to openjdk@11.

If it is the case that ktlint needs to be switching to openjdk@11, we also need to update the test to make sure it catches the correct error. That's why you need to install openjdk first so we can figure out what that error is.

@pagsantos
Copy link
Author

according to @shashachu ktlint will support jdk15 in the next 2 weeks so this shouldn't be a problem by then

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Apr 3, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Apr 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Reproducible Homebrew/homebrew-core bug outdated PR was locked due to age
Projects
None yet
Development

No branches or pull requests

4 participants