Do not attempt to set App Container ACLs on non-Windows.
This is needed for when doing a cross-compile of a Windows build and there is never any need to set ACLs. BUG=1212742 Change-Id: I54f7d26b260c7949c8de5bda7e908dfd213d6f08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2935817 Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#889421}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
38b7aa2213
commit
1dc8d49885
@ -191,7 +191,8 @@ static_library("sandbox") {
|
||||
|
||||
# NACL on 32-bit builds this target twice, once for 64-bit and once for 32-bit
|
||||
# so avoid this dep from running twice with the same output in that case.
|
||||
if (current_cpu == target_cpu) {
|
||||
# Additionally, exclude setting ACLs when build is not happening on Windows.
|
||||
if (current_cpu == target_cpu && host_os == "win") {
|
||||
deps += [ ":set_appcontainer_acls" ]
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user