Hi,

On Mon, May 23, 2016 at 11:42 PM, Thorsten Stettin
<thorsten.stet...@gmail.com> wrote:
> Am 23.05.2016 um 23:07 schrieb C R:
>>
>> Yes, if "save comment" is unchecked it saves fine.
>> I never touch the comment, so it's strange that there would be something
>> invalid in it.
>>
>> Thanks for looking into this.
>> Maybe we should create a bug report?
>
>
> You have precedence. You know, I'm just a packager. :-P
>
> In German: Bitte nach Ihnen.
>
> But seriously: From where you got this picture? Please explain the entire
> workflow from your camera.
>
> IMHO information like this is very important in order to handle issues like
> that:

Well from my point of view, it does not matter where the broken
metadata comes from, as long as it is not generated by GIMP (in which
case, yes it matters a lot! GIMP should not be able to create files
with broken metadata). Of course, if that's another Free Software
which created the metadata, it would be cool to report a bug there,
but that's still not an issue in GIMP.

The only issue I see in GIMP here is that broken metadata should not
be an export-stopper since it's a data bug which is easily catchable
and bypassed (by removing the wrong metadata).

Jehan

> This comes from exiftooll. The whole thing is here.
>
> *exiftool ac_a4_document_holder_vintage_leather_brown_4..png**
> **ExifTool Version Number         : 10.10**
> **File Name                       :
> ac_a4_document_holder_vintage_leather_brown_4..png**
> **Directory                       : .**
> **File Size                       : 1631 kB**
> **File Modification Date/Time     : 2016:05:23 23:28:11+02:00**
> **File Access Date/Time           : 2016:05:23 23:29:26+02:00**
> **File Inode Change Date/Time     : 2016:05:23 23:28:27+02:00**
> **File Permissions                : rw-rw-r--**
> **File Type                       : PNG**
> **File Type Extension             : png**
> **MIME Type                       : image/png**
> **Image Width                     : 1000**
> **Image Height                    : 1000**
> **Bit Depth                       : 8**
> **Color Type                      : RGB with Alpha**
> **Compression                     : Deflate/Inflate**
> **Filter                          : Adaptive**
> **Interlace                       : Noninterlaced**
> **Exif Byte Order                 : Little-endian (Intel, II)**
> **Bits Per Sample                 : 8 8 8**
> **Image Description               : ..:\w�**
> **Make                            : samsung**
> **Camera Model Name               : SM-N910F**
> **Orientation                     : Horizontal (normal)**
> **X Resolution                    : 72**
> **Y Resolution                    : 72**
> **Resolution Unit                 : inches**
> **Software                        : GIMP 2.9.3**
> **Y Cb Cr Positioning             : Centered**
> **Exposure Time                   : 1/100**
> **F Number                        : 2.2**
> **Exposure Program                : Program AE**
> **ISO                             : 50**
> **Exif Version                    : 0220**
> **Date/Time Original              : 2016:04:08 09:51:31**
> **Create Date                     : 2016:04:08 09:51:31**
> **Components Configuration        : Y, Cb, Cr, -**
> **Shutter Speed Value             : 1/100**
> **Aperture Value                  : 2.2**
> **Brightness Value                : 4.61**
> **Exposure Compensation           : -1/2**
> **Max Aperture Value              : 2.2**
> **Metering Mode                   : Center-weighted average**
> **Light Source                    : Unknown**
> **Flash                           : No Flash**
> **Focal Length                    : 4.8 mm**
> **User Comment                    : ..:\w�**
> **Flashpix Version                : 0100**
> **Color Space                     : sRGB**
> **Exif Image Width                : 5312**
> **Exif Image Height               : 2988**
> **Sensing Method                  : One-chip color area**
> **Scene Type                      : Directly photographed**
> **Exposure Mode                   : Auto**
> **White Balance                   : Auto**
> **Focal Length In 35mm Format     : 31 mm**
> **Scene Capture Type              : Standard**
> **Image Unique ID                 : H16USHH04SA**
> **GPS Version ID                  : 2.2.0.0**
> **Photometric Interpretation      : YCbCr**
> **Samples Per Pixel               : 3**
> **Thumbnail Offset                : 884**
> **Thumbnail Length                : 14883**
> **Date Created                    : 2016:05:23**
> **Time Created                    : 07:46:21-07:46**
> **XMP Toolkit                     : XMP Core 4.4.0-Exiv2**
> **Format                          : image/png**
> **Background Color                : 255 255 255**
> **Pixels Per Unit X               : 2835**
> **Pixels Per Unit Y               : 2835**
> **Pixel Units                     : meters**
> **Modify Date                     : 2016:05:23 06:46:23**
> **Aperture                        : 2.2**
> **Date/Time Created               : 2016:05:23 07:46:21-07:46**
> **Image Size                      : 1000x1000**
> **Megapixels                      : 1.0**
> **Scale Factor To 35 mm Equivalent: 6.5**
> **Shutter Speed                   : 1/100**
> **Thumbnail Image                 : (Binary data 14883 bytes, use -b option
> to extract)**
> **Circle Of Confusion             : 0.005 mm**
> **Field Of View                   : 60.3 deg**
> **Focal Length                    : 4.8 mm (35 mm equivalent: 31.0 mm)**
> **Hyperfocal Distance             : 2.25 m**
> **Light Value                     : 9.9**
> **
> **.**
> *
>>
>>
>> -C
>>
>> On Mon, May 23, 2016 at 6:59 PM, Thorsten Stettin
>> <thorsten.stet...@gmail.com> wrote:
>>
>>     Am 23.05.2016 um 15:00 schrieb C R:
>>
>>         Happens every time with this file:
>>
>> http://www.whitecatgraphics.com/opendesignstudio/gimp/samples/ac_a4_document_holder_vintage_leather_brown_4.png
>>
>>         steps to reproduce:
>>         1.open the downloaded file in gimp
>>         2.export as "ac_a4_document_holder_vintage_leather_brown_4a.png"
>>         3.these boxes in png export are checked: "save background
>>         colour", and all checkboxes after and including Save
>>         resolution, compression level 9, and everything under advanced
>>         checked.
>>
>>
>>     Hi
>>
>>     At home in Xenial I exported your picture with save comment.
>>     The above error occurred.
>>     Hmm.
>>
>>     I took a look at the comment field exactly the
>>     exif:ImageDescription: It contains an invalid UTF-8 string.
>>
>>     *identify -verbose
>>     ac_a4_document_holder_vintage_leather_brown_4.png | grep -i desc **
>>     **    exif:ImageDescription: ..:\w.*
>>
>>
>>     After replacing the picture comment content with Gimp's standard
>>     one - valid UTF-8, I think - I could export your picture without
>>     any problem.
>>
>>     *identify -verbose
>>     ac_a4_document_holder_vintage_leather_brown_4a.png | grep -i desc
>>         exif:ImageDescription: Created with GIMP*
>>
>>     I guess it's a bug regarding the comment string and/or character
>>     set handling. Maybe a wrong character set was using in the
>>     exif:ImageDescription field, I assume.
>>
>>
>>     Greetings
>>
>>     Thorsten
>>
>>
>>         On Mon, May 23, 2016 at 11:24 AM, Thorsten Stettin
>>         <thorsten.stet...@gmail.com <mailto:thorsten.stet...@gmail.com
>>         <mailto:thorsten.stet...@gmail.com>>> wrote:
>>
>>             Am 23.05.2016 um 08:44 schrieb C R:
>>             > Fails on export with a Plug-in crash:
>>             >
>>             > "Procedure 'file-png-save-defaults' returned no return
>>         values
>>             >
>>             >
>>             > Strangely, if you save as a jpeg first, it exports to
>>         png just fine.
>>             >
>>             > Steps to reproduce:
>>             > 1. Open File
>>             > 2. Export to png
>>             >
>>             > Results in the above crash.
>>             >
>>             > Ver: GIMP 2.9.3
>>             > Ubuntu 16.04 with gnome-shell 3.18.4
>>             >
>>             > -C
>>             Under Trusty I can't reproduce that behavior. :-(
>>
>>             gimp Version 2.9.3.81-0t0~ppa
>>             libgegl-0.3-0 1:0.3.6.20-0t0~ppa
>>
>>             Cheers
>>
>>             > _______________________________________________
>>             > gimp-developer-list mailing list
>>             > List address: gimp-developer-list@gnome.org
>>         <mailto:gimp-developer-list@gnome.org>
>>             <mailto:gimp-developer-list@gnome.org
>>         <mailto:gimp-developer-list@gnome.org>>
>>             > List membership:
>>         https://mail.gnome.org/mailman/listinfo/gimp-developer-list
>>             > List archives:
>>         https://mail.gnome.org/archives/gimp-developer-list
>>
>>
>>             _______________________________________________
>>             gimp-developer-list mailing list
>>             List address: gimp-developer-list@gnome.org
>>         <mailto:gimp-developer-list@gnome.org>
>>             <mailto:gimp-developer-list@gnome.org
>>
>>         <mailto:gimp-developer-list@gnome.org>>
>>             List membership:
>>         https://mail.gnome.org/mailman/listinfo/gimp-developer-list
>>             List archives:
>>         https://mail.gnome.org/archives/gimp-developer-list
>>
>>
>>
>>
>>     --     Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu
>>     schaffen.
>>     Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege.
>>     Der eine ist die Liebe, der zweite ist die Genügsamkeit, der
>>     dritte ist die Demut.
>>     Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur
>>     der Demütige ist fähig zu herrschen.
>>
>>     _______________________________________________
>>     gimp-developer-list mailing list
>>     List address: gimp-developer-list@gnome.org
>>     <mailto:gimp-developer-list@gnome.org>
>>     List membership:
>>     https://mail.gnome.org/mailman/listinfo/gimp-developer-list
>>     List archives: https://mail.gnome.org/archives/gimp-developer-list
>>
>>
>
>
> --
> Lao-Tse sagt: Nichtstun ist besser, als mit viel Mühe nichts zu schaffen.
> Und er sagt auch: Ich habe drei Schätze, die ich hüte und hege.
> Der eine ist die Liebe, der zweite ist die Genügsamkeit, der dritte ist die
> Demut.
> Nur der Liebende ist mutig, nur der Genügsame ist großzügig, nur der
> Demütige ist fähig zu herrschen.
>
> _______________________________________________
> gimp-developer-list mailing list
> List address:    gimp-developer-list@gnome.org
> List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list
> List archives:   https://mail.gnome.org/archives/gimp-developer-list



-- 
ZeMarmot open animation film
http://film.zemarmot.net
Patreon: https://patreon.com/zemarmot
_______________________________________________
gimp-developer-list mailing list
List address:    gimp-developer-list@gnome.org
List membership: https://mail.gnome.org/mailman/listinfo/gimp-developer-list
List archives:   https://mail.gnome.org/archives/gimp-developer-list

Reply via email to