Build fixed #19
Merged
Build fixed #19
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This looks great. So far as I can see, this is a rewrite of the build scripts etc, including a move to using depot_tools to get v8. I note that you have removed VS2008 support, which I think is fine. I guess that the nuget build process has been broken for a while, so I don't think its lack is a problem. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello. As i promised in #3, here is my working version. It is possible now to automatically rebuild v8 and js.net with desired MSVC++, processor architecture, release/debug mode and .NET with specified Visual Studio version. It is easy to get full V8 build environment too. See readme and all the .bat`s.
There is one drawback: if you want to build with MSVC++2010 using Visual Studio 2012/2013, you will need VS2010 installed. I know it is possible to use Windows SDKs but i can not test this because SDK7/7.1 fail to install.
And one last note: i don't know how to properly make nuget packages for multiple platforms and i am unsure if you merge this pull request or there's need to edit something. So no nuget packages here for now.