Skip to content

modules/rofi: init#28

Merged
Lassulus merged 2 commits intoLassulus:mainfrom
altacountbabi:rofi
Nov 6, 2025
Merged

modules/rofi: init#28
Lassulus merged 2 commits intoLassulus:mainfrom
altacountbabi:rofi

Conversation

@altacountbabi
Copy link
Copy Markdown
Contributor

Add wrapper module for rofi

} name value)
+ "\n";

toRasi =
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it might be interesting to add such formats in the lib. ie wlib.formats.rasi if this format is used elsewhere

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only rofi uses rasi, but there are some formats, like hyprlang used for all the hyprwm things

@Lassulus
Copy link
Copy Markdown
Owner

Lassulus commented Nov 1, 2025

I guess this is not compatible with macos, I need to finish #19 and add a meta field afterwards so we can specify platforms and maintainers

};

config.flags = {
"-config" = config.pkgs.writeTextFile {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it would be nice if this textfile could be generated through a wlib.types.file option. so people could override the path if they want to have a config file outside the store (or read from an environment file)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

something like:

      options."rofi.cfg" = lib.mkOption {
        type = wlib.types.file config.pkgs;
        default.content = toRasi ...;
      };
 config.flags."--config" = config."rofi.cfg".path;

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah I'll implement it

@Lassulus Lassulus merged commit 85aec5d into Lassulus:main Nov 6, 2025
2 checks passed
@Lassulus
Copy link
Copy Markdown
Owner

Lassulus commented Nov 6, 2025

pretty nice work!

@altacountbabi altacountbabi deleted the rofi branch November 6, 2025 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants