Bug 447445

Summary: appimage extension should be AppImage for desktop integrations
Product: [Applications] krita Reporter: Justin Garrison <justin>
Component: GeneralAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: ahab.greybeard, amy, anna.medonosova, halla, tomtomtomreportingin, vitamorus.art
Priority: NOR    
Version: 5.2.6   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Justin Garrison 2021-12-23 18:57:14 UTC
SUMMARY
***
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
***
The appimage extension should be capitalized so other AppImage tooling can work to keep it updated and integrate it with a users desktop.

STEPS TO REPRODUCE
1. Install appimaged via https://github.com/probonopd/go-appimage
2. Download krita .appimage file (integrations won't work)
3. rename file extension to .AppImage (desktop shortcut launcher will be created)

OBSERVED RESULT
Downloading the appimage file wouldn't let me launch it from my desktop search and a shortcut wasn't created. Renaming the extension made the integrations work.

EXPECTED RESULT
The app would be integrated into my desktop launcher like other app image files

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

ADDITIONAL INFORMATION
I really appreciate that you create an app image option because it's the easiest way for me to download and manage the application.
Comment 1 tomtomtomreportingin 2021-12-23 22:50:57 UTC
To be honest, I would consider this more of a problem of the program you linked than a problem of Krita... but I'm not a dev and this is likely a trivial fix.
Comment 2 tomtomtomreportingin 2021-12-23 23:01:22 UTC
Related issue: https://github.com/probonopd/go-appimage/issues/157
Comment 3 Ahab Greybeard 2021-12-24 05:47:15 UTC
When you download the krita appimage, you have to set Execution permissions in Properties and while you're doing that it would be a simple task to also change the name capitalisation if you want to.
Making desktop launchers or panel launchers or menu items is a simple task.

Also, there may be people who do have AppImage tooling who don't want krita integrated in this way, e.g. people who download and use many different versions for testing purposes, sometimes on a daily basis. They would be inconvenienced by that.

(I've tried using the probono appimages in the past and had to stop because I was concerned and annoyed by how they tried to integrate with my desktop.)
Comment 4 Justin Garrison 2021-12-24 07:48:57 UTC
(In reply to Ahab Greybeard from comment #3)
> When you download the krita appimage, you have to set Execution permissions
> in Properties and while you're doing that it would be a simple task to also
> change the name capitalisation if you want to.
> Making desktop launchers or panel launchers or menu items is a simple task.
> 
> Also, there may be people who do have AppImage tooling who don't want krita
> integrated in this way, e.g. people who download and use many different
> versions for testing purposes, sometimes on a daily basis. They would be
> inconvenienced by that.
> 
> (I've tried using the probono appimages in the past and had to stop because
> I was concerned and annoyed by how they tried to integrate with my desktop.)

I agree it would be simple but it may also cause other bugs because it is something that is unexpected from a user who uses app image files and wants to have them integrated into my desktop. The integration is provided from an optional daemon and not automatic.

Not having the extension properly capitalized has already caused two tickets (one here and one on the daemon) and I'm sure more will happen in the future.
Comment 5 amyspark 2022-01-03 22:54:28 UTC
Introduced in https://invent.kde.org/graphics/krita/-/commit/28ae5eceb65f6a3cf37a98f69aa50ca4d92c6c6a . Halla, Anna, what should we do here?
Comment 6 Vitamorus 2024-11-12 14:10:22 UTC
Re-confirmed. From a quick search online, using '.AppImage' seems to be the norm out there on the internet (including official appimages by QTractor, Kate, and Kdenlive). Chatted with dmitryK about it as well and it seems reverting back to .AppImage is in order.
Comment 7 Bug Janitor Service 2024-11-15 08:04:53 UTC
A possibly relevant merge request was started @ https://invent.kde.org/graphics/krita/-/merge_requests/2269
Comment 8 Bug Janitor Service 2024-11-15 08:25:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/websites/autoconfig-kde-org/-/merge_requests/48
Comment 9 Bug Janitor Service 2024-11-15 08:53:35 UTC
A possibly relevant merge request was started @ https://invent.kde.org/documentation/docs-krita-org/-/merge_requests/348
Comment 10 Dmitry Kazakov 2024-11-15 09:30:56 UTC
Git commit fea7aca2032fea7f4ea5b434e6d3cd916a5e324c by Dmitry Kazakov.
Committed on 15/11/2024 at 08:21.
Pushed by bcooksley into branch 'master'.

[krita] Add redirects for .AppImage packages as well

The related MR in Krita that renames the AppImage packages:
https://invent.kde.org/graphics/krita/-/merge_requests/2269

M  +11   -0    krita/.htaccess

https://invent.kde.org/websites/autoconfig-kde-org/-/commit/fea7aca2032fea7f4ea5b434e6d3cd916a5e324c
Comment 11 Dmitry Kazakov 2024-11-15 10:14:37 UTC
Git commit 4deb0e148faeeabc0fddde49ac790bd3a2245d58 by Dmitry Kazakov.
Committed on 15/11/2024 at 10:13.
Pushed by dkazakov into branch 'master'.

Change case of AppImage packages: .appimage -> .AppImage

M  +1    -1    build-tools/ci-scripts/upload-nightly-packages.py
M  +1    -1    krita/org.kde.krita.appdata.xml
M  +2    -2    packaging/linux/appimage/build-image.sh
D  +0    -44   packaging/linux/appimage/generate_zsync.sh
M  +1    -1    packaging/steam/INSTRUCTIONS.md
M  +2    -2    packaging/steam/launch.sh

https://invent.kde.org/graphics/krita/-/commit/4deb0e148faeeabc0fddde49ac790bd3a2245d58
Comment 12 Dmitry Kazakov 2024-11-15 10:17:25 UTC
Git commit bd14ad5b691ec93b404a5d63ac92de40ecd071e7 by Dmitry Kazakov.
Committed on 15/11/2024 at 10:17.
Pushed by dkazakov into branch 'krita/5.2'.

Change case of AppImage packages: .appimage -> .AppImage

M  +1    -1    build-tools/ci-scripts/upload-nightly-packages.py
M  +2    -2    packaging/linux/appimage/build-image.sh
D  +0    -44   packaging/linux/appimage/generate_zsync.sh
M  +1    -1    packaging/steam/INSTRUCTIONS.md
M  +2    -2    packaging/steam/launch.sh

https://invent.kde.org/graphics/krita/-/commit/bd14ad5b691ec93b404a5d63ac92de40ecd071e7
Comment 13 Dmitry Kazakov 2024-11-15 10:18:05 UTC
Git commit 147ef3252ba2d84eb8c93407641baf7de5edde3c by Dmitry Kazakov.
Committed on 15/11/2024 at 08:53.
Pushed by dkazakov into branch 'master'.

Add a note about appimage filename extension switch

M  +16   -0    untranslatable_pages/release_krita.rst

https://invent.kde.org/documentation/docs-krita-org/-/commit/147ef3252ba2d84eb8c93407641baf7de5edde3c