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

CMake build fixes for windows #265

Merged
merged 5 commits into from Dec 16, 2016

Conversation

TcT2k
Copy link
Contributor

@TcT2k TcT2k commented Dec 16, 2016

  • Fixes various small issues to enable building with visual studio
  • Removed dependency on boost (from @beached's fork)
  • Fix to build with 3.1.1
  • Modified appveyor
    • build with VS2013 using wxWidgets 3.0.2
    • build with VS2015 usign wxWidgets 3.1.0
    • build using mingw

The mingw build currently fails when building stack.cpp and I could use some help what is missing there. All packages should be installed as the premake build worked.

@TcT2k TcT2k mentioned this pull request Dec 16, 2016
@patlkli
Copy link
Contributor

patlkli commented Dec 16, 2016

Thank you!

I think we have to add the binutils subdirectory to the include path for the MinGW build.
Can you do that and check if it builds for you then?

@TcT2k TcT2k force-pushed the cmake_build_windows branch 4 times, most recently from 299130f to 0199fb9 Compare December 16, 2016 20:20
beached and others added 5 commits December 16, 2016 21:27
Converted boost::shared_ptr, boost::weak_ptr,
boost::enable_shared_from_this to their std counterparts
Moved AppVeyor commands to batch file for easier handling of various build configurations.
@TcT2k
Copy link
Contributor Author

TcT2k commented Dec 16, 2016

The binutils include path and linking the correct libraries seems to do the trick.

@patlkli patlkli merged commit 1df26c6 into wxFormBuilder:cmake_build Dec 16, 2016
@TcT2k TcT2k deleted the cmake_build_windows branch December 19, 2016 16:02
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.

None yet

3 participants