Skip to content

dazel: migrate to by-name and modernize#489544

Merged
wegank merged 2 commits intoNixOS:masterfrom
guylamar2006:migrate-dazel
Apr 13, 2026
Merged

dazel: migrate to by-name and modernize#489544
wegank merged 2 commits intoNixOS:masterfrom
guylamar2006:migrate-dazel

Conversation

@guylamar2006
Copy link
Copy Markdown
Contributor

@guylamar2006 guylamar2006 commented Feb 11, 2026

This pull request refactors the packaging of the dazel Python application to align with the new package structure and modern Nix conventions. The most important changes include renaming and relocating the package, updating the build process to use python3Packages, and removing the old package reference from the top-level package list.

Packaging refactor:

  • Renamed and moved the dazel package from pkgs/development/tools/dazel/default.nix to pkgs/by-name/da/dazel/package.nix, following the new naming and directory conventions.
  • Updated the build process to use python3Packages.buildPythonApplication and python3Packages.setuptools instead of the older buildPythonApplication and direct setuptools reference, improving compatibility and maintainability.

Top-level package list update:

  • Removed the old reference to dazel from pkgs/top-level/all-packages.nix, reflecting the package's migration and preventing duplicate or outdated entries.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Feb 11, 2026
@guylamar2006
Copy link
Copy Markdown
Contributor Author

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 489544
Commit: 2c39b0b82206c3c4f1614d37e63ac3045e377d6b (subsequent changes)
Merge: 7cbcff48a29bed16c8dc69ef34c0ef9718975ca6

Logs: https://github.com/guylamar2006/nixpkgs-review-gha/actions/runs/21923715111


x86_64-linux

No rebuilds


aarch64-linux

No rebuilds


x86_64-darwin

No rebuilds


aarch64-darwin

No rebuilds

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Feb 18, 2026
@wegank wegank added this pull request to the merge queue Apr 13, 2026
Merged via the queue into NixOS:master with commit 67ebd84 Apr 13, 2026
33 checks passed
@guylamar2006 guylamar2006 deleted the migrate-dazel branch April 13, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any 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.

3 participants