2017-10-16 23:17 GMT+05:00 Selva <selva.n...@gmail.com>:
> Hi,
>
> On Mon, Oct 16, 2017 at 12:57 PM, Gert Doering <g...@greenie.muc.de>
> wrote:
>
>> Hi,
>>
>> On Mon, Oct 16, 2017 at 11:33:02AM -0400, Selva wrote:
>> > I did a quick test on ubuntu trusty (14.04.05 LTS), and as you pointed
>> > out, it fails while
>> > compiling block_dns.c because of some errors in the headers for the
>> > windows filtering platform API. All the bits are there so its possible
>> to
>> > build
>> > after tweaking the installed headers (only a couple of lines) but not
>> > something
>> > one would normally want to do.
>> >
>> > It does look like targeting 14.04 is no more relevant.
>>
>> I have some sort of mixed feedback from Samuli's buildbots here - it
>> *used* to compile just fine, but since the latest patch (removing the
>> "manual" dynamic linking stuff), it fails with
>>
>> openvpnserv-interactive.o:interactive.c:(.text+0xee2): undefined
>> reference to `InitializeUnicastIpAddressEntry'
>>
>> ... so it seems 14.04 mingw does not truly expose all the Vista API bits.
>>
>
> My bad, I missed that warning when tested on a 14.04 instance (it was a
> google compute engine
> with 14.04.5 LTS image with mingw-w64 3.1.0). I do now see that
> 'InitializeUnicastIpAddressEntry'
> is missing in there.
>
> As these buildbots used to succeed the other header issues I saw (an extra
> ifdef
> and a duplicate declaration) must be already patched in them.
>
> The function stub is there in the import library library
> (libiphlpapi.lib), so we could add
> VOID NETIOAPI_API_ InitializeUnicastIpAddressEntry
> (PMIB_UNICASTIPADDRESS_ROW Row);
> to interactive.c.
>
> But.
>
>
>> OTOH, we can just document "16.04 is what you need" - that would work
>> for me, as cross-building is really not "I have this <any breed of unix
>> OS> and want to build and run OpenVPN here!" building... (I have much
>> stricter requirements wrt breaking platforms for "run here" builds :) )
>>
>
> I would prefer that option and and we just require 16.04 or better from
> now on.
>
based on the above, I would say that adding configure check for
"InitializeUnicastIpAddressEntry" would be nice (and making configure
gently complain like "please use Ubuntu 16.04")
>
> Selva
>
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
