Bug 482500 - Installing Okular from snap breaks home directory
Summary: Installing Okular from snap breaks home directory
Status: RESOLVED FIXED
Alias: None
Product: okular
Classification: Applications
Component: Comicbook backend (show other bugs)
Version: unspecified
Platform: Snap Linux
: NOR major
Target Milestone: ---
Assignee: Okular developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-03-05 22:22 UTC by rice_pout618
Modified: 2024-05-09 13:39 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
broken directories in terminal (77.33 KB, image/png)
2024-03-05 22:22 UTC, rice_pout618
Details
Video of the bug on a fresh Ubuntu 22.04 VM (1.66 MB, video/webm)
2024-04-11 08:51 UTC, Nils
Details

Note You need to log in before you can comment on or make changes to this bug.
Description rice_pout618 2024-03-05 22:22:50 UTC
Created attachment 166450 [details]
broken directories in terminal

SUMMARY
After installing Okular a bunch of simlinks in my home directory got broken
***
NOTE: If you are reporting a crash, please try to attach a backtrace with debug symbols.
See https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports
***


STEPS TO REPRODUCE
1. 
2. 
3. 

OBSERVED RESULT


EXPECTED RESULT


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Albert Astals Cid 2024-03-05 23:50:03 UTC
Scarlett?
Comment 4 Sune Vuorela 2024-04-09 12:51:36 UTC
(In reply to Scarlett Moore from comment #3)
> Fix is in once merged:
> https://github.com/canonical/snapcraft/pull/4698/commits/
> 221e23cc28896b1098ed4a0c646f00b4b61464f0

Scarlett: thanks

Though this report might be about the qt5 snap ?
Comment 5 Scarlett Moore 2024-04-09 14:19:09 UTC
The file actually resides in desktop/common which will be picked up by both the qt5 and qt6 varient.
Comment 6 Nils 2024-04-11 08:51:37 UTC
Created attachment 168380 [details]
Video of the bug on a fresh Ubuntu 22.04 VM

The bug only affects empty directories. The file in the Documents folder is still there after launching Okular.
Comment 7 Nils 2024-04-11 09:04:33 UTC
This seems to be the cause. Has already been fixed upstream but the fix is not part of any snap build yet.
https://github.com/sergio-costas/snapcraft-desktop-integration/blob/2b439f9/common/desktop-exports#L237
Comment 8 林博仁(Buo-ren, Lin) 2024-04-11 09:08:21 UTC
(In reply to Scarlett Moore from comment #3)
> Fix is in once merged:
> https://github.com/canonical/snapcraft/pull/4698/commits/
> 221e23cc28896b1098ed4a0c646f00b4b61464f0

The pull request seems to be merged now, could you (or other personnel that can do so) please rebuild and publish the Okular snap on the stable and candidate channel?  This bug, although doesn't do much damage, does indeed destroy user data and should be dealt with as soon as possible.
Comment 9 Scarlett Moore 2024-04-11 09:20:27 UTC
rebuilding now
Comment 10 Scarlett Moore 2024-04-11 09:54:57 UTC
Unfortunately due to some renames with qt6 this needs to be merged before this will rebuild.
https://invent.kde.org/graphics/okular/-/merge_requests/967
Comment 11 林博仁(Buo-ren, Lin) 2024-04-11 09:57:35 UTC
(In reply to Scarlett Moore from comment #10)
> Unfortunately due to some renames with qt6 this needs to be merged before
> this will rebuild.
> https://invent.kde.org/graphics/okular/-/merge_requests/967

I suggest temporarily unlisting the snap from the store to avoid accidental installs in the meantime.
Comment 12 Scarlett Moore 2024-04-11 14:27:19 UTC
(In reply to 林博仁(Buo-ren, Lin) from comment #11)
> (In reply to Scarlett Moore from comment #10)
> > Unfortunately due to some renames with qt6 this needs to be merged before
> > this will rebuild.
> > https://invent.kde.org/graphics/okular/-/merge_requests/967
> 
> I suggest temporarily unlisting the snap from the store to avoid accidental
> installs in the meantime.

done.
Comment 13 Scarlett Moore 2024-04-12 12:06:27 UTC
@Albert(In reply to Albert Astals Cid from comment #1)
> Scarlett?

Can you approve the merge request so I can get this fixed? Thanks
Comment 14 林博仁(Buo-ren, Lin) 2024-05-09 12:22:28 UTC
(In reply to Scarlett Moore from comment #13)
> @Albert(In reply to Albert Astals Cid from comment #1)
> Can you approve the merge request so I can get this fixed? Thanks

Hello, the merge request seems to be merged, can we proceed to fix this problem?
Comment 15 Scarlett Moore 2024-05-09 13:39:32 UTC
(In reply to 林博仁(Buo-ren, Lin) from comment #14)
> (In reply to Scarlett Moore from comment #13)
> > @Albert(In reply to Albert Astals Cid from comment #1)
> > Can you approve the merge request so I can get this fixed? Thanks
> 
> Hello, the merge request seems to be merged, can we proceed to fix this
> problem?

It has already been released to the stable channel.