nixos/awesome: add wrapped awesome to systemPackages #95750
Conversation
| @@ -11,6 +11,9 @@ let | |||
| " --search " + (getLuaPath path "share") + | |||
| " --search " + (getLuaPath path "lib") | |||
| ); | |||
| awesomeWrapped = (writeScriptBin ".awesome-wrapped" '' | |||
There was a problem hiding this comment.
Normally with wrapProgram the wrapper becomes available under the normal name and the original executable is moved to .$exeName-wrapped - the opposite of what you're doing here.
I don't see a reason to keep the non-wrapped awesome around, so to avoid confusion I think it makes sense to use plain awesome as the name for the wrapper and remove the original awesome from systemPackages.
If there's a good reason to keep the original around, you could use a name like awesome-wrapped or something similar.
There was a problem hiding this comment.
The Idea was to not break anything. On closer inspection, I wouldn't know what could break though. Your reasoning sounds perfectly reasonable to me, so I will adjust accordingly.
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
|
@GrahamcOfBorg eval |
stale
bot
removed
the
2.status: stale
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
stale
bot
removed
the
2.status: stale
wucke13
force-pushed
the
awesome
branch
3 times, most recently
from
0b38813
to
983fdc5
Compare
Lets user start awesome in the same way as the display-manager would do, including luaPaths.
|
@wucke13 Do you know how I can build this from the command line for testing? I am wondering whether the |
I would use Or build a virtual machine with a config using the checkout of my repo as nixpkgs.
Valid points which I have not thought about |
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
|
I don't have interest any more in pursuing this. |
Motivation for this change
Lets user start awesome in the same way as the display-manager would do,
including luaPaths.
I prefer not using a display manager at all, and instead starting my wm using
xinit/startx. However, that breaks my awesome setup as I use some the nixos config to add some lua modules to awesomes search path: these search path additions are only made available for the xsession script. This change makes it possible to started a wrapped awesome with the same lua search paths from startx.Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)