Skip to content

Commit 9ac1734

Browse files
authored
Revert "Add an optional prefix_with_time_cmd flag to compiled_action." (#904)
Reverts #902. This command doesn't work, and @zanderso's advice to use a LED build with a local branch works for now.
1 parent 26cb9de commit 9ac1734

File tree

1 file changed

+1
-24
lines changed

1 file changed

+1
-24
lines changed

build/compiled_action.gni

+1-24
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,6 @@
2828
# of these change. If inputs is empty, the step will run only when the
2929
# binary itself changes.
3030
#
31-
# prefix_with_time_cmd (optional)
32-
# [bool] If true, the command will be prefixed with "time" to measure the
33-
# time, CPU and memory usage of the tool, i.e. "time <command>" with
34-
# appropriate arguments for the current platform.
35-
#
3631
# visibility
3732
# deps
3833
# args (all optional)
@@ -132,27 +127,9 @@ template("compiled_action") {
132127
depfile = invoker.depfile
133128
}
134129

135-
args = []
136-
if (defined(invoker.prefix_with_time_cmd) && invoker.prefix_with_time_cmd) {
137-
if (host_os == "linux") {
138-
args += [
139-
"time",
140-
"-v",
141-
]
142-
} else if (host_os == "mac") {
143-
args += [
144-
"time",
145-
"-l",
146-
]
147-
} else {
148-
print("Warning: prefix_with_time_cmd is not supported on Windows.")
149-
}
150-
}
151-
152130
# The script takes as arguments the binary to run, and then the arguments
153131
# to pass it.
154-
args += [ rebase_path(host_executable, root_build_dir) ]
155-
args += invoker.args
132+
args = [ rebase_path(host_executable, root_build_dir) ] + invoker.args
156133
}
157134
}
158135

0 commit comments

Comments
 (0)