Unverified Commit c28f3f4b authored by Nick Cao's avatar Nick Cao Committed by GitHub
Browse files

Merge pull request #211135 from malob/fix-kitty-x86_64-darwin

kitty: unbreak on `x86_64-darwin` and fix build failure on all systems due to Bash update in `nixpkgs`
parents c7c5251a 3b1880dd
Loading
Loading
Loading
Loading
+12 −10
Original line number Diff line number Diff line
{ lib, stdenv, fetchFromGitHub, python3Packages, libunistring
{ lib, stdenv, fetchFromGitHub, fetchpatch, python3Packages, libunistring
, harfbuzz, fontconfig, pkg-config, ncurses, imagemagick
, libstartup_notification, libGL, libX11, libXrandr, libXinerama, libXcursor
, libxkbcommon, libXi, libXext, wayland-protocols, wayland
@@ -7,14 +7,11 @@
, openssl
, installShellFiles
, dbus
, Libsystem
, Cocoa
, CoreGraphics
, Foundation
, IOKit
, Kernel
, UniformTypeIdentifiers
, UserNotifications
, OpenGL
, libcanberra
, libicns
, libpng
@@ -47,16 +44,14 @@ buildPythonApplication rec {
    openssl.dev
  ] ++ lib.optionals stdenv.isDarwin [
    Cocoa
    CoreGraphics
    Foundation
    IOKit
    Kernel
    OpenGL
    UniformTypeIdentifiers
    UserNotifications
    libpng
    python3
    zlib
  ] ++ lib.optionals (stdenv.isDarwin && stdenv.isx86_64) [
    Libsystem
  ] ++ lib.optionals stdenv.isLinux [
    fontconfig libunistring libcanberra libX11
    libXrandr libXinerama libXcursor libxkbcommon libXi libXext
@@ -80,6 +75,13 @@ buildPythonApplication rec {
  outputs = [ "out" "terminfo" "shell_integration" ];

  patches = [
    # Fix clone-in-kitty not working on bash >= 5.2
    # TODO: Removed on kitty release > 0.26.5
    (fetchpatch {
      url = "https://github.com/kovidgoyal/kitty/commit/51bba9110e9920afbefeb981e43d0c1728051b5e.patch";
      sha256 = "sha256-1aSU4aU6j1/om0LsceGfhH1Hdzp+pPaNeWAi7U6VcP4=";
    })

    # Gets `test_ssh_env_vars` to pass when `bzip2` is in the output of `env`.
    ./fix-test_ssh_env_vars.patch

@@ -110,6 +112,7 @@ buildPythonApplication rec {
    '';
  in ''
    runHook preBuild
    ${ lib.optionalString (stdenv.isDarwin && stdenv.isx86_64) "export MACOSX_DEPLOYMENT_TARGET=11" }
    ${if stdenv.isDarwin then ''
      ${python.interpreter} setup.py build ${darwinOptions}
      make docs
@@ -219,7 +222,6 @@ buildPythonApplication rec {
    license = licenses.gpl3Only;
    changelog = "https://sw.kovidgoyal.net/kitty/changelog/";
    platforms = platforms.darwin ++ platforms.linux;
    broken = (stdenv.isDarwin && stdenv.isx86_64);
    maintainers = with maintainers; [ tex rvolosatovs Luflosi adamcstephens ];
  };
}
+7 −1
Original line number Diff line number Diff line
@@ -2368,7 +2368,13 @@ with pkgs;
  kitty = darwin.apple_sdk_11_0.callPackage ../applications/terminal-emulators/kitty {
    harfbuzz = harfbuzz.override { withCoreText = stdenv.isDarwin; };
    inherit (darwin.apple_sdk_11_0.frameworks) Cocoa CoreGraphics Foundation IOKit Kernel UniformTypeIdentifiers OpenGL UserNotifications;
    inherit (darwin.apple_sdk_11_0) Libsystem;
    inherit (darwin.apple_sdk_11_0.frameworks)
      Cocoa
      Kernel
      UniformTypeIdentifiers
      UserNotifications
    ;
  };
  kitty-themes  = callPackage ../misc/kitty-themes {};