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

Autostart doesn't work on Linux #5

Closed
hexion opened this issue Feb 25, 2017 · 6 comments
Closed

Autostart doesn't work on Linux #5

hexion opened this issue Feb 25, 2017 · 6 comments
Assignees

Comments

@hexion
Copy link

hexion commented Feb 25, 2017

Running on Ubuntu 16.04, with a gnome-flashback DE.

The application is installed by default on /opt/Desktop Dimmer/ if installed via the .deb package.
Selecting the "Launch on startup" setting does create an entry to auto start the application, but it fails to escape the space in the path, rendering the autostart unusable.

/opt/Desktop Dimmer/desktop-dimmer --hidden
should be
/opt/Desktop\ Dimmer/desktop-dimmer --hidden

Or even simpler, and considering the .deb package installs a link in /usr/local/bin, another option is to remove the path from the launcher command:
desktop-dimmer --hidden

@sidneys
Copy link
Owner

sidneys commented Feb 27, 2017

@hexion
Thanks for the report. Please re-test with v2.5.2 and comment if this helps.

@hexion
Copy link
Author

hexion commented Feb 27, 2017

Same results.
I deleted the startup entry, updated the app to 2.5, then re-launched it.
A new startup entry was created:

Name: desktop-dimmer
Command: /opt/Desktop Dimmer/desktop-dimmer --hidden
Comment: desktop-dimmerstartup script

The space in "Desktop Dimmer" is still not escaped, and so the application fails to auto start:

$ /opt/Desktop Dimmer/desktop-dimmer --hidden
bash: /opt/Desktop: No such file or directory

@sidneys
Copy link
Owner

sidneys commented Feb 27, 2017

@hexion

Thanks for the follow-up. The issue has been addressed, please re-test with version v2.7.0 and post your results. Cheers.

@sidneys sidneys assigned sidneys and unassigned sidneys Feb 27, 2017
@hexion
Copy link
Author

hexion commented Feb 28, 2017 via email

@sidneys
Copy link
Owner

sidneys commented Mar 1, 2017

@hexion

A third-party module, node-auto-launch, is the culprit here, regardless of it having a near 100% test coverage. Hence we integrated our own patched version of this library, in v2.7.2. Please re-test. Thanks for your accurate and timely reports.

@hexion
Copy link
Author

hexion commented Mar 1, 2017

I can confirm it is fixed now, the startup link created is now valid and autostart is working fine.
Thanks for fixing it!

@sidneys sidneys closed this as completed Mar 1, 2017
@sidneys sidneys self-assigned this Nov 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants