Skip to content

Commit

Permalink
Merge pull request #85357 from mhilbrunner/fix-generate-vsproj
Browse files Browse the repository at this point in the history
Fix generating vsproj with SCons 4.6.0+
  • Loading branch information
akien-mga authored Nov 25, 2023
2 parents 1ba920f + 7211013 commit 5df9867
Showing 1 changed file with 11 additions and 8 deletions.
19 changes: 11 additions & 8 deletions methods.py
Original file line number Diff line number Diff line change
Expand Up @@ -708,25 +708,28 @@ def detect_visual_c_compiler_version(tools_env):


def find_visual_c_batch_file(env):
from SCons.Tool.MSCommon.vc import (
get_default_version,
get_host_target,
find_batch_file,
)
from SCons.Tool.MSCommon.vc import get_default_version, get_host_target, find_batch_file, find_vc_pdir

# Syntax changed in SCons 4.4.0.
from SCons import __version__ as scons_raw_version

scons_ver = env._get_major_minor_revision(scons_raw_version)

version = get_default_version(env)
msvc_version = get_default_version(env)

if scons_ver >= (4, 4, 0):
(host_platform, target_platform, _) = get_host_target(env, version)
(host_platform, target_platform, _) = get_host_target(env, msvc_version)
else:
(host_platform, target_platform, _) = get_host_target(env)

return find_batch_file(env, version, host_platform, target_platform)[0]
if scons_ver < (4, 6, 0):
return find_batch_file(env, msvc_version, host_platform, target_platform)[0]

# Scons 4.6.0+ removed passing env, so we need to get the product_dir ourselves first,
# then pass that as the last param instead of env as the first param as before.
# We should investigate if we can avoid relying on SCons internals here.
product_dir = find_vc_pdir(env, msvc_version)
return find_batch_file(msvc_version, host_platform, target_platform, product_dir)[0]


def generate_cpp_hint_file(filename):
Expand Down

0 comments on commit 5df9867

Please sign in to comment.