Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 338651 - media-gfx/geeqie doesn't handle +xmp -exif correctly
Summary: media-gfx/geeqie doesn't handle +xmp -exif correctly
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-25 09:11 UTC by Corbin Simpson
Modified: 2010-10-05 20:43 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch for geeqie-1.0-r1.ebuild (geeqie-xmp-only-fix.patch,543 bytes, patch)
2010-09-25 09:12 UTC, Corbin Simpson
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Corbin Simpson 2010-09-25 09:11:44 UTC
If a user has xmp in his USE flags, but not exif, then geeqie doesn't build with exiv2 and so XMP support is not enabled. The attached patch fixes the issue for me.

Reproducible: Always

Steps to Reproduce:
1. Emerge geeqie with +xmp -exif in USE flags.
2. Run geeqie; go into preferences menu, try to enable XMP, notice the error message claiming geeqie wasn't built with exiv2 support.
3. ???

Actual Results:  
XMP doesn't work in geeqie.

Expected Results:  
XMP should work in geeqie when xmp is set in USE flags.

Only tested with geeqie-1.0-r1, although the other version in Portage probably has a similar bug. I didn't test because 1.0-r1 is newest and unmasked.
Comment 1 Corbin Simpson 2010-09-25 09:12:42 UTC
Created attachment 248588 [details, diff]
Patch for geeqie-1.0-r1.ebuild
Comment 2 Bernard Cafarelli gentoo-dev 2010-10-05 20:43:37 UTC
Thanks for the report! And nice catch, the ebuild did depend on exiv2 for +xmp -exif, but still disabled exif... Fixed in 1.0 in CVS, I removed the old beta at the same time