Skip to content
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

Mark broken master #85417

Closed
wants to merge 4 commits into from
Closed

Conversation

worldofpeace
Copy link
Contributor

Motivation for this change

Forward port the thing #85331

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

disassembler and others added 2 commits April 16, 2020 22:09
(cherry picked from commit 48c5aaa)
@worldofpeace
Copy link
Contributor Author

Need to unmark #85415. Really might need to put a sign up to not conflict with this work.

@@ -93,5 +93,6 @@ buildPythonPackage rec {
license = licenses.bsd3;
maintainers = with maintainers; [ drewkett marsam ];
platforms = platforms.unix;
broken = true;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

scrapy is not broken:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. Looks like a pattern. Same as in https://github.com/NixOS/nixpkgs/pull/85417/files#r410125598

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it broken on release-20.03 though?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -42,5 +42,6 @@ stdenv.mkDerivation rec {
'';
license = licenses.gpl2Plus;
maintainers = with maintainers; [ orivej ];
broken = true;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please don't mark muse; backport of the fixes: #85424

@@ -79,5 +79,6 @@ buildPythonPackage rec {
homepage = "https://mitmproxy.org/";
license = licenses.mit;
maintainers = with maintainers; [ fpletz kamilchm ];
broken = true;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mitmproxy is not at fault for the failure of its TestMasterSecretLogger.test_log on release-20.03. It does not fail on master, yet there is no difference between mitmproxy definitions between release-20.03 and master.

@@ -57,5 +57,6 @@ stdenv.mkDerivation rec {
license = with licenses; [ asl20 bsd3 gpl2 lgpl21 lgpl21Plus openldap ];
platforms = [ "x86_64-linux" ];
maintainers = with maintainers; [ markuskowa ];
broken = true;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Problem is addressed in this open PR: #81980.
Please don't mark broken just yet.

@@ -39,5 +40,6 @@ buildPythonPackage rec {
description = "Command-line tool and library to interact with an aria2c daemon process with JSON-RPC";
license = licenses.isc;
maintainers = with maintainers; [ koral ];
broken = isPy38;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We use disabled = isPy38 instead of broken for pythonPackages.
cc @FRidh

@symphorien symphorien mentioned this pull request Apr 17, 2020
10 tasks
@@ -34,5 +34,6 @@ buildPythonPackage rec {
description = "A Python 2 and 3 asynchronous JSON RPC server";
license = licenses.mit;
maintainers = [ maintainers.mic92 ];
broken = true;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is not broken afaik. It is needed for the python-language-server,
which is important.

Build Status for nixpkgs.python36Packages.python-jsonrpc-server.x86_64-linux on unstable
✔ python3.6-python-jsonrpc-server-0.0.2 from 2018-10-24 - https://hydra.nixos.org/build/83221618

Tested for all python versions!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. python38 is actually broken. Disabled this one in ee6f5a3
Also backported: 9094d35

@symphorien
Copy link
Member

paperwork is not broken, see #85436

@@ -20,5 +20,6 @@ buildGoModule rec {
homepage = "https://github.com/nektos/act";
license = licenses.mit;
maintainers = with maintainers; [ filalex77 ];
broken = true;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also repaired this one quiet recently.
Should be not broken as well.

Copy link
Member

@peti peti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes related to the Haskell package set should be dropped.

@bcdarwin
Copy link
Member

python3Packages.spyder is not broken on master (#83038).

@@ -53,5 +53,6 @@ stdenv.mkDerivation rec {
maintainers = with maintainers; [ etu ];
license = licenses.gpl2;
platforms = platforms.linux;
broken = true;
Copy link
Member

@bachp bachp Apr 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@worldofpeace
Copy link
Contributor Author

With some feedback from some community members, forward porting such changes to master isn't going to be very fruitful. Hoping to have automated mark as broken soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet