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

AppImage : Default path detection doesn't work properly because it uses mount path #89

Closed
artemv opened this issue Nov 8, 2018 · 5 comments
Assignees
Labels
Milestone

Comments

@artemv
Copy link
Contributor

artemv commented Nov 8, 2018

  • The target platform(s) the problem occurs on (Linux, Mac, Mac App Store, and or Windows): Linux, Ubuntu 18.04.1 LTS
  • Node version (run node -v): 10.11.0
  • auto-launch version: 5.0.5
  • The options you're passing to auto-launch: {name: 'My App', isHidden: false}
  • Are you using NW.js or Electron? Electron
    • If so, which version? 2.0.7
    • If you're using Electron, are you using auto-launch from the main process or a renderer process? main process
  • If needs be, would you be able to provide us with a simple app (as simple as possible) which reproduces the problem? yes

I'm packaging my Electron app as AppImage for Linux, and the auto-launch puts the process.execPath to created desktop entry, which looks like /tmp/.mount-myapp-des20NgDd/my-app. The .mount-... dir is temporary so it doesn't work with auto-launch.

Workaround is to pass {path: process.env.APPIMAGE} to AutoLaunch.

@artemv
Copy link
Contributor Author

artemv commented Nov 12, 2018

the workaround above will work fine only with #91 though - see #90

erichs pushed a commit to erichs/stethoscope-app that referenced this issue Mar 7, 2019
per Teamwork/node-auto-launch#89,
this resolves the desktop autostart path to the executable, which
defaulted to current working path of the running process. Due to the
AppImage architecture, that default doesn't work cleanly and requires
override.
rmcvey pushed a commit to Netflix-Skunkworks/stethoscope-app that referenced this issue Mar 7, 2019
per Teamwork/node-auto-launch#89,
this resolves the desktop autostart path to the executable, which
defaulted to current working path of the running process. Due to the
AppImage architecture, that default doesn't work cleanly and requires
override.
@adam-lynch
Copy link
Contributor

Please see that we're looking for contributors / maintainers: #64.

I'm happy to give access to people who are willing to help improve things, merge pull-requests, close issues, etc.

@Fiser12
Copy link

Fiser12 commented Nov 26, 2019

To us the auto-launch fails with the packaging of AppImage. The reason is that the system mounts the AppImage temporary folder and it no longer exists when the app is closed.

image

@Oxalin Oxalin self-assigned this Dec 4, 2023
@Oxalin Oxalin added the bug label Dec 4, 2023
@Oxalin Oxalin changed the title Default options.path doesn't work with electron & AppImage AppImage : Default path detection doesn't work properly because it uses mount path Dec 6, 2023
@Oxalin
Copy link
Contributor

Oxalin commented Dec 7, 2023

This should be fixed by #91

@Oxalin
Copy link
Contributor

Oxalin commented Dec 7, 2023

Feel free to give feedback

@Oxalin Oxalin closed this as completed Dec 7, 2023
@Oxalin Oxalin added this to the 5.1.0 milestone Mar 22, 2024
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