[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 caulier.gil...@gmail.com changed: What|Removed |Added Resolution|--- |FIXED Status|REPORTED|RESOLVED Latest Commit||https://invent.kde.org/grap ||hics/digikam/commit/f490d0c ||7b62454a4935754be87e84985f0 ||296115 Version Fixed In||8.0.0 --- Comment #9 from caulier.gil...@gmail.com --- Git commit f490d0c7b62454a4935754be87e84985f0296115 by Gilles Caulier. Committed on 10/04/2022 at 03:42. Pushed by cgilles into branch 'master'. ExifTool Support: Add new option to write metadata to files using ExifTool nackend instead Exiv2. This option replace all calls to Exiv2 to patch files by ExifTool as well. This will prevent to corrupt files due to lack of stability support from Exiv2. This includes large XMP section in JPEG and MPO files. For RAW and DNG files 2 options still here but are depends of ExifTool availability. We will never write at all to DNG or RAW files with Exiv2. This new option to replace Exiv2 by ExifTool is turned off by default for the moment, but if no side-effects is discovered in the future before official 8.0.0 release, this option will be turned on by default to be safe in all cases. Related: bug 325458 FIXED-IN: 8.0.0 M +10 -8NEWS M +1-0core/libs/metadataengine/dmetadata/dmetadata.cpp M +0-63 core/libs/metadataengine/dmetadata/dmetadata_exiftool.cpp M +2-48 core/libs/metadataengine/dmetadata/dmetadata_fileio.cpp M +14 -4core/libs/metadataengine/engine/metaengine.cpp M +13 -4core/libs/metadataengine/engine/metaengine.h M +171 -30 core/libs/metadataengine/engine/metaengine_p.cpp M +9-2core/libs/metadataengine/engine/metaengine_p.h M +1-1core/libs/metadataengine/engine/metaenginesettings.cpp M +65 -59 core/libs/metadataengine/engine/metaenginesettingscontainer.cpp M +1-0core/libs/metadataengine/engine/metaenginesettingscontainer.h M +36 -31 core/libs/widgets/metadata/exiftool/exiftoolconfpanel.cpp M +8-0core/libs/widgets/metadata/exiftool/exiftoolconfpanel.h M +3-0core/showfoto/setup/showfotosetupmetadata.cpp M +16 -30 core/utilities/setup/metadata/setupmetadata.cpp M +2-1core/utilities/setup/metadata/setupmetadata.h M +48 -17 core/utilities/setup/metadata/setupmetadata_behavior.cpp M +5-1core/utilities/setup/metadata/setupmetadata_p.cpp M +4-0core/utilities/setup/metadata/setupmetadata_p.h https://invent.kde.org/graphics/digikam/commit/f490d0c7b62454a4935754be87e84985f0296115 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 --- Comment #8 from caulier.gil...@gmail.com --- Git commit 3a672f38cda7738356f19707a6a634ca91c269e6 by Gilles Caulier. Committed on 07/04/2022 at 04:45. Pushed by cgilles into branch 'master'. Write metadata with ExifTool backend if Exiv2 fails to process, for exemple with RAW files or Video files. Tested successfuly with a MP4 video to insert a comment. For RAW and other king of files as MPO or JPEG with more than 64kB of Exif segement, Exiv2 must be bannned to uses in cases of writting operations as files can be corrupted. This require more regression tests to validate ExifTool uses in place of Exiv2. Related: bug 406540, bug 416516, bug 237504, bug 384092, bug 264210, bug 326408, bug 421464, bug 134486, bug 309341, bug 219856, bug 377622, bug 325458, bug 170693, bug 338075, bug 436876, bug 366348 M +1-0core/libs/metadataengine/dmetadata/dmetadata.h M +43 -0core/libs/metadataengine/dmetadata/dmetadata_exiftool.cpp M +49 -3core/libs/metadataengine/dmetadata/dmetadata_fileio.cpp M +2-2core/libs/metadataengine/exiftool/exiftoolparser.h https://invent.kde.org/graphics/digikam/commit/3a672f38cda7738356f19707a6a634ca91c269e6 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 --- Comment #7 from José Oliver-Didier --- Good question. From https://github.com/Exiv2/exiv2/issues/396 : "tvisitor.cpp which is a stripped down version of Exiv2. It handles JPEG>64k (and BigTiff and ISOBMFF and lots of other stuff not in Exiv2)." Seems that Exiv2 has not resolved it yet. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 --- Comment #6 from caulier.gil...@gmail.com --- If it have been solved why this issue exist ? -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 --- Comment #5 from José Oliver-Didier --- Excellent. Good to know. Seems like the exiv2 issue has not seen much activity in a while. Apparently it was resolved in tvisitor.cpp (https://github.com/Exiv2/exiv2/issues/396) -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 448729] Metadata is not written back to file when size of XMP JPEG segment is larger than 65535 bytes - Use ExifTool instead Exiv2
https://bugs.kde.org/show_bug.cgi?id=448729 caulier.gil...@gmail.com changed: What|Removed |Added Summary|Metadata is not written |Metadata is not written |back to file when size of |back to file when size of |XMP JPEG segment is larger |XMP JPEG segment is larger |than 65535 bytes|than 65535 bytes - Use ||ExifTool instead Exiv2 Component|Metadata-Xmp|Metadata-ExifTool --- Comment #4 from caulier.gil...@gmail.com --- Yes, digiKam in the future will use ExifTool with specific cases not supported by Exiv2. We have already few reports about these topics. The code is not yet finalized and needs regression tests. Gilles Caulier -- You are receiving this mail because: You are watching all bug changes.
