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
Update jupyter and enable a few tests #30762
Conversation
}; | ||
|
||
LC_ALL = "en_US.UTF-8"; | ||
LC_ALL = "en_US.utf8"; |
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.
Is en_US.utf8
a valid identifier?
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.
$ locale -a
C
en_GB.utf8
en_US.utf8
POSIX
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.
Interesting on ubuntu/archlinux always the first notation is used in /etc/locale.gen
. Python seems to accept both variants:
$ LC_ALL=en_US.utf8 python -c 'import locale; print(locale.getdefaultlocale())'
('en_US', 'UTF-8')
$ LC_ALL=en_US.UTF-8 python -c 'import locale; print(locale.getdefaultlocale())'
('en_US', 'UTF-8')
Thanks! |
@Fuuzetsu I also just ran into build failures for Jupyter on OS X. I suspect it's caused by this PR. Is there a dedicated issue for that? Is it worth opening one? |
I haven't opened one. I have set doCheck to false in own checkout at the time to get around that. We should create an issue and/or just disable to test on OSX in the expression. |
cc @FRidh
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)