-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
python312Packages.kaleido: init at 0.2.1 #339136
python312Packages.kaleido: init at 0.2.1 #339136
Conversation
"aarch64-linux" | ||
"aaarch64-darwin" | ||
]; | ||
sourceProvenance = with lib.sourceTypes; [ binaryNativeCode ]; # Trust me, I'm not happy. But after literal hours of trying to reverse-engineer their build system and getting nowhere, I'll use the stupid binaries >:( |
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.
This is the third attempt, as far as I know, but no one has been able to build it from the source.😢
4ce530b
to
f30a433
Compare
6135aa7
to
8cf971e
Compare
9add00d
to
a411299
Compare
Also please use the full attrname in commits so that borgo knows what to build (i.e. |
a411299
to
78d34e5
Compare
Just noticed there weren't any maintainers for plotly, so this PR also has me adopting it. |
please use python312Packages, not python3Packages, in the commit message. |
@ofborg build python312Packages.kaleido python311Packages.kaleido |
Right, hence the note here to explicitly specify the version: nixpkgs/doc/languages-frameworks/python.section.md Lines 2020 to 2023 in 105030f
|
Okay, that's not particularly clear. I'll open a PR to rephrase that. |
31417ef
to
6c98676
Compare
Tests fail on darwin with the following log. I suspect
|
@natsukium in the meantime, I've marked the package as broken on darwin. Does that address that concern for the time being? |
I think it looks good. The darwin fix can be done in another PR. |
cc @GaetanLepage, @Sigmanificient if you're interested. |
Result of 10 packages marked as broken and skipped:
2 packages failed to build:
120 packages built:
|
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-24.05
git worktree add -d .worktree/backport-339136-to-release-24.05 origin/release-24.05
cd .worktree/backport-339136-to-release-24.05
git switch --create backport-339136-to-release-24.05
git cherry-pick -x 25ad2d11ec45cbc6c908b78de63ad49aba401f9f 534c90d223c32bf88b7aba3399fa69936c25fbf6 46c1ce890a749f6c17b07ce87237e7ebccf5af42 6c9867637668f00d0e7f8dac0f3312c525500c9d 7edc3b600d3e35e422bed873b9956797aeec801b |
Manually backported: #339839 |
Here's a weirdness: it builds fine on Darwin when run as a stand-alone build. The error you saw (essentially "we don't know how to find the system UI configuration") may be due to the building machine being headless. Recommendation: disable the failing test. It passes when run on a desktop/laptop. |
It’s just the Nix sandbox forbidding access to OS stuff. Happens a lot. |
Unfortunately I don't have access to a darwin machine to debug this. I suspect it might be |
I think @emilazy has identified the issue. I'm attempting to verify it on a Darwin machine so we can disable the appropriate tests. |
Yea, I was just noting that |
I’m not sure what you mean; |
Loading libGL is done impurely in NixOS, because nvidia. |
I wouldn’t quite call that an impurity. Anyway, that’s not too relevant to macOS, where every system framework is built inside Nix against purely‐packaged stubs listing the available symbols, and which is then linked against the OS‐level framework code at runtime by the dynamic loader. (So in a sense, every system library is like |
|
I can't get the build to break for me on Darwin, even with sandboxing enabled. Should I file a PR that conditionally disables the passthrough test on Darwin and removes the |
Go right ahead! |
Description of changes
I'm sorry, I literally couldn't figure out how to do a source build. If someone would like to figure it out for themselves, uh, good luck!
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.