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
Fontconfig for Android Studio #21462
Conversation
@phunehehe, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bjornfor, @taku0 and @edwtjo to be potential reviewers. |
6c08224
to
26cb613
Compare
# jdk = pkgs.oraclejdk8; | ||
# }; | ||
# }; | ||
# } |
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.
Why delete this comment?
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.
Because it's no longer true. This version of android studio bundles its own JDK.
26cb613
to
a5cb303
Compare
Android Studio now bundles its own JDK ¯\_(ツ)_/¯
a5cb303
to
4b3f79b
Compare
Is this still good to merge? |
It is. The only concern I had was about fontconfig. If nobody chips in I guess it's not really a problem. |
If someone is impacted by this, change, hopefully they'll file an issue :) |
Motivation for this change
With the fonts.conf by Ubuntu (I use Nix instead of Apt on Ubuntu), Android Studio crashes during startup because it fails to do some font stuff. With a config file built with
makeFontsConf
(even an empty one!?), it works fine. The workaround is found via docker-library/openjdk#73.On the one side, it could be argued that I should fix my fonts.conf, but then it means the package isn't really usable out of NixOS.
On the other side, overriding FONTCONFIG_FILE probably means the package can't use all the installed fonts, but then I also see other packages doing the same.
Thoughts?
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)