<div dir="ltr"><div>1.<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">This patch makes no sense.<br></blockquote><span class="im"><a href="http://msdn.microsoft.com/en-us/library/windows/desktop/ms737931%28v=vs.85%29.aspx">http://msdn.microsoft.com/en-us/library/windows/desktop/ms737931%28v=vs.85%29.aspx</a><br></span>We should be using this and not one that comes with vlc sources <br></div><div>Needless to say duplication linker errors etc. vlc won't compile on trunk mingw-w64 without it.<br></div><div><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span class="im">If defines a macro that is not used anywhere.</span><br></blockquote>The author told me it is there for readability so that config.h describes the features that are and are not there.<br>If you don't want any unused defines I can change it to ])],[],[AC_LIBOBJ([freeaddrinfo])])<br>There is no issue there.<br><br></div><div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span class="im">I expect its only effect will be to break the OS/2 build.<span class="im"></span></span><br></blockquote><div>I agree with you that it would be a concern but<br>I highly doubt that it will as that program should build successfully for OS/2 still resulting in AC_LIBOBJ([freeaddrinfo])<br><br></div><div>He wrote it because vlc commit 3180272a56bdb3933219e1cc2c57d0285a7adbd broke building from mingw-w64<br></div><div><br>2.<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">There is already code for this, so this patch is either redundant or wrong.<span class="im"></span><br></blockquote><br><div>You are correct it is there already, I somehow missed this before.<br><br>You do have AC_CHECK_TYPES([struct pollfd]<br>but that comes with a<br># define _WIN32_WINNT 0x502 <br></div><div>which forces windows xp<br></div><div>I could submit a patch that removes this line if that is better suited then actually detecting it via compiling.<br></div><div>Would that be acceptable ??<br><br></div><div>3.<br></div></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span class="im">So fix mingw64? </span></blockquote><div> I actually submitteed a patch that was merged to create actual functions for the time functions over previous defines<br></div><div>It was merged but there has been a lenghty discussion since where they have opted for inline<br></div><div>I am not in charge of mingw-w64 so I can't change this.<br> <br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span class="im">No ways we had kilometers of copy paste code like this because they abuse inlines.</span></blockquote><div>Yes you have a list of 2 items like this in <a href="http://configure.ac">configure.ac</a> asprintf and vasprintf.<br></div><div>This is just prepended to that list and is a must or the linker goes absolutely crazy<br><br></div><div>Essentially I need this because I have a dx11 renderer that I want to submit but it can only be built on trunk.<br></div><div>As much as I agree with you on abuse of inlines this one is a must for building vlc with this :D<br><br></div><div>Kind Regards<br></div><div>Martell<br> </div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 13, 2015 at 7:33 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 2015-01-13 15:41, Martell Malone a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Attached are 3 patches that enhance building for windows with<br>
</span><a href="http://configure.ac" target="_blank">configure.ac</a> [1]<span class=""><br>
<br>
1. Search for freeaddrinfo in SOCKET_LIBS<br>
<br></span>
This patch was created by Ray Donnelly who I have CCd in this email.<br>
</blockquote>
<br>
This patch makes no sense. If defines a macro that is not used anywhere. I expect its only effect will be to break the OS/2 build.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2. Search for pollfd incase we are building for a target beyond<br>
windows xp.<br>
</blockquote>
<br></span>
There is already code for this, so this patch is either redundant or wrong.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
3. localtime_r and gmtime_r are now defined as inline in mingw64 and<br>
are behind a guard _POSIX this patch fixes their discovery.<br>
</blockquote>
<br></span>
So fix mingw64? No ways we had kilometers of copy paste code like this because they abuse inlines.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Rémi Denis-Courmont<br>
<br>
______________________________<u></u>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br></div>