Opened 14 years ago
Closed 13 years ago
#137 closed Bug / Defect ( fixed )
Visual Studio 2008 builds fail on "master" branch
| Reported by: | Samuli Seppänen | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | beta 2.3 |
| Component: | Building / Compiling | Version: | OpenVPN git master branch (Community Ed) |
| Severity: | Not set (select this one, unless your'e a OpenVPN developer) | Keywords: | |
| Cc: | gert@… |
Description
It's not possible to build
openvpn.exe
from openvpn.git's "master" branch with Visual Studio 2008 due to compiler errors. Error log attached.
Attachments (12)
Change History (17)
Changed 14 years ago by
| Attachment: | vs2008-build-errors.txt added |
|---|
comment:1 Changed 14 years ago by
| Component: | Generic / unclassified → Building / Compiling |
|---|
The messages related to openvpn-plugin.h should be fixed with the following commit
commit 555fc5e34a9b1aca4bfa435023fe1aa336557ec7
Author: David Sommerseth <davids@redhat.com>
Date: Fri Apr 29 13:28:06 2011 +0200
Fix const declarations in plug-in v3 structs
Microsoft Visual Studio complains about const char const **ptr declarations
and expects them to be be const char ** const ptr. The latter is what was the
intention, that neither the pointer nor the value(s) it points at can be changed.
Signed-off-by: David Sommerseth <davids@redhat.com>
Acked-by: Gert Doering <gert@greenie.muc.de>
Changed 14 years ago by
| Attachment: | 0001-Fixed-a-number-of-fatal-build-errors-on-Visual-Studi.patch added |
|---|
Patch fixing a number of these build errors
Changed 14 years ago by
| Attachment: | vs-2008-build-errors-2.txt added |
|---|
New build log after the (first) attached patch and commit 555fc5e have been applied
Changed 14 years ago by
| Attachment: | 0001-Fix-Microsoft-Visual-Studio-incompatibility-in-plugin-c.patch added |
|---|
Fixes more plug-in issues for MS Visual Studio
Changed 14 years ago by
| Attachment: | 0001-Fixed-a-Visual-Studio-2008-build-error-in-tun.c.patch added |
|---|
Fix to tun.c build issue by cron2
Changed 14 years ago by
| Attachment: | 0001-Fix-to-a-Visual-Studio-2008-build-error-in-options.c.patch added |
|---|
Changed 14 years ago by
| Attachment: | buildlog11.txt added |
|---|
Build log of "master" branch with all patches in this ticket included
Changed 14 years ago by
| Attachment: | vs-2008-build-errors-4.txt added |
|---|
New build log with the patches applied on top of a clean master branch. Notice tun.c build errors.
Changed 14 years ago by
| Attachment: | 0001-Replace-32-bit-based-add_in6_addr-implementation-by-.patch added |
|---|
patch to change add_in6_addr() in socket.c to make it work on Windows
comment:2 Changed 14 years ago by
| Cc: | gert@… added |
|---|
Changed 14 years ago by
| Attachment: | winbuildfix-log-1.txt added |
|---|
Build logs of openvpn-testing.git's tmp/winbuildfix branch, when HEAD is at ddd65e4b5f55b
Changed 14 years ago by
| Attachment: | winbuildfix-log-2.txt added |
|---|
Same as winbuildfix-log.txt, but after applying the "Additional Visual Studio 2008 build fixes to tun.c" patch
Changed 14 years ago by
| Attachment: | 0001-Additional-Visual-Studio-2008-build-fixes-to-tun.c.patch added |
|---|
comment:3 Changed 14 years ago by
Final Visual studio compile fixes are available
here
and have been ACKed. Apply these on top of
tmp/winbuildfix
branch (based on "master") and VS builds work again.
comment:4 Changed 14 years ago by
| Milestone: | → beta 2.3 |
|---|
comment:5 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
| Version: | 2.3-beta / 2.3-RC → git master branch |
I believe this is fixed and needed patches have reached the master branch., so closing this one. Please re-open if there are more issues.
Visual Studio 2008 build errors