freshlybakedca.ke / patisserie
Your one-stop-cake-shop for everything Freshly Baked has to offer
fork atom

feat: use nix-output-monitor for builds #18

merged
opened by a.starrysky.fyi targeting main from private/minion/push-tqrsnzyqotlz

It'd be nice to use nix output monitor for builds - there's a project which automatically uses nix output monitor when nix is called (even supporting nix-direnv/nixos-rebuild)... this looks probably good for us to use.

There is one issue of some importance on the nix-monitored tracker which suggests that nix-monitored incorrectly transforms some 'nix run' invocations... fortunately, we don't use flakes so this shouldn't be a huge issue

If it is, using

NIX_MONITOR=disable

will temporarily disable nix monitor, allowing you to run whatever you needed without issue while using nom for everything else

STACK 1
#18 feat: use nix-output-monitor for builds
4
#0
0
by a.starrysky.fyi 4 comments
expand 1 commit
6ab3a2b8
feat: use nix-output-monitor for builds
thecoded.prof

I wonder if we should rename the ingredient to nix? We also set the GC rules in there so it could make some sense to do so. Either way though I'm happy with this change.

a.starrysky.fyi

@thecoded.prof - presuming you mean the file ('lix.nix') instead of the ingredient ('common')? -- if so, I have no particular preference on the name but I'm happy for you to switch it in a followup PR... we shouldn't rename the ingredient itself - if we split this out it'd be because we decided we don't want it to apply on all systems...

thecoded.prof

sorry yep. that's what I meant haha.

a.starrysky.fyi

ok - in that case I'm going to merge this, feel free to rename to nix.nix in a followup without my review

pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

requested-reviewers

None yet.

approved

None yet.

tested-working

None yet.

rejected

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:uuyqs6y3pwtbteet4swt5i5y/sh.tangled.repo.pull/3lz4ricklun22