bug-gnu-libiconv
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug-gnu-libiconv] libiconv build system don't link windows resource fil


From: Roumen Petrov
Subject: [bug-gnu-libiconv] libiconv build system don't link windows resource files
Date: Thu, 18 Dec 2008 21:49:42 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.18) Gecko/20081114 SeaMonkey/1.1.13

What about to resolve issue ?

Roumen Petrov wrote:
Hi Bruno,

Michael start two mail threads:
1)
http://lists.gnu.org/archive/html/bug-gnu-libiconv/2008-11/msg00002.html
2)
http://lists.gnu.org/archive/html/bug-gnu-libiconv/2008-11/msg00012.html

The first (1) is with two parts a) for sed and resource file b) resource isn't linked. Since I think that sed problem (if I remember well) is fixed in CVS HEAD so I will not comment it in this email.
The patch for resource may be work (adding "-Wl,libiconv.res" solves it).

The second (2) is only for resource isn't linked. The patch from this thread is obsolete and is part from 1.12.

I check the reported "resource issue" and confirm that problem exist.
The CVS HEAD version (makefiles with resources) is not changed I to me problems is not resolved.


I would like to propose a solution based on libtool. To the email is attach file libiconv-1.12.patch-RC. This file is only to show how libtool can compile and link windows resources.

The libtool provide a macro AC_LIBTOOL_RC (v<2). This macro isn't well documented. It has to be called before AC_PROG_LIBTOOL. For libtool (v>=2+) it is replaced with LT_LANG([Windows Resource]) called after LT_INIT. This macro detect resource compilers (windres for mingw) and enable a libtool tag (--tag=RC) to work. For details search internet for LTRCCOMPILE author Charles Wilson. So the attached file follow libtool rules for windows resources.

Bruno what you think about this ?
If is acceptable I'm ready to propose a patch to CVS HEAD.

Roumen




reply via email to

[Prev in Thread] Current Thread [Next in Thread]