https://bugzilla.redhat.com/show_bug.cgi?id=2284089

Mamoru TASAKA mtasaka@fedoraproject.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Component|rubygem-mini_magick |ImageMagick Doc Type|--- |If docs needed, set a value Assignee|pvalena@redhat.com |luya_tfz@thefinalzone.net CC| |blaise@gmail.com, | |davide@cavalca.name, | |epel-packagers-sig@lists.fe | |doraproject.org, | |fedora@famillecollet.com, | |luya_tfz@thefinalzone.net, | |michel@michel-slm.name, | |ngompa13@gmail.com, | |pampelmuse@gmx.at, | |sergio@serjux.com

--- Comment #1 from Mamoru TASAKA mtasaka@fedoraproject.org --- ``` Failures: 1) With ImageMagick MiniMagick::Image#[] reads exif Failure/Error: expect(subject["EXIF:Flash"]).to eq "0" expected: "0" got: "0, " ```

This test is okay with ImageMagick 7.1.1-15, broken with -26, starting with this commit: https://github.com/ImageMagick/ImageMagick/commit/bca743becfa97355f3905eab6e...

Small reproducer is, when you try `$ convert exif.jpg "json:" | grep exif:Flash`,

* with the previous commit https://github.com/ImageMagick/ImageMagick/commit/20fbdeaa82bc6b67ddebe85a46... output was something like: ``` "exif:Flash": "0", "exif:FlashPixVersion": "0100", ```

* with https://github.com/ImageMagick/ImageMagick/commit/bca743becfa97355f3905eab6e... ``` "exif:Flash": "0, ", "exif:FlashPixVersion": "0100", ```

You can see extra comma to the output, even with 7.1.1-26.

After some additional refactoring and etc, the above issue is fixed with https://github.com/ImageMagick/ImageMagick/commit/c4b3e141a764ff1f638aa5ac19...

The above commit is included in -28.