Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add script to build with cmake #444

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mattock
Copy link
Member

@mattock mattock commented Aug 19, 2021

I'm opening this PR for discussion as I'm not sure how we want to handle the "build openvpn-gui on Windows with MSVC" challenge.

The simple script in this PR is good enough for my immediate needs, that is, for building OpenVPN-GUI on a Windows Server 2019 server in the context of my new dockerized buildbot environment:

If having a build script like this is ok, I can make it more generic/better. I suppose we could also add support for building openvpn-gui with msbuild, like we do with OpenVPN. I'm not sure how much work that would entail, though.

@selvanair
Copy link
Collaborator

I've not been following build system changes. @lstipakov may have some input.
At the minimum we could just make hard-coded vars like CMAKE_TOOLCHAIN_FILE configurable (say, take from $Env if defined, else use a default)

@mattock
Copy link
Member Author

mattock commented Sep 1, 2021

@selvanair that parameterization was what I actually did in openvpn-vagrant where I merged this script while waiting for approval/direction:

@lstipakov do you think that updated version of this script is good enough, or should we have a more proper Windows build system?

@lstipakov
Copy link
Member

I am not strictly against it, but I am not sure that this script needs to be in this repo. I think it fits better either in openvpn-vagrant or openvpn-build, it is mainly needed for building MSI installers - it builds for 3 architectures using specific directory layout.

I personally use Visual Studio, and build system just works there - you open CMake project and, assuming you have vcpkg installed and integrated into msbuild, everything works out of the box. Those who use cmake from command line are probably knowledgeable enough to run cmake with required arguments - we could also add those to README.

@mattock
Copy link
Member Author

mattock commented Sep 1, 2021

I think we could just keep this script in openvpn-vagrant and point people to it in this repo's documentation.

@cron2
Copy link
Contributor

cron2 commented Sep 1, 2021 via email

@mattock
Copy link
Member Author

mattock commented Sep 1, 2021

I'm not opposed to that approach, either.

@cron2
Copy link
Contributor

cron2 commented Sep 13, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants