Fix for compiling error with cygwin #2

Merged
merged 1 commit into from Feb 10, 2012

Conversation

Projects
None yet
2 participants
@das-peter
Contributor

das-peter commented Feb 10, 2012

Just made it, to compile this under cygwin yay
But to do so, I had to adjust the make file slightly.
I've added a comment to the commit how / why: das-peter/varnish-geoip@4e88987#commitcomment-954054

Thank you very much for this handy tool & best regards
Peter

@das-peter

This comment has been minimized.

Show comment
Hide comment
@das-peter

das-peter Feb 10, 2012

Owner

If you try to compile this using cygwin you might get an error like this:

$ make
cc -Wall -Wno-unused -pedantic  -lGeoIP -o geoip geoip.c
/tmp/ccmbUZq8.o:geoip.c:(.text+0x1f): undefined reference to `_GeoIP_open_type'
/tmp/ccmbUZq8.o:geoip.c:(.text+0x8e): undefined reference to `_GeoIP_record_by_addr'
/tmp/ccmbUZq8.o:geoip.c:(.text+0x179): undefined reference to `_GeoIP_record_by_addr'
collect2: ld returned 1 exit status
Makefile:8: recipe for target `geoip' failed
make: *** [geoip] Error 1

With the above change the error was gone.
Frankly speaking I've no clue why, mainly because I've no clue about compiling at all =)
The following article encouraged me to give the change above a shot: http://lists.opensuse.org/opensuse-buildservice/2009-12/msg00173.html

Owner

das-peter commented on 4e88987 Feb 10, 2012

If you try to compile this using cygwin you might get an error like this:

$ make
cc -Wall -Wno-unused -pedantic  -lGeoIP -o geoip geoip.c
/tmp/ccmbUZq8.o:geoip.c:(.text+0x1f): undefined reference to `_GeoIP_open_type'
/tmp/ccmbUZq8.o:geoip.c:(.text+0x8e): undefined reference to `_GeoIP_record_by_addr'
/tmp/ccmbUZq8.o:geoip.c:(.text+0x179): undefined reference to `_GeoIP_record_by_addr'
collect2: ld returned 1 exit status
Makefile:8: recipe for target `geoip' failed
make: *** [geoip] Error 1

With the above change the error was gone.
Frankly speaking I've no clue why, mainly because I've no clue about compiling at all =)
The following article encouraged me to give the change above a shot: http://lists.opensuse.org/opensuse-buildservice/2009-12/msg00173.html

cosimo added a commit that referenced this pull request Feb 10, 2012

Merge pull request #2 from das-peter/master
Fix for compiling error with cygwin

@cosimo cosimo merged commit d3753d4 into cosimo:master Feb 10, 2012

@cosimo

This comment has been minimized.

Show comment
Hide comment
@cosimo

cosimo Feb 10, 2012

Owner

Thanks! Appreciated.

Are you able to run varnish on cygwin? That's interesting.

Owner

cosimo commented Feb 10, 2012

Thanks! Appreciated.

Are you able to run varnish on cygwin? That's interesting.

@das-peter

This comment has been minimized.

Show comment
Hide comment
@das-peter

das-peter Feb 11, 2012

Contributor

Wow, nice reaction time :)

There are some special steps documented about how to use varnish with cygwin (and even compile it): https://www.varnish-cache.org/trac/wiki/VarnishOnCygwinWindows
So far it looks like everything works.

Contributor

das-peter commented Feb 11, 2012

Wow, nice reaction time :)

There are some special steps documented about how to use varnish with cygwin (and even compile it): https://www.varnish-cache.org/trac/wiki/VarnishOnCygwinWindows
So far it looks like everything works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment