-
Notifications
You must be signed in to change notification settings - Fork 1k
WIP: Allow building with visual studio #5347
base: master
Are you sure you want to change the base?
Conversation
46e8844
to
93d5199
Compare
Great work, it will be really nice if qTox builds also natively under windows. There seems to be an issue with finding OpenAL in our current windows builds though: https://travis-ci.org/qTox/qTox/jobs/431400408#L3519 |
If you need a faster way than our CI to reproduce these errors you can use: https://github.com/qTox/qTox/tree/master/windows/cross-compile it's the same as executed on Travis CI. |
daa7830
to
15ff9d4
Compare
I think I'm going to put this on hold for now. I actually don't like builds that much but I thought I could quickly get something in that worked but it turns out it's not easy :(. If someone else wants to push this one through to the end feel free, but I'll probably come back to it after writing some actual code :) |
Changes needed to build under MSVC compiler. Mostly buildsystem changes with minor changes to source to deal with new header search systems with cmake and cross-platform compat changes
15ff9d4
to
390d621
Compare
What do you think about breaking this up into two chunks to make it smaller? One chunk would be all the changes in |
Thanks for understanding :). Ill make sure the include changes work with pkgconfigs generated include paths but if so i think that makes a lot of sense |
@sphaerophoria Hello,Are your method is OK?I need build qTox by Visual studio 2017 nowhere else,but only here . can you help me? |
PR for building qTox with MSVC.
I wanted to develop a piece of windows software in a windows IDE. Here are the changes that were needed :).
Ran sanity tests on the following
Along with the PR I'd like to put the following on the wiki (or in an md file somewhere)
Building qTox with visual studio
Disclaimer 1: This is from the perspective of a linux developer just trying to run software on thier windows PC. This may result in things being done in non-idiomadic ways. If there are "more windowsy" ways of doing this please edit this article.
Disclaimer 2: By the end of this process I was just happy that it worked. This means I was not willing to go back and do it again. This guide may be incomplete.
Prerequisites
choco install yasm
Build qTox itself
This change is