Make dependency to iconv optional
Due to licensing issue with iconv and Commercial license. It would be a good feature to make iconv optional.
Like Windows fix, which use Win32 function to handle conversion.
It seems that Win32 implementation only handle 3 conversions: utf8 <-> ucs2 and iso 8859-1 -> utf8 which remains basic.
It would be nice to have a iconv-less compilation dependency with only these 3 conversions.
I have included a small patch we use to handle these conversions, but we have not yet fully tested. It is only for informational purpose.
Updated by Robin Mills about 6 years ago
- Status changed from New to Assigned
- Assignee set to Robin Mills
- Target version set to 0.24
Thanks for suggesting this, and for sending the patch. I have rather a lot on my plate at the moment, and I'd like to defer this for a couple of months. Although you're welcome to persuade my that this is urgent and low risk.
Updated by Robin Mills almost 6 years ago
Thanks, Daniel. I'll have a look at this. I'm on sabbatical at the moment and mentoring a student on the Google Summer of Code. I don't want to say when I'll investigate this, however I will. Apart from dealing with the student, my next priority is to publish our 0.24 release (which is overdue since December 2012). We have 2 students this year and Abhinav is mentoring one of them. However I want to see 0.24 complete so that we can integrate the students' code before they return to college in September.
So there's a lot of Exiv2 work being done over the summer. Once that lots done, I'll have a look at ICU - oh and better CMake support (for you and DigiKam).
Naturally, I'm hoping the students will continue as team members after they return to college and pick up some tasks.