-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
keybase: 2.1.0 -> 2.5.0 #44698
keybase: 2.1.0 -> 2.5.0 #44698
Conversation
@GrahamcOfBorg build keybase |
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
Darwin failure is a regression, builds with current version. |
I can take a look, but I don't have a Mac and am not familiar with their toolchain/libraries/etc. We might want to find a related issue on keybase repo or file one, unless other folks have a solid idea. (Lots of SO hits, but I don't know what we would need to do for mitigation, etc) |
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
@colemickens That Darwin failure is due to missing |
Alright, I've attempted the change. Builds under Linux. I don't think I'm trusted, but I'm going to try anyway... @GrahamcOfBorg build keybase btw: thank you for the explanation and pointers @jonafato |
@GrahamcOfBorg build keybase |
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
Hm, I'm not sure why it doesn't work. Any more suggestions? |
|
You might need to inherit |
@GrahamcOfBorg build keybase |
@GrahamcOfBorg build keybase (not everybody can start builds) |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: keybase Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: keybase Partial log (click to expand)
|
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: keybase Partial log (click to expand)
|
@@ -1,8 +1,8 @@ | |||
{ stdenv, buildGoPackage, fetchFromGitHub }: | |||
{ stdenv, lib, buildGoPackage, fetchFromGitHub, darwin.apple_sdk.frameworks.AVFoundation ? null }: |
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.
Oh but this is not even valid syntax. You should've noticed this locally already. Look at the file @xeji linked and replicate that in all-packages.nix
, then this line here should just be AVFoundation ? null
Sorry about the bad syntax. I'd cloned another copy of the repo and forgot I wasn't building/evaling from it. I've more closely followed the recommended pattern for pulling in the required library on Darwin/osx. Can someone spot check it and kick off the test again? Thanks. |
@GrahamcOfBorg build keybase |
Failure on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
We're slowly getting there. You also need to inherit |
@xeji Thanks. Want to kick it again? I added AudioToolbox, tried to follow the same pattern. edit: I need to edit top-level too, one second. |
@xeji updated, ptal. Thanks for the guidance and patience. |
@GrahamcOfBorg build keybase |
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
CoreMedia as well ... |
This appears to be a list of the frameworks we need to include: https://github.com/keybase/client/blob/68bb8c893c5214040d86ea36f2f86fbb7fac8d39/go/chat/attachments/preview_darwin.go#L7. |
Done. (I would've thought |
@GrahamcOfBorg build keybase |
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
@jonafato thanks! Hopefully including everything on that list will fix it. |
I'm concerned that list isn't exhaustive? It doesn't include Also, another question, it looks like https://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/darwin/apple-sdk/frameworks.nix#L77 For now I'm going to just add everything from the linked file (except |
Added a comment to Hopefully it builds this time? |
Let's see |
Success on x86_64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: keybase Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: keybase Partial log (click to expand)
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)