Drop functionality that depends on libraw features removed in 0.19
AbandonedPublic

Authored by arojas on Aug 28 2018, 9:37 PM.

Details

Summary

Drop functions and API that depend on removed libraw features. The soversion was intended to be set to '30.0.0', quite an unusual value. This didn't work due to a typo, so in practice the soversion was 5 (the major lib version). Bump it to 6 to account for the removed API.

Patch based on digikam commit f5bc69c288dda2509dbdfd9c3c54cd82d88c38ef

Test Plan

Builds with libraw 0.18 and 0.19. Gwenview, kdegraphics-thumbnailers and kphotoalbum build fine after the patch.

Diff Detail

Repository
R377 KDCraw Library
Lint
Lint Skipped
Unit
Unit Tests Skipped
arojas requested review of this revision.Aug 28 2018, 9:37 PM
arojas created this revision.
pino requested changes to this revision.Aug 28 2018, 9:52 PM
pino added a subscriber: pino.

No need to break the API/ABI of libkdcraw just because of changes in libraw -- just mark the public stuff affected as deprecated.
Regarding the internal usage: please keep the old code for older versions of libraw; see the libraw_version.h header (provided by libraw) with some defines & macros for this.

This revision now requires changes to proceed.Aug 28 2018, 9:52 PM
arojas abandoned this revision.Aug 29 2018, 5:04 PM

Alright. IMO it's cleaner to just drop API that is eventually no longer going to work (deprecated != not working), but in practice it won't make any difference since nothing uses this code.

I've opened D15149 to fix the soversion mess, I'd appreciate if someone could give it a quick look (I don't know if there's anybody currently in charge of this library - agateau is listed as maintainer who if I'm not mistaken is no longer active)