-
Notifications
You must be signed in to change notification settings - Fork 69
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
App automatically installs in /usr/lib/ since 0.5.0 #56
Comments
@dlech pointed out that Debian guidelines dictate that apps like the ones generated through Electron must live in |
Hi @unindented , sorry it took a bit to respond. In our case, we got reports from Linux users because their shortcuts stopped working. This is the way we actually found out about the change: our app used to be installed in /usr/share, and when we got the report we realized about the change in electron-installer-debian. That was not a very serious problem, but discussing this further actually made us think that our app should be in /opt according to the guidelines, and not in /usr/share or /usr/lib, but I couldn't find a way at all to do this with electron-installer-debian. Maybe I can rephrase this issue or open a different one, but the real limitation is that we are not able to choose where the app is installed using this tool. Thanks, and sorry again for taking a couple of days to answer. |
Since it is installed with a |
This also caught us out in Etcher balena-io/etcher#1290 Perhaps this change-of-path in 0.5.0 ought to be mentioned somewhere prominent in the README? |
@lurch you are right. I'll try to add a note in the README. |
@malept could you also change the README to point out where the |
Since I upgraded to electron-installer-debian 0.5.0, I noticed that this commit causes my app to install in /usr/lib. This is absolutely not desirable in our case, and I haven't found a way to at least configure this path.
I rolled back the dependency to 0.4.0 momentarily to fix this, but I wouldn't like to be stuck with that forever. Are there plans to make this path configurable (or am I missing this option somewhere)?
Thanks!
The text was updated successfully, but these errors were encountered: