Refactoring wscript. Add execute flag on waf

wscript

@@ -3,27 +3,28 @@
 # a1batross, mittorn, 2018
 
 from __future__ import print_function
-from waflib import Logs
-
+from waflib import Logs, Options
 import os
 import sys
 
+VERSION = '0.99'
+APPNAME = 'xash3d-fwgs'
+SUBDIRS = [ 'game_launch', 'mainui', 'vgui_support', 'engine' ]
+top = '.'
+
 def get_git_version():
 	# try grab the current version number from git
-	version = "notset"
-	if os.path.exists(".git"):
+	version = 'notset'
+	if os.path.exists('.git'):
 		try:
-			version = os.popen("git describe --dirty --always").read().strip()
+			version = os.popen('git describe --dirty --always').read().strip()
 		except Exception as e:
-			print(e)
-	return version
+			pass
 
-VERSION = '0.99'
-APPNAME = 'xash3d-fwgs'
-GIT_SHA = get_git_version()
-SUBDIRS = [ 'game_launch', 'vgui_support', 'engine', 'mainui' ]
+	if(len(version) == 0):
+		version = 'notset'
 
-top = '.'
+	return version
 
 def options(opt):
 	opt.load('compiler_cxx compiler_c')
@@ -31,17 +32,21 @@ def options(opt):
 		opt.load('msvc msvs')
 	
 	opt.add_option(
-		'--dedicated', action = 'store_true', dest = 'DEDICATED', default=False,
+		'--dedicated', action = 'store_true', dest = 'DEDICATED', default = False,
 		help = 'build Xash Dedicated Server(XashDS)')
 	
 	opt.add_option(
-		'--64bits', action = 'store_true', dest = 'ALLOW64', default=False,
+		'--64bits', action = 'store_true', dest = 'ALLOW64', default = False,
 		help = 'allow targetting 64-bit engine')
 	
 	opt.add_option(
-		'--release', action = 'store_true', dest = 'RELEASE', default=False,
+		'--release', action = 'store_true', dest = 'RELEASE', default = False,
 		help = 'strip debug info from binary and enable optimizations')
 		
+	opt.add_option(
+		'--no-download-deps', action = 'store_false', dest = 'AUTODL', default = True,
+		help = 'don\'t try to download dependencies from network')
+	
 	opt.add_option(
 		'--win-style-install', action = 'store_true', dest = 'WIN_INSTALL', default = False,
 		help = 'install like Windows build, ignore prefix, useful for development')
@@ -49,46 +54,53 @@ def options(opt):
 	opt.recurse(SUBDIRS)
 
 def configure(conf):
-	conf.env.MSVC_TARGETS = ['x86']
+	conf.env.MSVC_TARGETS = ['x86'] # explicitly request x86 target for MSVC
 	conf.load('compiler_cxx compiler_c')
-	if(conf.env.COMPILER_CC != 'msvc'):
+	if sys.platform == 'win32':
+		conf.load('msvc msvs')
+
+	# Check if we have 64-bit toolchain
+	conf.env.DEST_64BIT = False # predict state
+	try:
 		conf.check_cc(
 			fragment='''
-			#include <stdio.h>
-			int main( void ) { printf("%ld", sizeof( void * )); return 0; }
+			int main( void ) 
+			{ 
+				int check[sizeof(void*) == 4 ? 1: -1]; 
+				return 0; 
+			}
 			''',
-			execute      = True,
-			define_ret   = True,
-			uselib_store = 'SIZEOF_VOID_P',
-			msg          = 'Checking sizeof(void*)')
-	else:
-		conf.env.SIZEOF_VOID_P = '4' # TODO: detect target
+			msg          = 'Checking if compiler create 32 bit code')
+	except conf.errors.ConfigurationError:
+		# Program not compiled, we have 64 bit 
+		conf.env.DEST_64BIT = True
 	
-	if(int(conf.env.SIZEOF_VOID_P) != 4):
+	if(conf.env.DEST_64BIT):
 		if(not conf.options.ALLOW64):
