ubuntu / snapcraft-desktop-helpers

Various launchers for snapcraft wiki parts
MIT License
73 stars 60 forks source link

Fix Documents, Pictures, Music etc are symlinks in #220

Closed lucyllewy closed 3 years ago

lucyllewy commented 3 years ago

When a user has one or several XDG_SPECIAL_DIRS folders symlinked in their real home folder to elsewhere in their real home folder common/desktop-exports will attempt to mv those folders to themselves.

While this is not a fatal error here, the electron-builder copies are using bash -e so they fail hard. I will forward these changes to electron-builder once reviewed here... (currently symlinks for Documents etc, combined with the electron-builder default of plugging home, cause electron-builder-built snaps to refuse to launch because the mv calls break the execution due to bash -e.)

Signed-off-by: Daniel Llewellyn diddledan@ubuntu.com

kenvandine commented 3 years ago

This looks sensible. Can you also propose these changes to the snapcraft extensions?

lucyllewy commented 3 years ago

willdo