After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 617053 - nanohttp.c include resolv.h unconditionally
nanohttp.c include resolv.h unconditionally
Status: RESOLVED FIXED
Product: libxml2
Classification: Platform
Component: general
git master
Other other
: Normal normal
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2010-04-28 08:47 UTC by OBATA Akio
Modified: 2012-05-11 02:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch for nanohttp.c to include resolv.h conditionally (256 bytes, patch)
2010-04-28 08:47 UTC, OBATA Akio
none Details | Review

Description OBATA Akio 2010-04-28 08:47:54 UTC
Created attachment 159766 [details] [review]
patch for nanohttp.c to include resolv.h conditionally

In nanohttp.c of libxml2-2.7.7 (and head), resolv.h is included if SUPPORT_IP6 is defined.  It break building on resolv.h less platforms, at least my Interix-6.0.

attachment patch should resolve this issue.
Comment 1 André Klapper 2012-02-03 14:04:25 UTC
Comment on attachment 159766 [details] [review]
patch for nanohttp.c to include resolv.h conditionally

[Setting "patch" flag and correcting mime type so this can actually be queried for.]
Comment 2 Daniel Veillard 2012-05-11 02:11:30 UTC
Ah, right, the patch is correct but I found out in the current git head that
there is already

#ifdef HAVE_RESOLV_H
#ifdef HAVE_ARPA_NAMESER_H
#include <arpa/nameser.h>
#endif
#include <resolv.h>
#endif

  a few lines above, so the simplest is to remove the later inclusion
because if available that header was already included !

thanks,

Daniel