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

Use relative paths and more portable commands #173

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Jacoby6000
Copy link

@Jacoby6000 Jacoby6000 commented Jul 16, 2024

This is some prep-work that allows for more portable installations. This enables NixOS compatibility and a way for users to test things without messing with their existing rofi configurations.

  • The setup.sh script is updated to support custom directories for installation location. The script will still default to the old behavior, but users can have more control over where things are installed if they'd like.
  • To support custom install locations, scripts have stopped using absolute paths, instead resolving their own file path and then using relative paths from there. This should work on any system that has readlink -f support.
  • Themes have also been updated to use relative paths for imports rather than absolute paths.
  • To support read-only file-systems, users can now specify themes via the CLI. Examples below
    • applets/bin/appasroot.sh type-1/style-2.rasi
    • launchers/type-1/launcher.sh 4
    • powermenu/type-1/launcher.sh 3
  • Env vars can be used to set certain commands in the apps applet
  • uptime has multiple variations, some of which do not have a -p option. This implements a portable uptime in bash
  • Still some more work to be done wrt portability for the audio commands and music player stuff.

@Jacoby6000 Jacoby6000 changed the title Use relative paths and readlink for portability Use relative paths and more portable commands Jul 17, 2024
@Jacoby6000
Copy link
Author

Just realized I need to make some adjustments to the image urls in the rasi themes. The rest seems to work as expected

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.

1 participant