Skip to content

Add support for nix-darwin to talon-nix #20

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

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

ProfDoof
Copy link

Goal

Wanted to add support for nix-darwin to talon-nix so that I could share my talon set-up easily between my machines.

Changes

  • I switched from nixpkgs to nixpkgs-unfree to allow for the use of legacyPackages and prevent adding more instances of nixpkgs to any users.
  • Used an enum in the scraper to allow for adding other options later if that's ever possible. Might be something to consider using to allow for beta combined with some allowances from talonvoice.
  • Made modules each into their own file to prevent setting udev rules for Mac while breaking common code out into a file
  • Changed to using a json.dump on the file instead of manually writing it to the file
  • Made the packages handle the respective cases for Linux and Darwin
  • Added a folder for the various build derivations and added the build derivations for linux and darwin to it

@ProfDoof
Copy link
Author

@Mic92 myself and others have several PRs in this repo but not had any responses from the maintainers.

@Mic92
Copy link
Member

Mic92 commented Dec 29, 2024

I contacted adisbladis to figure out what the status of maintenance is.

@ProfDoof
Copy link
Author

ProfDoof commented May 4, 2025

@Mic92 have you heard back from adisbladis

@Mic92
Copy link
Member

Mic92 commented May 5, 2025

I don't think he ever responded. But I also no longer no on which channel I asked.

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

Successfully merging this pull request may close these issues.

2 participants