Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#457 closed Bug / Defect (fixed)

Building OpenVPN for Windows from Git using openvpn-build/generic fails

Reported by: Samuli Seppänen Owned by: Gert Döring
Priority: major Milestone:
Component: Building / Compiling Version: OpenVPN git master branch (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc:

Description

Building latest OpenVPN Git sources fails miserably:

socket.c: In function ‘socket_bind’:
socket.c:1120:7: warning: passing argument 4 of ‘setsockopt’ from
incompatible pointer type [enabled by default]
       if (setsockopt(sd, IPPROTO_IPV6, IPV6_V6ONLY, &v6only,
sizeof(v6only)))
       ^
In file included from ../../src/compat/compat.h:29:0,
                 from syshead.h:28,
                 from socket.c:31:
/usr/share/mingw-w64/include/winsock2.h:999:34: note: expected ‘const
char *’ but argument is of type ‘int *’
   WINSOCK_API_LINKAGE int WSAAPI setsockopt(SOCKET s,int level,int
optname,const char *optval,int optlen);
                                  ^
socket.c: In function ‘socket_recv_queue’:
socket.c:3098:45: error: ‘struct link_socket_addr’ has no member named
‘local’
    int addrlen = af_addr_size(sock->info.lsa->local.addr.sa.sa_family);
                                             ^
make[4]: *** [socket.o] Error 1

This is reproduceable on Ubuntu 14.04 64-bit configured using using the automated setup script.

On Ubuntu 12.04 that has an older mingw_w64 version there are additional issues, of which some (but not all) are fixed in the patched versions of NSIS packages.

Change History (5)

comment:1 Changed 6 years ago by Samuli Seppänen

I reproduced this by creating openvpn-2.3_git.tar.gz using make dist-gzip and putting that to a webserver. Then I simply overrode the build parameters:

OPENVPN_URL="http://build.openvpn.net/downloads/temp/thomas/openvpn-2.3_git.tar.gz"\
INSTALLER_VERSION="903"\
OPENVPN_VERSION="2.3_git"\
./build-complete

The build-snapshot script should produce the same results (provided it works).

comment:2 Changed 6 years ago by Gert Döring

I'll take it. We'll benefit a lot from being able to build master snapshots for windows again :-)

comment:3 Changed 6 years ago by Gert Döring

Owner: set to Gert Döring
Status: newaccepted

comment:4 Changed 6 years ago by Gert Döring

Resolution: fixed
Status: acceptedclosed

Now that was quite easily fixed, by letting d12fk and plaisthos do all the work :-)

Let's hope we'll see snapshot builds on the download area soon!

comment:5 Changed 6 years ago by Samuli Seppänen

The "automatically publish Windows snapshot builds" is next on my todo, so we don't have to wait too long.

Note: See TracTickets for help on using tickets.