Commit a03124d1 authored by scottmg's avatar scottmg Committed by Commit bot

move protect_file_posix to posix block

Otherwise, on Windows when 'All' is built the no-op build is broken --
no files are included in the .lib so none is generated, which means on
subsequent builds ninja always reruns that step.

This is currently breaking win_chromium_compile_dbg_ng trybots.

R=pasko@chromium.org, thakis@chromium.org
BUG=424562

Review URL: https://codereview.chromium.org/981623002

Cr-Commit-Position: refs/heads/master@{#319119}
parent cacdcdcd
...@@ -382,6 +382,8 @@ ...@@ -382,6 +382,8 @@
# TODO(pasko): Remove this target when crbug.com/424562 is fixed. # TODO(pasko): Remove this target when crbug.com/424562 is fixed.
# GN: //base:protect_file_posix # GN: //base:protect_file_posix
'target_name': 'protect_file_posix', 'target_name': 'protect_file_posix',
'conditions': [
['os_posix == 1', {
'type': 'static_library', 'type': 'static_library',
'dependencies': [ 'dependencies': [
'base', 'base',
...@@ -389,6 +391,10 @@ ...@@ -389,6 +391,10 @@
'sources': [ 'sources': [
'files/protect_file_posix.cc', 'files/protect_file_posix.cc',
], ],
}, {
'type': 'none',
}],
],
}, },
{ {
'target_name': 'base_prefs_test_support', 'target_name': 'base_prefs_test_support',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment