Commit 82125827 authored by Emily's avatar Emily
Browse files

Revert "gg-jj: init at 0.20.0"

Several confused users have reported that this
package does not work at all, and per the discussion in
<https://github.com/NixOS/nixpkgs/pull/345905>, the contributor who
added it does not appear to have tested its basic functionality at
all before opening a pull request.

That’s not acceptable; we rely on package maintainers to be able
to assess the basic functionality of software and ensuring that a
program can even start up before opening a pull request to package it
is basic due diligence. It’s harmful to have a package that doesn’t
work and lacks a maintainer who can assess whether it works or not,
so we should remove it until both of those can be addressed.

This reverts commit 5eb022fd.
parent 9ee9cac8
Loading
Loading
Loading
Loading

pkgs/by-name/gg/gg-jj/package.nix

deleted100644 → 0
+0 −80
Original line number Diff line number Diff line
{
  rustPlatform,
  callPackage,
  pkg-config,
  lib,
  fetchFromGitHub,
  libayatana-appindicator,
  openssl,
  webkitgtk_4_1,
}:

rustPlatform.buildRustPackage rec {
  pname = "gg-jj";
  version = "0.20.0";

  src = fetchFromGitHub {
    owner = "gulbanana";
    repo = "gg";
    rev = "refs/tags/v${version}";
    hash = "sha256-xOi/AUlH0FeenTXz3hsDYixCEl+yr22PGy6Ow4TKxY0=";
  };

  sourceRoot = "${src.name}/src-tauri";

  webui = callPackage ./webui.nix {
    inherit
      src
      pname
      version
      meta
      ;
  };

  env = {
    OPENSSL_NO_VENDOR = 1;
  };

  buildInputs = [
    webkitgtk_4_1
    openssl
  ];

  nativeBuildInputs = [
    pkg-config
  ];

  cargoHash = "sha256-u3SkRA7327ZwqEnB+Xq2JDbI0k5HfeKzV17dvQ8B6xk=";

  postPatch = ''
    buildRoot=$(pwd)
    pushd $cargoDepsCopy/libappindicator-sys
    oldHash=$(sha256sum src/lib.rs | cut -d " " -f 1)
    substituteInPlace src/lib.rs \
      --replace-fail "libayatana-appindicator3.so.1" "${libayatana-appindicator}/lib/libayatana-appindicator3.so.1"
    substituteInPlace .cargo-checksum.json \
      --replace-fail $oldHash $(sha256sum src/lib.rs | cut -d " " -f 1)
    popd

    pushd $cargoDepsCopy/jj-cli
    oldHash=$(sha256sum build.rs | cut -d " " -f 1)
    substituteInPlace build.rs \
      --replace-fail 'let path = std::env::var("CARGO_MANIFEST_DIR").unwrap();' "let path = \"$buildRoot\";"
    substituteInPlace .cargo-checksum.json \
      --replace-fail $oldHash $(sha256sum build.rs | cut -d " " -f 1)
    popd

    substituteInPlace ./tauri.conf.json \
      --replace-fail '"frontendDist": "../dist"' '"frontendDist": "${webui}"' \
      --replace-fail '"beforeBuildCommand": "npm run build"' '"beforeBuildCommand": ""'
  '';

  meta = {
    homepage = "https://github.com/gulbanana/gg";
    changelog = "https://github.com/gulbanana/gg/releases/tag/v${version}";
    description = "GUI for jj users";
    maintainers = with lib.maintainers; [ bot-wxt1221 ];
    mainProgram = "gg";
    license = lib.licenses.apsl20;
  };
}

pkgs/by-name/gg/gg-jj/webui.nix

deleted100644 → 0
+0 −31
Original line number Diff line number Diff line
{
  version,
  src,
  pname,
  pnpm,
  meta,
  buildNpmPackage,
}:

buildNpmPackage {
  inherit version src meta;
  pname = "${pname}-webui";

  npmDepsHash = "sha256-oHBFuX65D/FgnGa03jjpIKAdH8Q4c2NrpD64bhfe720=";

  buildPhase = ''
    runHook preBuild

    node --max_old_space_size=1024000 ./node_modules/vite/bin/vite.js build

    runHook postBuild
  '';

  installPhase = ''
    runHook preInstall

    cp -r dist $out

    runHook postInstall
  '';
}