Skip to content

Commit

Permalink
wscript: don't set subsystem flag, it's only needed for executables
Browse files Browse the repository at this point in the history
  • Loading branch information
a1batross committed Nov 27, 2024
1 parent 61f9455 commit 9a96bec
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 8 deletions.
1 change: 0 additions & 1 deletion cl_dll/wscript
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,6 @@ def build(bld):
defines = defines,
use = libs,
install_path = install_path,
subsystem = bld.env.MSVC_SUBSYSTEM,
idx = bld.get_taskgen_count()
)

1 change: 0 additions & 1 deletion dlls/wscript
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ def build(bld):
includes = includes,
defines = defines,
install_path = install_path,
subsystem = bld.env.MSVC_SUBSYSTEM,
idx = bld.get_taskgen_count()
)

6 changes: 0 additions & 6 deletions wscript
Original file line number Diff line number Diff line change
Expand Up @@ -47,12 +47,6 @@ def configure(conf):
conf.load('fwgslib reconfigure compiler_optimizations')
conf.env.MSVC_TARGETS = ['x86' if not conf.options.ALLOW64 else 'x64']

# Force XP compatibility, all build targets should add subsystem=bld.env.MSVC_SUBSYSTEM
if conf.env.MSVC_TARGETS[0] == 'x86':
conf.env.MSVC_SUBSYSTEM = 'WINDOWS,5.01'
else:
conf.env.MSVC_SUBSYSTEM = 'WINDOWS'

# Load compilers early
conf.load('xcompile compiler_c compiler_cxx gccdeps')

Expand Down

0 comments on commit 9a96bec

Please sign in to comment.