[master 5d15dc1] fix windows missing inet_ntop missing symbol when built on >= 
vista but run on < vista

I just pushed the above patch.  We were doing build time decision on using the 
replacement function.  I changed it to just always use the replacement 
function.  If anyone has issues with this, we can always change to runtime 
detection if need be, but I would prefer to avoid the complication.

Mike

On May 11, 2010, at 12:03 PM, Anthony Minessale wrote:

> I started using that function in a recent patch to fix an ipv6 bug and did 
> not realize it did not exist on windows.  There are complete implementations 
> of that function we could just always load when we know we are on windows to 
> avoid this as apparently windows 7 actually has POSIX support.
> 
> on the other hand it may also be interesting to drop support for everything 
> else and get rid of some of the half ass emulation of many POSIX function 
> found in all the portability libs mwa ha ha.
> 
> BTW what is git bash?  GIT as a shell?  Makes me laugh cos I was joking about 
> that.
> 
> I call git shit with a g, a new system to learn 200 commands you are 
> unfamiliar with to replace the 200 unix command that are equally hard but you 
> already know.  But I am not the build master just the lowly developer so what 
> we C is what we GIT =/
>  
> 
> On Tue, May 11, 2010 at 10:48 AM, Jan Berger <jan.ber...@video24.no> wrote:
> I checked out 1.0.6 and that works, but I am not at head anymore. Part of the 
> issue here is that I am new on git, but took to senses and started using Git 
> Bash and that works just fine.
> 
>  
> What exactly is the change that break’s it?
> 
> From: freeswitch-dev-boun...@lists.freeswitch.org 
> [mailto:freeswitch-dev-boun...@lists.freeswitch.org] On Behalf Of Anthony 
> Minessale
> Sent: 11. mai 2010 17:05
> 
> 
> To: freeswitch-dev@lists.freeswitch.org
> Subject: Re: [Freeswitch-dev] ws2_32 "missing" inet_ntop
>  
> we can always force it in our code to use the fake one when its windows to 
> make it compat.
> 
> This function is only used in 1 place (to generate hex version of ipv6 addrs)
> 
>  
> We can live with a permanent substitute in the case of windows so the code is 
> portable both ways.
> 
>  
> On Tue, May 11, 2010 at 6:24 AM, Peter Olsson 
> <peter.ols...@visionutveckling.se> wrote:
> 
> Well, actually I'm building on Windows 7, but wanted it to work on XP, so I 
> made some manual hacks in switch_utils.c for this... But if building on XP it 
> should work with the existing code…
> 
> -----Ursprungligt meddelande-----
> Från: freeswitch-dev-boun...@lists.freeswitch.org 
> [mailto:freeswitch-dev-boun...@lists.freeswitch.org] För Jan Berger
> 
> Skickat: den 11 maj 2010 12:55
> 
> Till: freeswitch-dev@lists.freeswitch.org
> Ämne: Re: [Freeswitch-dev] ws2_32 "missing" inet_ntop
> 
> How do you check out the project using git?
> 
> I just did a "git clean -f -d -x" from git bash and it compiles fine
> afterwards - but I still get the ws2_32.dll inet_ntop not found issue.
> 
> Jan
> 
> -----Original Message-----
> From: freeswitch-dev-boun...@lists.freeswitch.org
> [mailto:freeswitch-dev-boun...@lists.freeswitch.org] On Behalf Of Peter
> Olsson
> Sent: 11. mai 2010 11:52
> To: 'freeswitch-dev@lists.freeswitch.org'
> Subject: Re: [Freeswitch-dev] ws2_32 "missing" inet_ntop
> 
> I compiled git head on Windows yesterday, and it worked just fine. Did you
> make sure to set autocrlf=false in the git config before checking out the
> files from the repository?
> 
> Another thing, it must be compiled on XP to support the internal inet_ntop.
> If compiled on Windows 7, and then installed on Windows XP it won't work.
> 
> /Peter
> 
> -----Ursprungligt meddelande-----
> Från: freeswitch-dev-boun...@lists.freeswitch.org
> [mailto:freeswitch-dev-boun...@lists.freeswitch.org] För Jan Berger
> Skickat: den 11 maj 2010 11:29
> Till: freeswitch-dev@lists.freeswitch.org
> Ämne: Re: [Freeswitch-dev] ws2_32 "missing" inet_ntop
> 
> Welcome to the club :)
> 
> It is a fix for this they say, so if you check out head and re-compile it
> should work. But, I gave up last night due to git issues - the head did not
> compile. Going to clean out and re-start now to see if I get anywhere.
> 
> Jan
> 
> -----Original Message-----
> From: freeswitch-dev-boun...@lists.freeswitch.org
> [mailto:freeswitch-dev-boun...@lists.freeswitch.org] On Behalf Of Tamas
> Cseke
> Sent: 11. mai 2010 10:18
> To: freeswitch-dev@lists.freeswitch.org
> Subject: Re: [Freeswitch-dev] ws2_32 "missing" inet_ntop
> 
> Hello,
> 
> It doesn't work on winXP with 2008 express release build.
> It compiles.
> but if I run FS it says "The procedure entry point inet_ntop cold not be
> located in the dynamic link library WS2_32.dll"
> 
> Regards,
> Tamas
> 
> 
> 
> Michael Jerris wrote:
> > This should be fixed now as well.  If it is not yet, let us know.
> >
> > Mike
> >
> > On May 10, 2010, at 12:56 PM, Anthony Minessale wrote:
> >
> >
> >> A patch was added that requires it.  So we had to add a work-around to
> support it in HEAD.
> >>
> >>
> >> On Mon, May 10, 2010 at 11:46 AM, Jan Berger <jan.ber...@video24.no>
> wrote:
> >> Hi,
> >>
> >>
> >> I have an error on Windows XP Professional that puzzle me - FreeSwitch
> compiles fine after downloading from GIT, but FreeSWITCH.dll generate an
> unusual bug claiming that inet_ntop is not found in Winsock2 (ws2_32.dll)
> >>
> >>
> >> What puzzle me is that I have a 1.0.5 source tree in parrallell that
> don't have this issue using the same dll, and I fail to find any relevant
> difference in the project settings or files.
> >>
> >>
> >> Does anyone recognise this issue?
> >>
> 

_______________________________________________
FreeSWITCH-dev mailing list
FreeSWITCH-dev@lists.freeswitch.org
http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
http://www.freeswitch.org

Reply via email to