On Thu, Dec 20, 2012 at 05:02:05PM +0100, Fabian Groffen wrote: > On 20-12-2012 09:57:13 -0600, Derek Martin wrote: > > On Thu, Dec 20, 2012 at 12:55:07PM +0100, Fabian Groffen wrote: > > > On Solaris/OpenIndiana there is a libidn, and the headers are in > > > /usr/include/idn but they aren't exactly working like GNU libidn, it > > > seems. So check for the existence of the headers, before assuming they > > > are there (see the code, e.g. main.c, mutt_idna.h). > > > > Does this mean that on Solaris, the code (with or without your patch) > > is not using libidn? Do we need additional code to use IDN on > > Solaris? > > Yes. Actually, compilation fails if not explicitly given --disable-idn > to configure. I haven't looked into using Solaris' idn/*.h, because > including idn/idna.h fails. Apparently it requires -I/usr/include/idn > to resolve the idn_int.h it relies on. > > I guess I could try and see how far I can come with that later, if > that's preferred. (The configure flag mentions GNU idn, so I figured > one would just install GNU idn on Solaris.)
For Solaris 11 onward I believe the libidn found in the pkg:/library/libidn package is GNU idn. I do not know why the include files are in /usr/include/idn but as you point out it's easy enough to include -I/usr/include/idn to mutt's compile environment. -- Will Fiveash
