simplify #11

Merged
emery merged 1 commit from simple into trunk 2025-11-11 19:50:12 +00:00
Collaborator
No description provided.
@ -5,3 +4,1 @@
localSystem = prev.stdenv.buildPlatform.system;
crossSystem = prev.stdenv.hostPlatform.system;
pkgs = prev;
wciit = import ./pkgs/by-name {
Owner

I don't mind, but why did you switch to pkgs/by-name

I don't mind, but why did you switch to `pkgs/by-name`
Owner

I rebased on commits I added to trunk and resolved the conflicts.

I rebased on commits I added to trunk and resolved the conflicts.
Owner

Do we need to pin nixos-unstable with this change? I'm confused as to why that's still necessary. Pros and cons?

To be clear, I'm just confused about that conceptually, in general. I don't understand the implications.

Do we need to pin nixos-unstable with this change? I'm confused as to why that's still necessary. Pros and cons? To be clear, I'm just confused about that conceptually, in general. I don't understand the implications.
Owner

Other an the questions I posed above, I think this simplifies things in a good way, and also makes our packages look more similar to nixpkgs which has benefits. I tested building unetd, and it worked when I did nix-build -A pkgs.unetd.

Other an the questions I posed above, I think this simplifies things in a good way, and also makes our packages look more similar to nixpkgs which has benefits. I tested building unetd, and it worked when I did `nix-build -A pkgs.unetd`.
Owner

When added as a channel I have to reference like this nix-env -iA wciitpkgs.pkgs.unetd, and that works. However I can't seem to get anything to work with nix-shell -p.

Channel was added like this:
nix-channel --add "https://git.informatics.coop/nix/wciitpkgs/archive/simple.tar.gz" wciitpkgs

When added as a channel I have to reference like this `nix-env -iA wciitpkgs.pkgs.unetd`, and that works. However I can't seem to get anything to work with `nix-shell -p`. Channel was added like this: ` nix-channel --add "https://git.informatics.coop/nix/wciitpkgs/archive/simple.tar.gz" wciitpkgs`
Author
Collaborator

We could remove the nixpkgs pin, I left it pinned at where I found it. It's useful to have a reference but if we start building our own packages we'll need to rethink how we manage it.

It's best to install packages from here as an overlay on to the nixpkgs you already have.

There is a note on the wiki about using ~/.config/nixpkgs/overlays to install overlays but I've not done it that way.
https://wiki.nixos.org/wiki/Overlays

We could remove the nixpkgs pin, I left it pinned at where I found it. It's useful to have a reference but if we start building our own packages we'll need to rethink how we manage it. It's best to install packages from here as an overlay on to the nixpkgs you already have. There is a note on the wiki about using ~/.config/nixpkgs/overlays to install overlays but I've not done it that way. https://wiki.nixos.org/wiki/Overlays
Owner

For me this is ready to merge whenever you're ready

For me this is ready to merge whenever you're ready
devan approved these changes 2025-11-11 16:58:15 +00:00
emery merged commit 845aaa413b into trunk 2025-11-11 19:50:12 +00:00
devan deleted branch simple 2025-11-11 20:58:47 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
nix/wciitpkgs!11
No description provided.