After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734032 - Clicking on .png file looks at wrong MIME type image/x-apple-ios-png
Clicking on .png file looks at wrong MIME type image/x-apple-ios-png
Status: RESOLVED FIXED
Product: gnome-commander
Classification: Other
Component: application
1.4.x
Other Linux
: Normal normal
: 1.4
Assigned To: GNOME Commander maintainer(s)
GNOME Commander maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2014-07-31 09:56 UTC by Aleksander (Andy) Rožman
Modified: 2015-05-30 15:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Possible patch (7.29 KB, patch)
2015-05-27 15:40 UTC, Mamoru TASAKA
none Details | Review
Patch after review (7.09 KB, patch)
2015-05-30 13:19 UTC, Mamoru TASAKA
accepted-commit_now Details | Review

Description Aleksander (Andy) Rožman 2014-07-31 09:56:05 UTC
When I click on any .png image in gnome commander, I always get this error:

No default application found for the MIME type image/x-apple-ios-png.

When clicking on .png MIME type of image/png should be used and not image/x-apple-ios-png

I get this error on 1.2.x version (originally installed with Ubuntu 14.04 (same problem on 13.04 LTS)) and on new version 1.4.3 (manually installed in hope that this problem was already fixed.
Comment 1 John Duchek 2014-10-24 14:27:36 UTC
This has been bugging me in Fedora 20 for the last year.  There appears to be no way to manually correct it.  (I went through the instructions on changing the defaults manually on the gnome-commander.org site and that does not work.  Not being able to automatically load PNG files is obnoxious.
Comment 2 Uwe Scholz 2014-10-26 21:16:46 UTC
Hi,

just to inform you: this is a known issue. Currently I'm out of time to solve this but it is still on my agenda.

Best,
Uwe
Comment 3 f38fb49e 2014-12-08 09:56:04 UTC
Hi,

just to tell that I'm experiencing the same problem on Ubuntu 14.10

Best,
Edgar
Comment 4 Uwe Scholz 2014-12-08 21:05:03 UTC
As a workaround by now please refer to the answer given here:
http://askubuntu.com/questions/501177/14-04-mime-type-png

Best
Uwe
Comment 5 Mamoru TASAKA 2015-05-27 15:40:40 UTC
Created attachment 304081 [details] [review]
Possible patch

I don't know if this is still in wishlist, however if it is still so
one of the easiest fix is to install own (gnome-commander specific,
not seem by other application by default) mime xml file and add
it to the search path. Patch is attached.

On Fedora this is reproducible on F-20 (will be end-of-life in 1 month),
F-21 (will be supported for 7 months). Note that F-22 does not reproduce
this, because shared-mime-info 0.14 removed x-apple-ios-png entry
(Fedora 20 uses 0.13)

http://cgit.freedesktop.org/xdg/shared-mime-info/commit/?id=f8ed2998bffe481ab267d20dc2b641acbffbf021
Comment 6 Mamoru TASAKA 2015-05-27 15:46:50 UTC
The above is for gcmd-1-4 branch
Comment 7 Uwe Scholz 2015-05-30 12:05:32 UTC
Review of attachment 304081 [details] [review]:

Thanks for this bugfix. I can confirm that it fixes the issue with the png-files using shared-mime-info < 1.4. One thing: In data/Makefile.am:

for target in $(DESTDIR)/$(mimetopdir)/* ; \
    test "x$${target}" != "x$(DESTDIR)/$(mimexmldir)" && rm -f $${target} ; done

should be

for target in $(DESTDIR)/$(mimetopdir)/* ; \
    test "x$${target}" != "x$(DESTDIR)/$(mimexmldir)" && rm -rf $${target} ; done
                                                            ^^^^
instead. Otherwise, "make distcheck" complains:

ERROR: files left after uninstall:
./share/gnome-commander/mime/image/png.xml
Comment 8 Mamoru TASAKA 2015-05-30 13:04:37 UTC
Ah, actually on my system update-mime-database creates mime/image/png.xml.
Thank you for review.
Comment 9 Mamoru TASAKA 2015-05-30 13:19:22 UTC
Created attachment 304308 [details] [review]
Patch after review

Patch take 2

Changed to rm -rf .
Also, src/Makefile.am does not use HAVE_UPDATE_MIME_DATABASE (this is
used in data/Makefile.am) so I deleted this line from src/Makefile.am.
Comment 10 Uwe Scholz 2015-05-30 14:35:54 UTC
Review of attachment 304308 [details] [review]:

Great work, thank you!