Skip to content

python3Packages.wslink: init at 2.3.4#416458

Merged
NickCao merged 1 commit intoNixOS:masterfrom
qbisi:wslink
Jun 22, 2025
Merged

python3Packages.wslink: init at 2.3.4#416458
NickCao merged 1 commit intoNixOS:masterfrom
qbisi:wslink

Conversation

@qbisi
Copy link
Contributor

@qbisi qbisi commented Jun 13, 2025

python3Packages.wslink: Internal server side implementation of trame

python dependency of vtk/trame.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@nix-owners nix-owners bot requested a review from natsukium June 13, 2025 14:07
@github-actions github-actions bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 6.topic: python Python is a high-level, general-purpose programming language. labels Jun 13, 2025
@wolfgangwalther
Copy link
Contributor

Re-running CI due to a odd failure fixed in #416448

@github-actions github-actions bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jun 14, 2025
Comment on lines 27 to 28
Copy link
Member

Choose a reason for hiding this comment

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

How about using sourceRoot = "${src.name}/python" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looks good in this case. However setting sourceRoot to subdir will not always work when the setup.py script requires files out of the subdir. e.g. in python3Packages.fenics-dolfinx,

preConfigure = ''
cd python
'';

pyproject wants to install a license file under ${src} out of ${src}/python,
https://github.com/FEniCS/dolfinx/blob/3be58ec98d266dcae6d57e16c725fa3328dd20d2/python/pyproject.toml#L61
thus setting sourceRoot = "${src.name}/python" will fail the build of python3Packages.fenics-dolfinx.

Copy link
Member

Choose a reason for hiding this comment

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

May want to add a comment noting this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Another concern about setting souce root via "${src.name}/python" is about the unpacked tarball source.

For example :

  src = fetchurl {
    url = "https://github.com/Kitware/wslink/archive/refs/tags/v2.3.4.tar.gz";
    hash = "sha256-SybJvGZaVxvd57BWW3D4VQ/bHUB8Da0yEaPWS35CO+E=";
  };
  
  sourceRoot = "${src.name}/python";

  # add missing version string to dist-info
  postPatch = ''
    sed -i "/name *= */a\    version='${version}'," setup.py
  '';  

will set sourceRoot = "v2.3.4.tar.gz/python" and result in build failure.

@qbisi
Copy link
Contributor Author

qbisi commented Jun 17, 2025

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 416458

Logs: https://github.com/qbisi/nixpkgs-review-gha/actions/runs/15695692004


x86_64-linux (sandbox = true)

✅ 4 packages built:
  • python312Packages.wslink
  • python312Packages.wslink.dist
  • python313Packages.wslink
  • python313Packages.wslink.dist

aarch64-linux (sandbox = true)

✅ 4 packages built:
  • python312Packages.wslink
  • python312Packages.wslink.dist
  • python313Packages.wslink
  • python313Packages.wslink.dist

x86_64-darwin (sandbox = true)

✅ 4 packages built:
  • python312Packages.wslink
  • python312Packages.wslink.dist
  • python313Packages.wslink
  • python313Packages.wslink.dist

aarch64-darwin (sandbox = true)

✅ 4 packages built:
  • python312Packages.wslink
  • python312Packages.wslink.dist
  • python313Packages.wslink
  • python313Packages.wslink.dist

@qbisi qbisi mentioned this pull request Jun 17, 2025
13 tasks
@NickCao NickCao merged commit 2b93f5c into NixOS:master Jun 22, 2025
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants