Project

General

Profile

Feature #902

Make dependency to iconv optional

Added by Eric TRINH over 8 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Robin Mills
Category:
build
Target version:
0.25
Start date:
20 May 2013
Due date:
% Done:

100%

Estimated time:

Description

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.

Thanks,
Eric

Files

exiv2_convert.patch (6.71 KB) exiv2_convert.patch Proposition patch to use internal charset encoding Eric TRINH, 20 May 2013 01:51

History

#1

Updated by Robin Mills over 8 years ago

  • Status changed from New to Assigned
  • Assignee set to Robin Mills
  • Target version set to 0.24

Eric

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.

Robin

#2

Updated by Daniel Kaneider over 8 years ago

Hi,

you may also consider switching to the ICU library. It supports a lot of character sets, is well maintained, supported on various platforms and used by many projects. Moreover it has a nonrestrictive OS license.

Best,
Daniel

#3

Updated by Robin Mills over 8 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.

#4

Updated by Robin Mills over 8 years ago

  • Target version changed from 0.24 to 0.25

Deferred for 0.25.

#5

Updated by Robin Mills about 7 years ago

  • Status changed from Assigned to Resolved

I'm not going to undertake the work involved to do this.

#6

Updated by Robin Mills over 6 years ago

  • % Done changed from 0 to 100
#7

Updated by Andreas Huggel over 6 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF