Drift/pvv-nixos-config
17
5
Fork 1
Code Issues Pull Requests 9 Actions Wiki Activity

shell.nix: wrap nixos-rebuild with nix-output-monitor #106

Closed
pederbs wants to merge 2 commits from nom into main
pull from: nom
merge into: Drift:main
Conversation 2 Commits 2 Files Changed 2 +14 -2
pederbs commented 2025-05-31 19:38:30 +02:00
Owner
Copy Link

#nix-output-monitor-and-chill

nom is only used if the output is a tty, and I provide a escape hatch in the form of NIX_NO_NOM=1.
This solution sidesteps issues like sudo nixos-rebuild ... |& nom capturing the sudo password prompt.

#nix-output-monitor-and-chill nom is only used if the output is a tty, and I provide a escape hatch in the form of NIX_NO_NOM=1. This solution sidesteps issues like `sudo nixos-rebuild ... |& nom` capturing the sudo password prompt.
pederbs added 1 commit 2025-05-31 19:38:31 +02:00
shell.nix: wrap nixos-rebuild with nix-output-monitor
Some checks failed
Eval nix flake / evals (push) Has been cancelled
Details
Eval nix flake / evals (pull_request) Has been cancelled
Details
c16378469e 
\#nix-output-monitor-and-chill

nom is only used if the output is a tty, and I provide a escape hatch in the form of NIX_NO_NOM=1.
This solution sidesteps issues like `sudo nixos-rebuild ... |& nom` capturing the sudo password prompt.
pederbs force-pushed nom from c16378469e to bce1f0e912 2025-05-31 19:45:12 +02:00 Compare
pederbs added 1 commit 2025-05-31 20:44:00 +02:00
justfile: fix nom use condition.
Some checks failed
Eval nix flake / evals (push) Has been cancelled
Details
Eval nix flake / evals (pull_request) Has been cancelled
Details
31f2f445f3 
turns out just will plumb the stdout of its running processes, so we detect ttys on stderr instead
oysteikt approved these changes 2025-06-07 23:02:25 +02:00
pederbs closed this pull request 2025-06-08 13:15:09 +02:00
pederbs commented 2025-06-08 13:15:40 +02:00
Author
Owner
Copy Link

not needed in this repo, it'll be more of an obstruction than benefit

not needed in this repo, it'll be more of an obstruction than benefit
Some checks failed
Eval nix flake / evals (push) Has been cancelled
Details
Eval nix flake / evals (pull_request) Has been cancelled
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
oysteikt
Labels
Clear labels
art
The issue needs someone to be creative
backup
big
No, not bug, **big** - this is gonna take a while
blocked
This issue/PR depends on one or more other issues/PRs
bug
Something is not working, and it's not the shield hero
crash report
Report an oopsie
disputed
kranglefanter
documentation
Documentation changes required
duplicate
This issue or pull request already exists
enhancement
New feature
good first issue
Get your hands dirty with a new project here
logging
networking
TTM4100
nixos
Requires changes to our nixos setup
question
More information is needed
salt
Requires changes to our salt setup
security
Skommel
servers n' hardware
Regards hardware, servers or VMs
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
adriangl albertba alfhj amalieem bjornoka chrisfjo chrivi danio davidk dungby eirikwit felixalb frero jonmro jovre karolds knuta krisleri larshhan oysteikt pederbs root torsteno yorinad
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Drift/pvv-nixos-config#106
No description provided.
Delete Branch "nom"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?