[FFmpeg-cvslog] r9992 - in trunk: ffserver.c libavformat/os_support.h

Måns Rullgård mans
Sun Aug 12 01:54:15 CEST 2007


Ramiro Polla <ramiro at lisha.ufsc.br> writes:

> Michael Niedermayer wrote:
>> Hi
>>
>> On Sat, Aug 11, 2007 at 06:07:54PM -0300, Ramiro Polla wrote:
>>
>>> Could this patch be reviewed, please? MinGW is currently broken
>>> because of this.
>>>
>>
>> patch rejected
>>
>> a simple #define closesocket close
>> worked before, now it stopped working, you dont even explain why but just
>> want to add bloat to workaround it by adding an unneeded function ...
>> if it where a static inline in a header but its not
>>
>> and of cousrse must this closesocket() not be exported
>>
>> id say just revert the untested changes which caused this or fix the code
>> (maybe moving the #define closesocket would fix it?)
>>
>>
>
> Thanks for reviewing. After a closer inspection I figured that patch
> was even wronger. closesocket() was not being found because the check
> in configure wasn't suited for Windows.
>
> New simpler patch attached.
>
> Ramiro Polla
> Index: configure
> ===================================================================
> --- configure	(revision 10076)
> +++ configure	(working copy)
> @@ -1630,7 +1630,8 @@
>  check_func2 windows.h GetProcessTimes
>
>  check_func fork
> -check_func closesocket
> +check_func closesocket ||
> +check_func2 winsock2.h closesocket

Please indent the second line 4 spaces.  OK otherwise.

-- 
M?ns Rullg?rd
mans at mansr.com




More information about the ffmpeg-cvslog mailing list