Skip to content
This repository has been archived by the owner on Jan 12, 2022. It is now read-only.

Emscripten support for Visual Studio 2012/2013 #40

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

james-allison
Copy link

Hi there,

Apologies - I'm pretty new to github, so not 100% with the correct etiquette for contributing to someone else's project.

ANYHOW - I was playing with Emscripten over the weekend, and made a few changes to vs-tool to support Visual Studio 2013. The changes are mostly taken from spotting 'what VS-Android did', so I'm not going to pretend I've a deep understanding for the reasoning behind them all. The emscripten integration does, from what I can tell, now work in VS2012 and VS2013. I don't have VS2010 however, so you might want to check that's not been broken....

Summary of changes:

  • Switched out the vs-android cpp tasks dll for the three different version specific dlls
  • Made MSBuild changes (mostly copied from VS-Android, searching for 'VisualStudioVersion')
  • Added an 'install.cmd' to install into relevant installations with the correct DLL.

TODO:

  • Check VS2010 still works....
  • Change the non emscripten platforms to add the VisualStudioVersion conditions - I might get round to this later, but I've no pressing need myself to do this.

Cheers,

James

@james-allison james-allison changed the title Vs2012/2013 support Emscripten support for Visual Studio 2012/2013 Sep 15, 2014
@juj
Copy link
Owner

juj commented Sep 15, 2014

Thanks for this. I'm going to keep this unmerged for a while, so this is best to live in your fork now. Hopefully I'll have time to focus on vs-tool in more detail in the future, but at present there's too many other distractions for me that vs-tool is currently in maintenance.

@james-allison
Copy link
Author

Ah the perils of distractions :o) No worries, feel free to give me a shout when you get round to looking at it again – I may have got round to looking at the other platforms by then.

James

From: juj [mailto:[email protected]]
Sent: 15 September 2014 15:07
To: juj/vs-tool
Cc: James Allison
Subject: Re: [vs-tool] Emscripten support for Visual Studio 2012/2013 (#40)

Thanks for this. I'm going to keep this unmerged for a while, so this is best to live in your fork now. Hopefully I'll have time to focus on vs-tool in more detail in the future, but at present there's too many other distractions for me that vs-tool is currently in maintenance.


Reply to this email directly or view it on GitHub #40 (comment) . https://github.com/notifications/beacon/8775767__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyNjQwOTIzNCwiZGF0YSI6eyJpZCI6NDI0NzgzNDN9fQ==--bb750970b09192a5c489477230127cfa16614fa7.gif

Corrected mistaken reference to debug build
@TurkeyMan
Copy link

Is there any chance of this resurrecting? I've been watching for a year now, quietly hoping it'll come good one day :P

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants