dyndnsc: 0.5.1 -> 0.6.1

[AluisioASG]

Motivation for this change

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.

[r-rmcgibbo]

Result of nixpkgs-review pr 120447 at d320b43f run on aarch64-linux 1

5 packages built successfully:

• dyndnsc
• python38Packages.json-logging
• python38Packages.pytest-console-scripts
• python39Packages.json-logging
• python39Packages.pytest-console-scripts

2 suggestions:

• warning: unused-argument

  Unused argument: fastapi.
  Near pkgs/development/python-modules/json-logging/default.nix:10:3:

     |
  10 | , fastapi
     |   ^
  

• warning: missing-patch-comment

  Please add a comment on the line above, explaining the purpose of this patch.
  Near pkgs/development/python-modules/json-logging/default.nix:26:5:

     |
  26 |     (fetchpatch {
     |     ^
  

---

Result of nixpkgs-review pr 120447 at d320b43f run on x86_64-linux 1

5 packages built successfully:

• dyndnsc
• python38Packages.json-logging
• python38Packages.pytest-console-scripts
• python39Packages.json-logging
• python39Packages.pytest-console-scripts

2 suggestions:

• warning: unused-argument

  Unused argument: fastapi.
  Near pkgs/development/python-modules/json-logging/default.nix:10:3:

     |
  10 | , fastapi
     |   ^
  

• warning: missing-patch-comment

  Please add a comment on the line above, explaining the purpose of this patch.
  Near pkgs/development/python-modules/json-logging/default.nix:26:5:

     |
  26 |     (fetchpatch {
     |     ^
  

[AluisioASG]

Updated pytest-console-scripts to 1.2.0 but got hit by a broken dependency during rebase.

[dotlambda]

python3Packages.openapi-spec-validator and python3Packages.sanic fail to build
cc @rvl @costrouc

[AluisioASG]

openapi-spec-validator is fixed at #120793. sanic needs a bunch of fixes after being updated to 21.3, I'm working on it.

[dotlambda]

LGTM, but we can only test after #120881 is merged.

[SuperSandro2000]

@ofborg build dyndnsc

[SuperSandro2000]

@AluisioASG I think connexion broke in the last week or so. Can you fix it so that we can check if dyndnsc still builds?

[AluisioASG]

@AluisioASG I think connexion broke in the last week or so. Can you fix it so that we can check if dyndnsc still builds?

@SuperSandro2000 it's incompatible with the currently packaged version of openapi-spec-validator (spec-first/connexion#1343). To fix it for the entirety of Nixpkgs we'd probably have to package a openapi-spec-validator_0_29 just for it.

However, in dyndnsc's case we can just disable the tests that use it in json-logging. We already do that with FastAPI, which is similarly broken (#112701 (comment)) and I only don't suggest doing the same with Sanic because #120881 is well underway.

[dotlambda]

@ofborg eval

[AluisioASG]

Rebased and built locally. Turns out there are no tests using connexion so we can just drop it from dependencies.