lumberyard icon indicating copy to clipboard operation
lumberyard copied to clipboard

Modify waf to build vs solution that actually uses msbuild.

Open caseymcc opened this issue 6 years ago • 4 comments

Requires setting 'msbuild = True' in user_settings.options under [Visual Studio Project Generator] section

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

caseymcc avatar Sep 05 '19 20:09 caseymcc

Hi @caseymcc, thanks for contributing this to LY we'll take a look at the change.

AMZN-nggieber avatar Sep 06 '19 17:09 AMZN-nggieber

For the CryLegacy gem it links to Lua but all of Lua needs to be linked using extern "C" however I see nothing in the files that is in place to handle that. Lua is included with the pre-processor define LUALIB_API (which is also use in CryLegacy) and if it was set to extern "C" would fix the problem however is only seems possible to set it to extern without modifying the code. How is waf handling this?

caseymcc avatar Sep 10 '19 16:09 caseymcc

In CloudGemFramework why does it matter if the build is a project generator? If it is a generator then is includes lmbraws_unsupported.waf_files no matter if aws_native is supported, even though in lumberyard_sdks.py it list project_generator as supporting aws_native. If lmbraws_unsupported.waf is included in the build then there are lots of things that get defined twice.

caseymcc avatar Sep 11 '19 20:09 caseymcc

think my frustration level has reached too high, done messing with this for a bit. Need to handle the az_code_gen feature, maybe the last item maybe not. Seems every time I fix something, something else pops up.

caseymcc avatar Sep 12 '19 21:09 caseymcc