Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(957)

Issue 15051: Convert from using env['PLATFORM'] directly to using the more flexible... (Closed)

Created:
12 years ago by sgk
Modified:
9 years, 5 months ago
Reviewers:
tony, bradn, evanm
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Convert from using env['PLATFORM'] directly to using the more flexible and better-thought-out Hammer env.Bits() idioms: * env['PLATFORM'] == 'win32' => env.Bit('windows') * env['PLATFORM'] == 'posix' => env.Bit('linux') * env['PLATFORM'] == 'darwin' => env.Bit('mac') New idioms: * env.Bit('posix') => really does mean "any POSIX platform" * env.AnyBits('mac', 'linux') => specifically mac or linux, excluding other POSIX platforms Where we were using compound conditionals (e.g., "env['PLATFORM'] in ('posix', 'darwin')") I tried to take my best shot at translating the intent (i.e., "env.Bits('posix')" for something POSIX, "not env.Bits('mac')" for something not yet ported to Mac, etc.) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=7270

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+248 lines, -225 lines) Patch
M base/base_lib.scons View 8 chunks +8 lines, -8 lines 0 comments Download
M base/base_unittests.scons View 6 chunks +7 lines, -7 lines 0 comments Download
M base/gfx/base_gfx.scons View 3 chunks +4 lines, -4 lines 0 comments Download
M breakpad/SConscript View 3 chunks +3 lines, -3 lines 0 comments Download
M build/SConscript.v8 View 2 chunks +2 lines, -2 lines 0 comments Download
M build/debug.scons View 1 chunk +1 line, -1 line 0 comments Download
M build/external_code.scons View 1 chunk +1 line, -1 line 0 comments Download
M build/googleurl_lib.scons View 1 chunk +1 line, -1 line 0 comments Download
M build/googleurl_unittests.scons View 1 chunk +1 line, -1 line 0 comments Download
M build/internal/essential.scons View 2 chunks +3 lines, -3 lines 0 comments Download
M build/internal/release_impl_dom_stats.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/SConscript View 9 chunks +9 lines, -9 lines 0 comments Download
M chrome/browser/browser.scons View 4 chunks +5 lines, -5 lines 0 comments Download
M chrome/browser/debugger/debugger.scons View 2 chunks +15 lines, -2 lines 0 comments Download
M chrome/browser/views/SConscript View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/common.scons View 4 chunks +7 lines, -7 lines 0 comments Download
M chrome/common/ipc_tests.scons View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/installer/mini_installer/installer_unittests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/mini_installer/mini_installer.scons View 4 chunks +5 lines, -5 lines 0 comments Download
M chrome/installer/setup/setup.scons View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/plugin/plugin.scons View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/renderer/renderer.scons View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/test/activex_test_control/activex_test_control.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automated_ui_tests/automated_ui_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/chrome_plugin/test_chrome_plugin.scons View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/test/interactive_ui/interactive_ui_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/memory_test/memory_test.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/mini_installer_test/mini_installer_test.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/page_cycler/page_cycler_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/perf/perftests.scons View 4 chunks +14 lines, -4 lines 0 comments Download
M chrome/test/plugin/plugin_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/reliability/reliability_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/security_tests/security_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/selenium/selenium_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/startup/startup_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/tab_switching/tab_switching_test.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/ui/ui_tests.scons View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/test/unit/unit_tests.scons View 5 chunks +7 lines, -7 lines 0 comments Download
M chrome/third_party/hunspell/SConscript View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/tools/convert_dict/convert_dict.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/crash_service/crash_service.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/perf/flush_cache/flush_cache.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/profiles/generate_profile.scons View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/test/image_diff/image_diff.scons View 1 chunk +1 line, -1 line 0 comments Download
M media/media_lib.scons View 1 chunk +4 lines, -4 lines 0 comments Download
M media/media_unittests.scons View 1 chunk +1 line, -1 line 0 comments Download
M net/crash_cache.scons View 2 chunks +2 lines, -3 lines 0 comments Download
M net/dump_cache.scons View 2 chunks +2 lines, -2 lines 0 comments Download
M net/net.scons View 2 chunks +2 lines, -2 lines 0 comments Download
M net/net_lib.scons View 6 chunks +6 lines, -6 lines 0 comments Download
M net/net_perftests.scons View 2 chunks +4 lines, -5 lines 0 comments Download
M net/net_unittests.scons View 3 chunks +5 lines, -5 lines 0 comments Download
M net/stress_cache.scons View 3 chunks +3 lines, -3 lines 0 comments Download
M net/tools/tld_cleanup/tld_cleanup.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/src/sandbox_lib.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/tests/common/sandbox_common.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/tests/integration_tests/sbox_integration_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/tests/unit_tests/sbox_unittests.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/tests/validation_tests/sbox_validation_tests.scons View 1 chunk +1 line, -1 line 0 comments Download
M sandbox/wow_helper/wow_helper.scons View 1 chunk +1 line, -1 line 0 comments Download
M sdch/SConscript View 3 chunks +4 lines, -4 lines 0 comments Download
M skia/SConscript View 4 chunks +6 lines, -6 lines 0 comments Download
M testing/SConscript.gtest View 1 chunk +1 line, -1 line 0 comments Download
M third_party/bspatch/bspatch.scons View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/bzip2/bzip2.scons View 1 chunk +1 line, -1 line 0 comments Download
M third_party/libevent/libevent.scons View 3 chunks +4 lines, -4 lines 0 comments Download
M third_party/libjpeg/libjpeg.scons View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/libpng/libpng.scons View 1 chunk +1 line, -1 line 0 comments Download
M third_party/libxml/libxml.scons View 3 chunks +4 lines, -4 lines 0 comments Download
M third_party/libxslt/libxslt.scons View 3 chunks +3 lines, -3 lines 0 comments Download
M third_party/lzma_sdk/lzma_sdk.scons View 1 chunk +1 line, -1 line 0 comments Download
M third_party/sqlite/SConscript View 1 chunk +1 line, -1 line 0 comments Download
M third_party/zlib/zlib.scons View 1 chunk +1 line, -1 line 0 comments Download
M webkit/SConscript View 5 chunks +6 lines, -6 lines 0 comments Download
M webkit/SConscript.javascriptcore_pcre View 1 chunk +1 line, -1 line 0 comments Download
M webkit/SConscript.port View 6 chunks +7 lines, -5 lines 0 comments Download
M webkit/build/JavaScriptCore/SConscript View 2 chunks +3 lines, -3 lines 0 comments Download
M webkit/build/V8Bindings/SConscript View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/build/WebCore/SConscript View 5 chunks +5 lines, -5 lines 0 comments Download
M webkit/build/localized_strings/SConscript View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/build/port/SConscript View 1 chunk +1 line, -1 line 0 comments Download
M webkit/default_plugin/SConscript View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/glue/SConscript View 4 chunks +7 lines, -7 lines 0 comments Download
M webkit/glue/plugins/test/SConscript View 2 chunks +2 lines, -2 lines 0 comments Download
M webkit/tools/npapi_layout_test_plugin/SConscript View 1 chunk +1 line, -1 line 0 comments Download
M webkit/tools/test_shell/SConscript View 9 chunks +10 lines, -10 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
sgk
Generating Visual Studio files (regardless of what platform you're running on) and eventually XCode files ...
12 years ago (2008-12-18 22:05:22 UTC) #1
bradn
LGTM
12 years ago (2008-12-18 22:32:07 UTC) #2
tony
12 years ago (2008-12-18 22:47:40 UTC) #3
If everything builds, rubber stamp, LGTM!

Powered by Google App Engine
This is Rietveld 408576698