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
termite: factor wrapper out into its own file #43691
Conversation
b5c6df0
to
e9ef19d
Compare
File name |
you're actually right, the wrapped version should be in |
Until now it's impossible to override the attrs of the actual build instruction for the `termite` package like this: ``` termite.overrideAttrs (_: { # ... }) ``` This issue occurs since the `termite/default.nix` expressions returns the `symlinkJoin` expression when I override termite (e.g. to provide a config file). I recently patched termite and wanted to apply this patch to my local termite installation in my system config which is impossible this, so splitting the wrapper and the build instruction into their own files makes this way easier to maintian.
e9ef19d
to
1f55aaa
Compare
good catch @mnacamura! I fixed it in the latest commit 1f55aaa |
No attempt on x86_64-linux (full log) The following builds were skipped because they don't evaluate on x86_64-linux: termite Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: termite Partial log (click to expand)
|
@GrahamcOfBorg build termite |
Success on x86_64-linux (full log) Attempted: termite Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: termite Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: termite Partial log (click to expand)
|
Motivation for this change
Until now it's impossible to override the attrs of the actual build
instruction for the
termite
package like this:This issue occurs since the
termite/default.nix
expressions returnsthe
symlinkJoin
expression when I override termite (e.g. to provide aconfig file).
I recently patched termite and wanted to apply this patch to my local
termite installation in my system config which is impossible this, so
splitting the wrapper and the build instruction into their own files
makes this way easier to maintian.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)