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

Remove hack for IntelliJ IDEs, replace with caveat #9943

Merged
merged 1 commit into from
Mar 8, 2015
Merged

Remove hack for IntelliJ IDEs, replace with caveat #9943

merged 1 commit into from
Mar 8, 2015

Conversation

radeksimko
Copy link
Contributor

Related issues:

TL;DR reasons:

  • hack was causing issues during upgrades
  • vendor officially doesn't support Java >6 anyway

Details:

@w7tek in #9572 (comment)

Unfortunately, according to the conversation along the same issue, it appears the idea.properties is picked up from a different location when using the CE than when using Ultimate.

True. I've been just playing around and I can confirm that this hack

echo "JVMVersion=1.6+" > ~/Library/Preferences/IntelliJIdea14/idea.properties

only works for IntelliJ IDEA 14 and AppCode, but doesn't for any other IDE (not even IntelliJ IDEA 14 CE). Maybe they have add this functionality into IntelliJ core and AppCode has already adopted it, so other IDEs may do in the future as well?

I wonder what's the motivation of JetBrains for being stuck 4 years behind (that's when Java 7 was released)... 😕

 - hack was causing issues during upgrades
 - vendor officially doesn't support Java >6 anyway
@fanquake fanquake added the cask label Mar 8, 2015
tapeinosyne pushed a commit that referenced this pull request Mar 8, 2015
Remove hack for IntelliJ IDEs, replace with caveat
@tapeinosyne tapeinosyne merged commit 4fbf0d8 into Homebrew:master Mar 8, 2015
@radeksimko radeksimko deleted the intellij-ides-unhack branch March 8, 2015 15:22
@ctwise
Copy link

ctwise commented Mar 8, 2015

"I wonder what's the motivation of JetBrains for being stuck 4 years behind (that's when Java 7 was released)..."

It's due to many GUI defects still outstanding in Java 7, 8 and in previews of 9 as well. Oracle doesn't seem to care about AWT/SWT anymore and have moved on to JavaFX. As a result outstanding GUI issues in OS/X aren't getting addressed (and obviously haven't been for a very long time). JetBrains is apparently trying to create a patched JVM that deals with at least the worst problems and is planning on bundling it with the IDEs. You can obviously use Java 7+ to run the IDEs but JetBrains gets the same defect reports over and over again when running on those.

@msheakoski
Copy link

This patch should also be applied to 0xdbe-eap. When using it with JVM>6, the font rendering on OSX is horrible.

@sebroeder
Copy link
Contributor

We should remove the plist hack from the JetBrains formulas in the homebrew-versions repository, too. Homebrew/homebrew-cask-versions#847 could help with this for the eap versions, if accepted by the maintainers.

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.

7 participants