SbarLua icon indicating copy to clipboard operation
SbarLua copied to clipboard

add nix-darwin package

Open Lalit64 opened this issue 11 months ago • 20 comments

Hey, I wrote a simple package for easier use with nix-darwin, because I found a lot of people having issues while trying to use it in nix-darwin. I have also tested it on my own nix-darwin system and it is working flawlessly

Lalit64 avatar Jan 19 '25 14:01 Lalit64

Hey, @FelixKratz, @amusingimpala75, @Pix-xiP can someone check this out?

Lalit64 avatar Jan 20 '25 07:01 Lalit64

Looks good in theory, I don't have nix-darwin to test it with if someone else can confirm I'd say its probably okay to merge assuming Felix is happy with having it packaged into Nix

Pix-xiP avatar Jan 20 '25 07:01 Pix-xiP

@amusingimpala75 let me make the changes and commit them.

Lalit64 avatar Jan 20 '25 12:01 Lalit64

@amusingimpala75 Check out commit, 1e0bf06 (#45)

I can confirm it is working on my system

Lalit64 avatar Jan 20 '25 13:01 Lalit64

With just a quick glance at that it seems probably fine; I should be able to find the time tomorrow to give it a proper look/test.

amusingimpala75 avatar Jan 22 '25 03:01 amusingimpala75

With just a quick glance at that it seems probably fine; I should be able to find the time tomorrow to give it a proper look/test.

Ok

Lalit64 avatar Jan 22 '25 03:01 Lalit64

@amusingimpala75 Hey, any updates?

Lalit64 avatar Jan 29 '25 09:01 Lalit64

Yeah sorry it's been a busy week. There are a few more changes I'd like to make; do you think you can select on the PR to allow maintainers to commit on your code?

amusingimpala75 avatar Jan 29 '25 11:01 amusingimpala75

@amusingimpala75 Changes are allowed by maintainers

Lalit64 avatar Jan 29 '25 12:01 Lalit64

Apparently I'm not a maintainer. So either I'm going to need to be promoted to one, or I'm going to need you to provide me (temporarily) with push access to your "nix-darwin-package" branch in your repository.

(I did have a few changes I made)

amusingimpala75 avatar Jan 29 '25 23:01 amusingimpala75

@FelixKratz I feel that me and @amusingimpala75 have done a good job and I don't see anything wrong with the PR? @amusingimpala75 your thoughts?

Lalit64 avatar Feb 22 '25 14:02 Lalit64

I am no expert with nix, but usually the distribution of a package occurs through a different repo than the one hosting the code, or am I mistaken? At least for sketchybar there is a nix formula in an official nix repo (which I myself am not maintaining).

FelixKratz avatar Feb 23 '25 18:02 FelixKratz

Nix can be used for packaging both in-tree (flake.nix etc. in the project's repo) as well as separately distributed (as in the nixpkgs repo). An example of both is Ghostty, which has a flake for packaging from source as well has a version in nixpkgs. Seeing as this is a relatively niche project that has no versioning and is intended to be built from source, I think it would make sense to package it within the project. It doesn't do any pinning of versions in repo, so if a user wanted to update to a later commit they would just update their personal pin (nix flake update in their dotfiles repo) without you needing to bump anything in this repo.

amusingimpala75 avatar Feb 24 '25 21:02 amusingimpala75

It can also just be (fairly easily) manually packaged in the user's dotfiles repo, but doing it this way removes the user having to figure out how packaging with Nix works.

amusingimpala75 avatar Feb 24 '25 21:02 amusingimpala75

If people start using it here and it's working we can add it to nixpkgs-unstable

Lalit64 avatar Feb 26 '25 10:02 Lalit64

@amusingimpala75 How about we file a package request in the nixpkgs repository?

Lalit64 avatar Mar 10 '25 08:03 Lalit64

I will try to look into that tomorrow evening.

amusingimpala75 avatar Mar 11 '25 15:03 amusingimpala75

Any updates on this?

h4ckd0tm3 avatar Apr 01 '25 09:04 h4ckd0tm3

I currently have an open PR in the nixpkgs repo; I may need to ping someone to get it reviewed.

amusingimpala75 avatar Apr 01 '25 14:04 amusingimpala75

any update?

phucisstupid avatar May 17 '25 10:05 phucisstupid

A package was submitted to nixpkgs a month ago, this can be closed

amusingimpala75 avatar Jun 27 '25 23:06 amusingimpala75