Skip to content

Added vc sln and vcxproj files - #26

Closed
Ofek (OfekShilon) wants to merge 1 commit into
microsoft:masterfrom
OfekShilon:master
Closed

Added vc sln and vcxproj files#26
Ofek (OfekShilon) wants to merge 1 commit into
microsoft:masterfrom
OfekShilon:master

Conversation

@OfekShilon

Copy link
Copy Markdown
Contributor

Currently required PlatformSDK is default (8.1), considering requiring the one bundled with latest VS (10.0.17134.0).

Included configurations for Release/Debug x MT/MD x x64/x86 .

This can save some mundane work for devs working in VS, wishing to use detours. It's time to modernize the build script a bit anyway.

…ault (8.1), considering requiring the one bundled with latest VS (10.0.17134.0)
@LonghronShen

Copy link
Copy Markdown

Maybe PR #48 also helps.
Using CMake to generate VS projects looks good, or just open the CMakeLists.txt using VS2017+ is better.

@dtarditi

Copy link
Copy Markdown
Contributor

Ofek (@OfekShilon), thanks for the proposed PR. What do you think of the suggestion from Steven Hangger (@LonghronShen) to use CMake instead?

@OfekShilon

Copy link
Copy Markdown
Contributor Author

Sounds like a good idea.

@dtarditi

Copy link
Copy Markdown
Contributor

Ok, closing this PR in favor of PR #48. Thanks for proposing this change and encouraging us to move things forward on the build system.

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.

3 participants