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
kitty: 0.13.3 -> 0.14.0 #62095
kitty: 0.13.3 -> 0.14.0 #62095
Conversation
@@ -67,18 +67,6 @@ buildPythonApplication rec { | |||
libstartup_notification = "${libstartup_notification}/lib/libstartup-notification-1.so"; | |||
}) | |||
] ++ stdenv.lib.optionals stdenv.isDarwin [ | |||
(fetchpatch { |
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.
See #61369
url = https://github.com/kovidgoyal/kitty/commit/af2c9a49b1ad31e94242295d88598591623fbf11.patch; | ||
sha256 = "0k3dmgbvmh66j8k3h8dw6la6ma6f20fng6jjypy982kxvracsnl5"; | ||
}) | ||
./macos-10.11.patch |
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 change was merged in kovidgoyal/kitty#1430
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 change was not merged. The pull request was rejected because kovidgoyal is not interested in providing support for more than 2-3 macOS versions. But I don't think anyone complained about this being broken on macOS 10.11 (I could be wrong though).
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.
Yes, I thought this was strange too. Anyway, we did an upgrade to 10.12 SDK, so the patch was to be removed.
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.
Works on darwin
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.
Works for my on NixOS, nix-review passes
Motivation for this change
Upstream update
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)