You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
run powerprofilesctl set performance (note that the 'set', 'performance' arguments are extra arguments not part of the suggested commands)
observe that it works
run powerprofilesctl set performance again (note that it now finds this command in the history)
observe that it doesn't work due to incorrect quoting
Expected behavior
Running a command from history should behave the exact same was as running a command the first time.
Actual behavior
Rofi applies quoting to the command from history, resulting in rofi trying to find a command named 'powerprofilesctl set performance' (including quotes), which fails.
Additional information
The following patch fixes the immediate problem by avoiding shell quoting of the command arguments:
Rofi version (rofi -v or git commit in case of build issue)
1.7.5 wayland3-2
Configuration
(shown for completeness, config is
/dev/null
)https://gist.github.com/Ferdi265/632b99b10f16440b3cdf43f8282b58cb
Theme
(shown for completeness, config is
/dev/null
)https://gist.github.com/Ferdi265/632b99b10f16440b3cdf43f8282b58cb
Timing report
No response
Launch command
rofi -config /dev/null -show run
Step to reproduce
powerprofilesctl set performance
(note that the 'set', 'performance' arguments are extra arguments not part of the suggested commands)powerprofilesctl set performance
again (note that it now finds this command in the history)Expected behavior
Running a command from history should behave the exact same was as running a command the first time.
Actual behavior
Rofi applies quoting to the command from history, resulting in rofi trying to find a command named
'powerprofilesctl set performance'
(including quotes), which fails.Additional information
The following patch fixes the immediate problem by avoiding shell quoting of the command arguments:
This patch works, and I've been using it on my system for weeks now, but I haven't worked on it a lot and there might be a better solution.
I'd be happy to work out a solution to this in more detail if required and can open a PR for this.
Note that this issue is specific to this fork, appeared rather recently, and doesn't occur with upstream rofi releases.
Using wayland display server protocol
I've checked if the issue exists in the latest stable release
The text was updated successfully, but these errors were encountered: