Problem building GnuTLS 2.2.2 on Solaris

[Simon Josefsson]

Hi Brian,

I'm about to release 2.2.2 this week, so if you want this change to make
it, we need the answers to the questions below.

Besides, any C++ programmers who can confirm that it is a good idea to
include C headers like stdio.h and string.h in C++ header files?

/Simon

Simon Josefsson <simon at josefsson.org> writes:

> Brian Cameron <Brian.Cameron at Sun.COM> writes:
>
>> To build on Solaris, it is necessary for the
>> includes/gnutls/gnutlsxx.h file to add the following includes:
>>
>> #include <stdio.h>
>> #include <string.h>
>>
>> I am attaching a patch to add these, can this go upstream?
>
> Yes, I think we can solve this for the upcoming gnutls 2.2.3.
>
> However, what error message do you get?
>
> I thought stdio.h + string.h wasn't typically needed for C++.  Possibly
> there is some other header file we should include instead.
>
> Thanks,
> Simon
>
>> Thanks,
>>
>> Brian
>> --- gnutls-2.2.2/includes/gnutls/gnutlsxx.h.old	2008-04-16 09:46:56.220441725 -0400
>> +++ gnutls-2.2.2/includes/gnutls/gnutlsxx.h	2008-04-16 09:46:47.120145257 -0400
>> @@ -1,6 +1,8 @@
>>  #ifndef GNUTLSXX_H
>>  # define GNUTLSXX_H
>>  
>> +#include <stdio.h>
>> +#include <string.h>
>>  #include <exception>
>>  #include <vector>
>>  #include <gnutls/gnutls.h>
>> _______________________________________________
>> Gnutls-devel mailing list
>> Gnutls-devel at gnu.org
>> http://lists.gnu.org/mailman/listinfo/gnutls-devel