Scons: Build modules in seperate env.

[27thLiz]

No description provided.

[27thLiz]

There's still one module left using the main env: svg.
That one seems to do some linker tricks

godot/modules/svg/SCsub

Lines 17 to 28 in b110813

	# Needs to be appended to arrive after libscene in the linker call,
	# but we don't want it to arrive *after* system libs, so manual hack
	# LIBS contains first SCons Library objects ("SCons.Node.FS.File object")
	# and then plain strings for system library. We insert between the two.
	inserted = False
	for idx, linklib in enumerate(env["LIBS"]):
	if isbasestring(linklib): # first system lib such as "X11", otherwise SCons lib object
	env["LIBS"].insert(idx, lib)
	inserted = True
	break
	if not inserted:
	env.Append(LIBS=[lib])

Not quite sure what that does exactly, so I decided not to touch it for now.

[akien-mga]

There's still one module left using the main env: svg.
That one seems to do some linker tricks

The comment is misleading as it was copy pasted from the freetype module without edit (

godot/modules/freetype/SCsub

Lines 71 to 86 in 76725b6

	lib = env.add_library("freetype_builtin", thirdparty_sources)
	# Needs to be appended to arrive after libscene in the linker call,
	# but we don't want it to arrive *after* system libs, so manual hack
	# LIBS contains first SCons Library objects ("SCons.Node.FS.File object")
	# and then plain strings for system library. We insert between the two.
	inserted = False
	for idx, linklib in enumerate(env["LIBS"]):
	if isbasestring(linklib): # first system lib such as "X11", otherwise SCons lib object
	env["LIBS"].insert(idx, lib)
	inserted = True
	break
	if not inserted:
	env.Append(LIBS=[lib])
	env.Append(CCFLAGS=['-DFT2_BUILD_LIBRARY'])
	if (env['target'] != 'release'):
	env.Append(CCFLAGS=['-DZLIB_DEBUG'])

), it should actually mention libeditor and not libscene. The idea is that this module is needed by the editor (

godot/editor/editor_themes.cpp

Line 113 in 378bb89

#ifdef SVG_ENABLED

), and based on the build order defined in

godot/SConstruct

Lines 449 to 461 in bf05309

	# build subdirs, the build order is dependent on link order.
	SConscript("core/SCsub")
	SConscript("servers/SCsub")
	SConscript("scene/SCsub")
	SConscript("editor/SCsub")
	SConscript("drivers/SCsub")
	SConscript("platform/SCsub")
	SConscript("modules/SCsub")
	SConscript("main/SCsub")
	SConscript("platform/" + selected_platform + "/SCsub") # build selected platform

it might be tricky. I haven't checked if it's actually needed here though.