python3Packages.sphinx-serve: init at 1.0.1

[FlorianFranzen]

Motivation for this change

sphinx-serve was missing from nixpkgs

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 120045 at c5c9e0e7 run on aarch64-linux 1

2 packages built successfully:

• python38Packages.sphinx-serve
• python39Packages.sphinx-serve

1 suggestion:

• warning: no-python-tests

  Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
  Near pkgs/development/python-modules/sphinx-serve/default.nix:13:0:

     |
  13 |     description = "Spawns a simple HTTP server to preview your sphinx documents";
     | ^
  

---

Result of nixpkgs-review pr 120045 at c5c9e0e7 run on x86_64-linux 1

2 packages built successfully:

• python38Packages.sphinx-serve
• python39Packages.sphinx-serve

1 suggestion:

• warning: no-python-tests

  Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
  Near pkgs/development/python-modules/sphinx-serve/default.nix:13:0:

     |
  13 |     description = "Spawns a simple HTTP server to preview your sphinx documents";
     | ^
  

[FlorianFranzen]

Fixed test handling based on @r-rmcgibbo feedback.

[SuperSandro2000]

Does this package contain a binary which could use a top level entry?

[FlorianFranzen]

Does this package contain a binary which could use a top level entry?

@SuperSandro2000 Indeed it does, but I refrained from exposing it, because sphinx itself also has no top level entry. But I can change that too, to be consistent.

[SuperSandro2000]

Indeed it does, but I refrained from exposing it, because sphinx itself also has no top level entry. But I can change that too, to be consistent.

Then we should probably add one for both to make it easier to find those binaries/scripts.

[FlorianFranzen]

@SuperSandro2000 Ok, done. Added each in a separate commit. Thanks again for your support getting this merged.