Commit 0dd2238a authored by Dmitry Ivankov's avatar Dmitry Ivankov Committed by Weijia Wang
Browse files

mitmproxy: fix macos build

https://hydra.nixos.org/build/240114924/nixlog/1
```
error: linking with `/nix/store/sa6hywsm1mqfyd1xakyzv4ljjsb3hawh-clang-wrapper-11.1.0/bin/cc` failed: exit status: 1
  |
...
  = note: ld: library not found for -liconv
          clang-11: error: linker command failed with exit code 1 (use -v to see invocation)
```

after adding libiconv it starts to fail with
```
  File "/nix/store/ncs3h7zqjl3cl2cwflka40rrirb6qg1m-python3.11-mitmproxy-rs-0.3.11/lib/python3.11/site-packages/mitmproxy_rs/__init__.py", line 1, in <module>
    from .mitmproxy_rs import *
ModuleNotFoundError: No module named 'mitmproxy_macos'
```
so it needs extra module
https://github.com/mitmproxy/mitmproxy_rs/tree/main/mitmproxy-macos

So the fixes are
- add `libiconv` dependency to mitmproxy-rs
- add `mitmproxy-macos` python package and add it to dependencies of
  `mitmproxy-rs` and `mitmproxy` itself (otherwise import fails,
  looks like the import is sneaky because it is platform-conditional)
parent bb0fe34d
Loading
Loading
Loading
Loading
+33 −0
Original line number Diff line number Diff line
{ lib
, buildPythonPackage
, fetchFromGitHub
, hatchling
}:

buildPythonPackage rec {
  pname = "mitmproxy-macos";
  version = "0.3.11";
  pyproject = true;

  src = fetchFromGitHub {
    owner = "mitmproxy";
    repo = "mitmproxy_rs";
    rev = version;
    hash = "sha256-V6LUr1jJiTo0+53jipkTyzG5JSw6uHaS6ziyBaFbETw=";
  };

  sourceRoot = "${src.name}/mitmproxy-macos";
  pythonImportsCheck = [ "mitmproxy_macos" ];
  nativeBuildInputs = [
    hatchling
  ];

  meta = with lib; {
    description = "The MacOS Rust bits in mitmproxy";
    homepage = "https://github.com/mitmproxy/mitmproxy_rs/tree/main/mitmproxy-macos";
    changelog = "https://github.com/mitmproxy/mitmproxy_rs/blob/${src.rev}/CHANGELOG.md";
    license = licenses.mit;
    maintainers = with maintainers; [ boltzmannrain ];
    platforms = platforms.darwin;
  };
}
+5 −1
Original line number Diff line number Diff line
@@ -4,6 +4,8 @@
, fetchFromGitHub
, rustPlatform
, darwin
, libiconv
, mitmproxy-macos
}:

buildPythonPackage rec {
@@ -34,6 +36,8 @@ buildPythonPackage rec {

  buildInputs = lib.optionals stdenv.isDarwin [
    darwin.apple_sdk.frameworks.Security
    libiconv
    mitmproxy-macos
  ];

  pythonImportsCheck = [ "mitmproxy_rs" ];
+4 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@
, fetchFromGitHub
, buildPythonPackage
, pythonOlder
, stdenv
  # Mitmproxy requirements
, aioquic
, asgiref
@@ -15,6 +16,7 @@
, hyperframe
, kaitaistruct
, ldap3
, mitmproxy-macos
, mitmproxy-rs
, msgpack
, passlib
@@ -81,6 +83,8 @@ buildPythonPackage rec {
    urwid
    wsproto
    zstandard
  ] ++ lib.optionals stdenv.isDarwin [
    mitmproxy-macos
  ];

  nativeCheckInputs = [
+2 −0
Original line number Diff line number Diff line
@@ -6877,6 +6877,8 @@ self: super: with self; {
  mitmproxy = callPackage ../development/python-modules/mitmproxy { };
  mitmproxy-macos = callPackage ../development/python-modules/mitmproxy-macos { };
  mitmproxy-rs = callPackage ../development/python-modules/mitmproxy-rs { };
  mitmproxy-wireguard = callPackage ../development/python-modules/mitmproxy-wireguard { };