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

Do not recommend restarting the system if it is not needed #303

Closed
probonopd opened this issue Dec 6, 2018 · 14 comments
Closed

Do not recommend restarting the system if it is not needed #303

probonopd opened this issue Dec 6, 2018 · 14 comments

Comments

@probonopd
Copy link

probonopd commented Dec 6, 2018

https://flatpak.org/setup/Ubuntu/ says

  1. Restart

To complete setup, restart your system.

Actually a restart may not be necessary, logging out of the desktop and logging in again may be sufficient.

(Why is this even needed?)

@TingPing
Copy link
Member

TingPing commented Dec 6, 2018

(Why is this even needed?)

It adds to $XDG_DATA_DIRS for desktop files to be picked up. This should probably be explained.

@probonopd
Copy link
Author

If it would use $HOME/.local/share/ then the log out and log in would not be needed?

@TingPing
Copy link
Member

TingPing commented Dec 6, 2018

It wouldn't but a core design of Flatpak is to never conflict with files provided elsewhere.

@probonopd
Copy link
Author

probonopd commented Dec 6, 2018

You are pointing out a deficiency of the current XDG desktop spec - it seems to assume that each application is available on the system in one version only. In AppImage, we are using an ugly workaround by using names such as /home/me/.local/share/applications/appimagekit_2cbea0c6c0e442620923ecddcaa1e87f-xchat.desktop in order to prevent collisions. Same goes for icon files and the like.

Would you like to collaborate on improving the XDG specs? (More on this topic)

@TingPing
Copy link
Member

TingPing commented Dec 6, 2018

Flatpak exports a few file types that are picked up via $XDG_DATA_DIRS such as icons, mime types, and dbus services. So improving the desktop-file-spec or doing ugly renaming cannot work in those cases.

@probonopd
Copy link
Author

The suggested desktop database (similar to Launch Services on the Mac) would do away with files altogether, like on the Mac. It would allow for multiple versions coming from multiple places to co-exist peacefully.

@TingPing
Copy link
Member

TingPing commented Dec 6, 2018

I can't speak for Flatpak but I believe such an invasive redesign of how applications work isn't the current goal.

@matthiasclasen
Copy link
Contributor

Reinventing the world to solve corner cases is going to be an uphiill battle.

And it will never work for all the distros that flatpak is targeting.

@probonopd
Copy link
Author

This was closed. Does this mean that it is no longer recommended to restart the system?

@matthiasclasen
Copy link
Contributor

matthiasclasen commented Jan 8, 2019

No, I think we want to keep the instructions simple and obvious. No need to go into details of how things are implemented.

The simple and obvious way to restart a session is to restart the system.

@probonopd
Copy link
Author

Only that on Live ISOs this completely breaks.

@matthiasclasen
Copy link
Contributor

True. But thats your special niche. We are not trying to provide setup instructions for live images.

notklaatu pushed a commit to notklaatu/flatpak.github.io that referenced this issue May 23, 2019
Shorten closing action item ("Now all you have to do is...") for clarity and to offset additional "log out" caveat.

Addresses issue 303 flatpak#303
@xandkar
Copy link

xandkar commented Oct 3, 2021

No, I think we want to keep the instructions simple and obvious. No need to go into details of how things are implemented.

The simple and obvious way to restart a session is to restart the system.

Not true. It makes it unnecessarily mysterious to anyone that has used non-Windows for more than 5 minutes.

@probonopd probonopd changed the title Do not recommand restarting the system if it is not needed Do not recommend restarting the system if it is not needed Oct 3, 2021
@probonopd
Copy link
Author

The simple and obvious way to restart a session is to restart the system.

For me, the simple and obvious way to restart a session is to log out and log in again.
Not?

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

4 participants