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

lxqt-config-session after saving autostart element, add new backslashes before quotes #301

Open
ghost opened this issue Dec 9, 2019 · 4 comments
Labels

Comments

@ghost
Copy link

ghost commented Dec 9, 2019

lxqt-config-session after saving autostart element, add new backslashes before quotes even if there is exist backslashe before quotes.

Expected Behavior

lxqt-config-session before saving autostart element has to check for existing backslashe before quotes and for this case do not add a new backslashe.

Current Behavior

lxqt-config-session after saving autostart element, add new backslashes before quotes even if there is exist backslashe before quotes.

Possible Solution
Steps to Reproduce (for bugs)
  1. Launch lxqt-config-session
  2. Create test autostart element with command sh -c "sleep 12 && _some_cmd"
  3. Save it
  4. Try to edit this autostart element: the command became sh -c "sleep 12 && _some_cmd"
  5. Save it. After each saving, there is more and more backslashes before quotes: sh -c \"sleep 12 && _some_cmd\"
Context
System Information
  • Distribution & Version: Debian buster
  • Kernel: 4.19
  • Qt Version: 0.14.1
  • liblxqt Version:0.14.1-1
  • lxqt-build-tools Version: 0.6.0-2
  • Package version: lxqt-session (0.14.1-2)
@agaida
Copy link
Member

agaida commented Dec 9, 2019

Confirmed on latest git

@tsujan
Copy link
Member

tsujan commented Dec 9, 2019

It may be a very old issue never reported before.

@luis-pereira
Copy link
Member

Confirmed on latest git.

@stefonarch
Copy link
Member

stefonarch commented Dec 17, 2023

I'm running into this, but it adds also backlashes were they aren't needed, resulting in an invalid autostart entry.

  1. Add an autostart entry
  2. Add something like bash -c "if [ "$XDG_SESSION_TYPE" = "wayland" ];then sfwbar" into the "command" lineedit
  3. Check the file, it will have
    Exec=bash -c \\"if [ \\"\\$XDG_SESSION_TYPE\\" = \\"wayland\\" ];then sfwbar\\"
  4. Edit it, and backlashes will double.

@stefonarch stefonarch added the bug label Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants