Edward, On Oct 18, 2012, at 7:03 AM, Edward d'Auvergne <[email protected]> wrote:
> Hi, > > I have tested out Scons 2.2.0 and have noticed that bug #2849 > (http://scons.tigris.org/issues/show_bug.cgi?id=2849) is still > present. I can see this also from the code: > > https://bitbucket.org/scons/scons/src/8764000345e06e326ef68fd0acf9366c1f3eb885/src/engine/SCons/Tool/MSCommon/vc.py?at=default > > in the lines: > > try_target_archs = [target_platform] > if not req_target_platform and target_platform in ('amd64','x86_64'): > try_target_archs.append('x86') > > The suggestion by Kyle Strand was to change to code from: > > try_target_archs = [target_platform] > if not req_target_platform and target_platform=='amd64' > try_target_archs.append('x86') > > to: > > try_target_archs = [target_platform] > if target_platform in ('amd64','x86_64'): > try_target_archs.append('x86') > > The key problem is that in this case is that 'req_target_platform' is > set to 'x86_64' when 32-bit C++ is used on 64-bit Windows (and no > 64-bit compiler is present). Taking out the 'req_target_platform' > check fixes the problem for Scons 2.2.0. Can you post your SConstruct/SConscript? env['TARGET_ARCH'] shouldn't have any default setting, this is what provides a value to req_target_platform. So in the case that a user explicitly requests a platform, we do not add any other platforms to be searched for. As far as I can tell that is the way the code is behaving. If you have an example showing a bug where TARGET_ARCH is not set, and it's not also trying 32bit, please let me know. Thanks, Bill _______________________________________________ Scons-dev mailing list [email protected] http://two.pairlist.net/mailman/listinfo/scons-dev
