Cygwin compile error rev 1506

Bug #238450 reported by leppie
2
Affects Status Importance Assigned to Milestone
Ikarus Scheme
Fix Committed
Medium
Abdulaziz Ghuloum

Bug Description

:)

leppie@UBER-VISTA ~/ikarus.dev
$ make
make all-recursive
make[1]: Entering directory `/home/leppie/ikarus.dev'
Making all in src
make[2]: Entering directory `/home/leppie/ikarus.dev/src'
make all-am
make[3]: Entering directory `/home/leppie/ikarus.dev/src'
gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -DNDEBUG -O3 -Wall -m32 -MT ikarus-io.o -MD -MP -MF .deps/ikarus-io.Tpo -c -o ikarus-io.o ikarus-io.c
ikarus-io.c: In function `do_connect':
ikarus-io.c:142: error: `EAI_SYSTEM' undeclared (first use in this function)
ikarus-io.c:142: error: (Each undeclared identifier is reported only once
ikarus-io.c:142: error: for each function it appears in.)
make[3]: *** [ikarus-io.o] Error 1
make[3]: Leaving directory `/home/leppie/ikarus.dev/src'
make[2]: *** [all] Error 2
make[2]: Leaving directory `/home/leppie/ikarus.dev/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/leppie/ikarus.dev'
make: *** [all] Error 2

Related branches

Revision history for this message
Michael D. Adams (mdmkolbe) wrote :

What version of Cygwin are you using?

(Some preliminary poking shows that EAI_SYSTEM is part of the standards that Cygwin should be following, but that at least in 2002 it didn't. *Shame on cygwin*. I haven't verified whether recent versions have fixed that.)

Revision history for this message
Derick Eddington (derick-eddington) wrote :

EAI_SYSTEM is from #include <netdb.h> and looking at Cygwin's netdb.h at:

http://cygwin.com/cgi-bin/cvsweb.cgi/src/winsup/cygwin/include/netdb.h?rev=1.10&content-type=text/x-cvsweb-markup&cvsroot=src

it does have EAI_SYSTEM, but you must not be using that same header. But this doesn't really matter because:

Cygwin does not implement getaddrinfo and so Ikarus implements it's own. So we'll need to make Ikarus's getaddrinfo work with this. Maybe an #if (!HAVE_GETADDRINFO) around that EAI_SYSTEM switch.

Revision history for this message
Derick Eddington (derick-eddington) wrote :

That netdb.h I linked to does have getaddrinfo and its changelog mentions it being added two years ago, but http://cygwin.com/cygwin-api/std-notimpl.html includes getaddrinfo in the not implemented list.

Revision history for this message
leppie (leppie) wrote :

I am using the latest, I think rev 1500 still compiled.

Revision history for this message
Abdulaziz Ghuloum (aghuloum) wrote :

Added conditional #define for both EAI_SYSTEM and ECANCELED in rev 1509. It was not compiling on my cygwin installation either.

Changed in ikarus:
assignee: nobody → aghuloum
importance: Undecided → Medium
status: New → Fix Committed
Changed in ikarus:
milestone: none → 0.0.4
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.