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
mitmproxy: 0.17.1 -> 1.0.2 #21639
mitmproxy: 0.17.1 -> 1.0.2 #21639
Conversation
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.
That's a lot of updates, thanks. Please fix tests or at least include the reason as a comment for disabling them. I suppose the test suites are run for all other packages with all main interpreter versions?
watchdog editorconfig | ||
]; | ||
|
||
doCheck = false; |
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.
no tests? Please always include a comment with the reason why tests are disabled.
}; | ||
|
||
propagatedBuildInputs = with self; [ six ]; | ||
|
||
doCheck = !isPy3k; |
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.
can you include a reason why
src = pkgs.fetchurl { | ||
url = "mirror://pypi/s/sortedcontainers/${name}.tar.gz"; | ||
sha256 = "1sjh8lccbmvwna91mlhl5m3z4320p07h063b8x8br4p4cll49w0g"; | ||
}; | ||
|
||
doCheck = false; |
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.
can you include the reason
}; | ||
|
||
propagatedBuildInputs = with self; [ ruamel_base ruamel_ordereddict ]; | ||
doCheck = false; |
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.
reason
Added comments with explanations for disabling the tests. I don't have time to fix the tests and packaging new and bumping existing python packages was tedious enough for me. As indicated above, I have not run |
a5a6278
to
d8646bd
Compare
Okay, so all reverse dependencies build for me with |
Motivation for this change
mitmproxy was marked as broken and needed fixing and a version bump.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)