-			conf.env.append_value('LINKFLAGS', '-m32')
-			conf.env.append_value('CFLAGS', '-m32')
-			conf.env.append_value('CXXFLAGS', '-m32')
+			conf.env.append_value('LINKFLAGS', ['-m32'])
+			conf.env.append_value('CFLAGS', ['-m32'])
+			conf.env.append_value('CXXFLAGS', ['-m32'])
 			Logs.info('NOTE: will build engine with 64-bit toolchain using -m32')
 		else:
 			Logs.warn('WARNING: 64-bit engine may be unstable')
 	
 	if(conf.env.COMPILER_CC != 'msvc'):
 		if(conf.env.COMPILER_CC == 'gcc'):
-			conf.env.append_value('LINKFLAGS', '-Wl,--no-undefined')
+			conf.env.append_unique('LINKFLAGS', ['-Wl,--no-undefined'])
 		if(conf.options.RELEASE):
-			conf.env.append_unique('CFLAGS', '-O2')
-			conf.env.append_unique('CXXFLAGS', '-O2')
+			conf.env.append_unique('CFLAGS',   ['-O2'])
+			conf.env.append_unique('CXXFLAGS', ['-O2'])
 		else:
-			conf.env.append_unique('CFLAGS', '-Og')
-			conf.env.append_unique('CFLAGS', '-g')
-			conf.env.append_unique('CXXFLAGS', '-Og')
-			conf.env.append_unique('CXXFLAGS', '-g')
+			conf.env.append_unique('CFLAGS',   ['-Og', '-g'])
+			conf.env.append_unique('CXXFLAGS', ['-Og', '-g'])
 	else:
-		if(not conf.options.RELEASE):
-			conf.env.append_unique('CFLAGS', '/Z7')
-			conf.env.append_unique('CXXFLAGS', '/Z7')
-			conf.env.append_unique('LINKFLAGS', '/DEBUG')
+		if(conf.options.RELEASE):
+			conf.env.append_unique('CFLAGS',   ['/O2'])
+			conf.env.append_unique('CXXFLAGS', ['/O2'])
+		else:
+			conf.env.append_unique('CFLAGS',   ['/Z7'])
+			conf.env.append_unique('CXXFLAGS', ['/Z7'])
+			conf.env.append_unique('LINKFLAGS', ['/DEBUG'])
 
 	if(conf.env.DEST_OS != 'win32'):
 		conf.check( lib='dl' )
@@ -96,7 +108,7 @@ def configure(conf):
 		conf.check( lib='pthread' )
 	
 	conf.env.DEDICATED     = conf.options.DEDICATED
-	conf.env.SINGLE_BINARY = conf.options.DEDICATED
+	conf.env.SINGLE_BINARY = conf.options.DEDICATED # We don't need game launcher on dedicated
 	
 	# indicate if we are packaging for Linux/BSD
 	if(not conf.options.WIN_INSTALL and 
@@ -107,15 +119,18 @@ def configure(conf):
 		# prefix is ignored
 		conf.env.LIBDIR = conf.env.BINDIR = '/'
 	
-	# global
-	conf.env.append_unique('XASH_BUILD_COMMIT', GIT_SHA)
+	conf.start_msg('Checking git hash')
+	git_version = get_git_version()
+	conf.end_msg(git_version)
+	conf.env.append_unique('DEFINES', 'XASH_BUILD_COMMIT="' + git_version + '"')
 	
 	for i in SUBDIRS:
 		conf.setenv(i, conf.env) # derive new env from global one
 		conf.env.ENVNAME = i
-		Logs.info('Configuring ' + i)
+		conf.msg(msg='Configuring ' + i, result='in progress', color='BLUE')
 		# configure in standalone env
 		conf.recurse(i)
+		conf.msg(msg='Configuring ' + i, result='done', color='BLUE')
 		conf.setenv('')
 
 def build(bld):