Allow building on Visual Studio 2022#13055
Conversation
|
While I certainly agree that allowing to build with VS 2022 is pretty important as some people may want to use the newest version available I'm not convinced preventing compilation with 2019 is a good idea as per the several reasons:
There would undoubtedly come a time when still allowing to build with VS 2019 would hold us back in some way and at this point we certainly should drop this support, but for now there is no real need in my opinion. |
I don't understand this one. You can perfectly install several editions of Visual Studio side by side, including enterprise and community.
I'm happy to do that, though I think it makes more sense to stick to Visual Studio 2019 on appveyor in this case and yet allow building with 2022. This ensures that, when we need to make a change that breaks Visual Studio 2019, we'll be notified automatically. |
Sure you can. The problem is that if you are a enterprise there are several restrictions for using Community version of VS as mentioned in its EULA, therefore it is likely you cannot use Community edition legally in that case.
Makes sense. |
|
I updated the pr description. |
See test results for failed build of commit c9426ea68c |
Support vs2019 and newer Update readme Restore appveyor back to 2019 Update nvdaHelper/archBuild_sconscript Bad commit fixup Update scons to 4.3.0
|
scons 4.3.0 is there, so we're ready to rock. |
|
This was in fact reverted by #13387 |
Link to issue number:
Closes #13033
Summary of the issue:
NVDA now builds with Visual Studio 2019. Version 2022 was released recently.
Description of how this pull request fixes the issue:
Testing strategy:
Tested locally and on appveyor.
Known issues with pull request:
It looks like MS made it pretty difficult to get to the downloads of 2019 through their website without logging in, therefore I'm linking to the executables directly using their short urls.
Change log entries:
For Developers
Code Review Checklist: