Refactored android default ArchType check in SConscript.
authorJoseph Morrow <joseph.l.morrow@intel.com>
Tue, 2 Jun 2015 15:37:32 +0000 (11:37 -0400)
committerErich Keane <erich.keane@intel.com>
Tue, 2 Jun 2015 21:51:43 +0000 (21:51 +0000)
According to SCons' wiki, checking strings should be done with "==" and
not "is". "is" is to be used for checking against the ID value for a
value and not the value, while "==" checks against the value.

Change-Id: I28710e324d998098cd22bb615a03c388f6903f18
Signed-off-by: Joseph Morrow <joseph.l.morrow@intel.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/1170
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Erich Keane <erich.keane@intel.com>
build_common/SConscript

index 2120653..c26e92d 100644 (file)
@@ -40,7 +40,7 @@ if target_os not in host_target_map[host]:
        print "\nError: Unknown target os: %s (Allow values: %s)\n" % (target_os, host_target_map[host])
        Exit(1)
 
-if target_os is 'android':
+if target_os == 'android':
        default_arch = 'x86'
 else:
        default_arch = platform.machine()