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

Conky manager fails to create correct startup script when using startx #41

Open
beastr45 opened this issue Jun 1, 2023 · 1 comment

Comments

@beastr45
Copy link

beastr45 commented Jun 1, 2023

When building the auto-start script conky manager will pull from $DESKTOP_SESSION and add whatever it finds in an if statement in order to auto start conky. When a user uses startx and refuses to use a display manager the desktop session is not set and is empty. Conky manager sees that it is empty and instead of using an empty string for null conky manager uses "(null)" obviously nothing happens because this is not valid bash syntax and conky will never start. In order to fix this somebody could ad logic to detect if $DESKTOP_SESSION is empty and just use and empty string "" instead of "(null)". If this is not fixed the user would have to make a new start script everytime they changed the config undermining the ease of use conky manager offers.

@zcot
Copy link
Owner

zcot commented Jun 4, 2023

That should handle an actual null string now: 85fd0b3

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

No branches or pull requests

2 participants