Skip to content
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

rofi: Add option to symlink dmenu #107146

Merged
merged 2 commits into from Mar 21, 2021
Merged

Conversation

SamirHafez
Copy link
Contributor

Rofi has the ability to handle input from scripts designed for dmenu. As per the documentation, all it's needed to get this working is a symlink:

If argv[0] (calling command) is dmenu, rofi will start in dmenu mode. This way it can be used as a drop-in replacement for dmenu. just copy or symlink rofi to dmenu in $PATH.

I'm adding this ability to the Rofi derivation by adding the symlink-dmenu bool argument

@tomberek
Copy link
Contributor

Requires a rebase. The previous change added a gappsWrapper which may mess/conflict with the symlink. Should it symlink the wrapped rofi?

Rofi has the ability to handle input from scripts designed for `dmenu`. As per the documentation, all it's needed to get this working is a symlink:

> If argv[0] (calling command) is dmenu, rofi will start in dmenu mode. This way it can be used as a drop-in replacement for dmenu. just copy or symlink rofi to dmenu in $PATH.

I'm adding this ability to the Rofi derivation by adding the `symlink-dmenu` bool argument
@SamirHafez
Copy link
Contributor Author

@tomberek

Requires a rebase. The previous change added a gappsWrapper which may mess/conflict with the symlink. Should it symlink the wrapped rofi?

I've rebased and tested locally and it still works.
Tried with linking the wrapper instead but I guess that doesn't work as the executable argument won't be named dmenu anymore so rofi won't try to emulate

Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
@SuperSandro2000 SuperSandro2000 merged commit f7ac029 into NixOS:master Mar 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants