Bug 2284089
| Summary: | ImageMagick json output adds extra comma with 7.1.1-26 | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Mamoru TASAKA <mtasaka> |
| Component: | ImageMagick | Assignee: | Luya Tshimbalanga <luya_tfz> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | blaise, davide, epel-packagers-sig, extras-qa, fedora, luya_tfz, michel, ngompa13, pampelmuse, pvalena, releng, sergio |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | ImageMagick-7.1.1.33-1.fc40 ImageMagick-7.1.1.33-1.fc39 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 2261661 | Environment: | |
| Last Closed: | 2024-06-24 06:37:09 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 2259429 | ||
| Bug Blocks: | 2261661 | ||
|
Description
Mamoru TASAKA
2024-05-31 05:59:09 UTC
```
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/bca743becfa97355f3905eab6e68ecc8b91300d9
Small reproducer is, when you try `$ convert exif.jpg "json:" | grep exif:Flash`,
* with the previous commit https://github.com/ImageMagick/ImageMagick/commit/20fbdeaa82bc6b67ddebe85a46650b3906b77031
output was something like:
```
"exif:Flash": "0",
"exif:FlashPixVersion": "0100",
```
* with https://github.com/ImageMagick/ImageMagick/commit/bca743becfa97355f3905eab6e68ecc8b91300d9
```
"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/c4b3e141a764ff1f638aa5ac19033dc573451978
The above commit is included in -28.
FEDORA-2024-6d8a3f52a5 (ImageMagick-7.1.1.33-1.fc40) has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2024-6d8a3f52a5 FEDORA-2024-93ffff4464 (ImageMagick-7.1.1.33-1.fc39) has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2024-93ffff4464 FEDORA-2024-93ffff4464 has been pushed to the Fedora 39 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-93ffff4464` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-93ffff4464 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2024-6d8a3f52a5 has been pushed to the Fedora 40 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-6d8a3f52a5` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-6d8a3f52a5 See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2024-6d8a3f52a5 (ImageMagick-7.1.1.33-1.fc40) has been pushed to the Fedora 40 stable repository. If problem still persists, please make note of it in this bug report. FEDORA-2024-93ffff4464 (ImageMagick-7.1.1.33-1.fc39) has been pushed to the Fedora 39 stable repository. If problem still persists, please make note of it in this bug report. |