Bug 915214

Summary:	media-libs/exiftool: add brotli USE flag for JPEG XL metadata
Product:	Gentoo Linux	Reporter:	David Korth <gerbilsoft>
Component:	Current packages	Assignee:	Gentoo Perl team <perl>
Status:	UNCONFIRMED ---
Severity:	normal
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	exiftool-12.64: Add brotli USE flag exiftool-12.64: Add jpegxl USE flag

Description David Korth 2023-10-05 14:36:23 UTC

libjxl 0.8.1's 'cjxl' can losslessly convert a regular JPEG image to JPEG XL format. The resulting .jxl file has Exif metadata compressed using the Brotli algorithm.

exiftool supports reading Brotli-compressed Exif metadata, but it needs an additional Perl module installed. (dev-perl/IO-Compress-Brotli) This patch adds a brotli USE flag and adds dev-perl/IO-Compress-Brotli as a runtime dependency.

Reproducible: Always

Steps to Reproduce:
1. Convert a JPEG image to JPEG XL using `cjxl`.
2. Attempt to view metadata using `exiftool file.jxl`.
Actual Results:  
`exiftool` shows:
Warning : Install IO::Uncompress::Brotli to decode Brotli-compressed metadata

Expected Results:  
`exiftool` shows the JPEG XL metadata.

Comment 1 David Korth 2023-10-05 14:36:46 UTC

Created attachment 872189 [details, diff]
exiftool-12.64: Add brotli USE flag

Comment 2 Andreas Sturmlechner gentoo-dev

2023-10-06 07:19:41 UTC

It would seem very non-obvious by USE=brotli enabling JPEG XL features in packages.

USE=jpegxl is an established flag for that purpose.

Comment 3 David Korth 2023-10-06 11:09:01 UTC

Created attachment 872219 [details, diff]
exiftool-12.64: Add jpegxl USE flag

I used brotli as the USE flag because exiftool does support some JPEG XL functionality without it, but shows a warning that IO::Compress::Brotli is needed for full metadata spuport. Since it's mainly used for JPEG XL, it does make sense to rename it.

Attached is the same patch as before but with USE=jpegxl instead of USE=brotli.