Project

General

Profile

Bug #654

libaudcore/charset.cc seems to ignore HAVE_ICONV macro.

Added by Carlo Bramini over 8 years ago. Updated over 8 years ago.

Status:
Rejected
Priority:
Minor
Assignee:
-
Category:
-
Target version:
-
Start date:
July 31, 2016
Due date:
% Done:

0%

Estimated time:
Affects version:

Description

I noticed that libaudcore/charset.cc ignores the result of test on ICONV presence made by configure script.
If there is not ICONV, the link stage will fails because iconv_open, iconv_close and iconv symbols are undefined.
I can just install that library to bypass this, but I'm wondering if libaudcore/charset.cc should be fixed to handle that missing capability or the configure script should halt if it is not found, saying that ICONV is not an option but a requirement.

Sincerely.

History

#1 Updated by John Lindgren over 8 years ago

iconv is a hard requirement. If you have a patch to make ./configure halt when it's not found, I can apply it, but I have no systems lacking iconv to test on.

#2 Updated by John Lindgren over 8 years ago

  • Status changed from New to Rejected

Not a bug since iconv() is in POSIX, which is our target platform. (Windows is an exception and only supported if you exactly follow the instructions we provide.)

#3 Updated by Carlo Bramini over 8 years ago

John Lindgren wrote:

Not a bug since iconv() is in POSIX, which is our target platform. (Windows is an exception and only supported if you exactly follow the instructions we provide.)

I agree, actually the solution is simply to install libiconv or winiconv because, as you wrote, it is a requirement and not an option.

Also available in: Atom PDF