Tobias Bosch | cfa8c24 | 2019-07-19 03:29:40 -0700 | [diff] [blame] | 1 | // Copyright 2019 The Chromium OS Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 5 | package main |
| 6 | |
| 7 | import ( |
Tobias Bosch | a50a9c1 | 2019-08-16 11:47:00 -0700 | [diff] [blame] | 8 | "bytes" |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 9 | "fmt" |
| 10 | "io" |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 11 | "path/filepath" |
Tobias Bosch | 5f98f2d | 2019-08-15 17:17:57 -0700 | [diff] [blame] | 12 | "strings" |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 13 | ) |
| 14 | |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 15 | func callCompiler(env env, cfg *config, inputCmd *command) int { |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 16 | var compilerErr error |
Tobias Bosch | 5edca50 | 2019-08-26 12:53:41 -0700 | [diff] [blame] | 17 | |
| 18 | if !filepath.IsAbs(inputCmd.Path) && !strings.HasPrefix(inputCmd.Path, ".") && |
| 19 | !strings.ContainsRune(inputCmd.Path, filepath.Separator) { |
Tobias Bosch | 5f98f2d | 2019-08-15 17:17:57 -0700 | [diff] [blame] | 20 | if resolvedPath, err := resolveAgainstPathEnv(env, inputCmd.Path); err == nil { |
| 21 | inputCmd = &command{ |
| 22 | Path: resolvedPath, |
| 23 | Args: inputCmd.Args, |
| 24 | EnvUpdates: inputCmd.EnvUpdates, |
| 25 | } |
| 26 | } else { |
| 27 | compilerErr = err |
| 28 | } |
| 29 | } |
| 30 | exitCode := 0 |
| 31 | if compilerErr == nil { |
| 32 | if cfg.oldWrapperPath != "" { |
| 33 | exitCode, compilerErr = callCompilerWithRunAndCompareToOldWrapper(env, cfg, inputCmd) |
| 34 | } else { |
| 35 | exitCode, compilerErr = callCompilerInternal(env, cfg, inputCmd) |
| 36 | } |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 37 | } |
| 38 | if compilerErr != nil { |
| 39 | printCompilerError(env.stderr(), compilerErr) |
| 40 | exitCode = 1 |
| 41 | } |
| 42 | return exitCode |
| 43 | } |
| 44 | |
| 45 | func callCompilerWithRunAndCompareToOldWrapper(env env, cfg *config, inputCmd *command) (exitCode int, err error) { |
Tobias Bosch | a50a9c1 | 2019-08-16 11:47:00 -0700 | [diff] [blame] | 46 | stdinBuffer := &bytes.Buffer{} |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 47 | recordingEnv := &commandRecordingEnv{ |
Tobias Bosch | a50a9c1 | 2019-08-16 11:47:00 -0700 | [diff] [blame] | 48 | env: env, |
Tobias Bosch | 6f59a66 | 2019-08-20 15:37:11 -0700 | [diff] [blame] | 49 | stdinReader: teeStdinIfNeeded(env, inputCmd, stdinBuffer), |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 50 | } |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 51 | // Note: this won't do a real exec as recordingEnv redirects exec to run. |
| 52 | if exitCode, err = callCompilerInternal(recordingEnv, cfg, inputCmd); err != nil { |
| 53 | return 0, err |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 54 | } |
Tobias Bosch | a50a9c1 | 2019-08-16 11:47:00 -0700 | [diff] [blame] | 55 | if err = compareToOldWrapper(env, cfg, inputCmd, stdinBuffer.Bytes(), recordingEnv.cmdResults, exitCode); err != nil { |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 56 | return exitCode, err |
| 57 | } |
| 58 | return exitCode, nil |
| 59 | } |
| 60 | |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 61 | func callCompilerInternal(env env, cfg *config, inputCmd *command) (exitCode int, err error) { |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 62 | if err := checkUnsupportedFlags(inputCmd); err != nil { |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 63 | return 0, err |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 64 | } |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 65 | mainBuilder, err := newCommandBuilder(env, cfg, inputCmd) |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 66 | if err != nil { |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 67 | return 0, err |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 68 | } |
Tobias Bosch | 5847281 | 2019-07-11 04:24:52 -0700 | [diff] [blame] | 69 | processPrintConfigFlag(mainBuilder) |
| 70 | processPrintCmdlineFlag(mainBuilder) |
| 71 | env = mainBuilder.env |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 72 | var compilerCmd *command |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 73 | clangSyntax := processClangSyntaxFlag(mainBuilder) |
Tobias Bosch | 8cb363f | 2019-10-17 07:44:13 -0700 | [diff] [blame] | 74 | if cfg.isAndroidWrapper { |
| 75 | // FIXME: This combination of using the directory of the symlink but the |
| 76 | // basename of the link target is strange but is the logic that old android |
| 77 | // wrapper uses. Change this to use directory and basename either from the |
| 78 | // absWrapperPath or from the builder.path, but don't mix anymore. |
| 79 | mainBuilder.path = filepath.Join(filepath.Dir(mainBuilder.path), filepath.Base(mainBuilder.absWrapperPath)+".real") |
| 80 | |
| 81 | switch mainBuilder.target.compilerType { |
| 82 | case clangType: |
| 83 | mainBuilder.addPreUserArgs(mainBuilder.cfg.clangFlags...) |
| 84 | mainBuilder.addPreUserArgs(mainBuilder.cfg.commonFlags...) |
| 85 | if _, err := processGomaCccFlags(mainBuilder); err != nil { |
| 86 | return 0, err |
| 87 | } |
| 88 | compilerCmd = mainBuilder.build() |
| 89 | case clangTidyType: |
| 90 | compilerCmd = mainBuilder.build() |
| 91 | default: |
| 92 | return 0, newErrorwithSourceLocf("unsupported compiler: %s", mainBuilder.target.compiler) |
| 93 | } |
| 94 | } else if mainBuilder.target.compilerType == clangType { |
Tobias Bosch | 38f3c42 | 2019-07-08 11:03:26 -0700 | [diff] [blame] | 95 | cSrcFile, useClangTidy := processClangTidyFlags(mainBuilder) |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 96 | sysroot, err := prepareClangCommand(mainBuilder) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 97 | if err != nil { |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 98 | return 0, err |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 99 | } |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 100 | allowCCache := true |
Tobias Bosch | 38f3c42 | 2019-07-08 11:03:26 -0700 | [diff] [blame] | 101 | if useClangTidy { |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 102 | allowCCache = false |
| 103 | clangCmdWithoutGomaAndCCache := mainBuilder.build() |
| 104 | if err := runClangTidy(env, clangCmdWithoutGomaAndCCache, cSrcFile); err != nil { |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 105 | return 0, err |
Tobias Bosch | 38f3c42 | 2019-07-08 11:03:26 -0700 | [diff] [blame] | 106 | } |
| 107 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 108 | if err := processGomaCCacheFlags(sysroot, allowCCache, mainBuilder); err != nil { |
| 109 | return 0, err |
| 110 | } |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 111 | compilerCmd = mainBuilder.build() |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 112 | } else { |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 113 | if clangSyntax { |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 114 | allowCCache := false |
| 115 | clangCmd, err := calcClangCommand(allowCCache, mainBuilder.clone()) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 116 | if err != nil { |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 117 | return 0, err |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 118 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 119 | gccCmd, err := calcGccCommand(mainBuilder) |
| 120 | if err != nil { |
| 121 | return 0, err |
| 122 | } |
Tobias Bosch | a50a9c1 | 2019-08-16 11:47:00 -0700 | [diff] [blame] | 123 | return checkClangSyntax(env, clangCmd, gccCmd) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 124 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 125 | compilerCmd, err = calcGccCommand(mainBuilder) |
| 126 | if err != nil { |
| 127 | return 0, err |
| 128 | } |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 129 | } |
Tobias Bosch | 9d60930 | 2019-07-10 06:16:04 -0700 | [diff] [blame] | 130 | rusageLogfileName := getRusageLogFilename(env) |
Tobias Bosch | 9780ea9 | 2019-07-11 01:19:42 -0700 | [diff] [blame] | 131 | bisectStage := getBisectStage(env) |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 132 | if shouldForceDisableWError(env) { |
Tobias Bosch | 9d60930 | 2019-07-10 06:16:04 -0700 | [diff] [blame] | 133 | if rusageLogfileName != "" { |
| 134 | return 0, newUserErrorf("GETRUSAGE is meaningless with FORCE_DISABLE_WERROR") |
| 135 | } |
Tobias Bosch | 9780ea9 | 2019-07-11 01:19:42 -0700 | [diff] [blame] | 136 | if bisectStage != "" { |
| 137 | return 0, newUserErrorf("BISECT_STAGE is meaningless with FORCE_DISABLE_WERROR") |
| 138 | } |
Tobias Bosch | f6d9f4f | 2019-07-09 08:09:01 -0700 | [diff] [blame] | 139 | return doubleBuildWithWNoError(env, cfg, compilerCmd) |
| 140 | } |
Tobias Bosch | c88ee8a | 2019-10-01 15:00:52 -0700 | [diff] [blame] | 141 | if shouldCompileWithFallback(env) { |
| 142 | if rusageLogfileName != "" { |
| 143 | return 0, newUserErrorf("GETRUSAGE is meaningless with FORCE_DISABLE_WERROR") |
| 144 | } |
| 145 | if bisectStage != "" { |
| 146 | return 0, newUserErrorf("BISECT_STAGE is meaningless with FORCE_DISABLE_WERROR") |
| 147 | } |
| 148 | return compileWithFallback(env, cfg, compilerCmd, mainBuilder.absWrapperPath) |
| 149 | } |
Tobias Bosch | 9d60930 | 2019-07-10 06:16:04 -0700 | [diff] [blame] | 150 | if rusageLogfileName != "" { |
Tobias Bosch | 9780ea9 | 2019-07-11 01:19:42 -0700 | [diff] [blame] | 151 | if bisectStage != "" { |
| 152 | return 0, newUserErrorf("BISECT_STAGE is meaningless with GETRUSAGE") |
| 153 | } |
Tobias Bosch | 9d60930 | 2019-07-10 06:16:04 -0700 | [diff] [blame] | 154 | return logRusage(env, rusageLogfileName, compilerCmd) |
| 155 | } |
Tobias Bosch | 9780ea9 | 2019-07-11 01:19:42 -0700 | [diff] [blame] | 156 | if bisectStage != "" { |
Tobias Bosch | 820bffa | 2019-09-30 15:53:52 -0700 | [diff] [blame] | 157 | compilerCmd, err = calcBisectCommand(env, cfg, bisectStage, compilerCmd) |
| 158 | if err != nil { |
| 159 | return 0, err |
| 160 | } |
Tobias Bosch | 9780ea9 | 2019-07-11 01:19:42 -0700 | [diff] [blame] | 161 | } |
Tobias Bosch | 9332d21 | 2019-07-10 06:23:57 -0700 | [diff] [blame] | 162 | // Note: We return an exit code only if the underlying env is not |
| 163 | // really doing an exec, e.g. commandRecordingEnv. |
| 164 | return wrapSubprocessErrorWithSourceLoc(compilerCmd, env.exec(compilerCmd)) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 165 | } |
| 166 | |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 167 | func prepareClangCommand(builder *commandBuilder) (sysroot string, err error) { |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 168 | sysroot = "" |
Tobias Bosch | 8cb363f | 2019-10-17 07:44:13 -0700 | [diff] [blame] | 169 | if !builder.cfg.isHostWrapper { |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 170 | sysroot = processSysrootFlag(builder) |
| 171 | } |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 172 | builder.addPreUserArgs(builder.cfg.clangFlags...) |
Caroline Tice | 8ac33e0 | 2019-10-28 09:48:18 -0700 | [diff] [blame^] | 173 | builder.addPostUserArgs(builder.cfg.clangPostFlags...) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 174 | calcCommonPreUserArgs(builder) |
| 175 | if err := processClangFlags(builder); err != nil { |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 176 | return "", err |
| 177 | } |
| 178 | return sysroot, nil |
| 179 | } |
| 180 | |
| 181 | func calcClangCommand(allowCCache bool, builder *commandBuilder) (*command, error) { |
| 182 | sysroot, err := prepareClangCommand(builder) |
| 183 | if err != nil { |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 184 | return nil, err |
| 185 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 186 | if err := processGomaCCacheFlags(sysroot, allowCCache, builder); err != nil { |
| 187 | return nil, err |
| 188 | } |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 189 | return builder.build(), nil |
| 190 | } |
| 191 | |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 192 | func calcGccCommand(builder *commandBuilder) (*command, error) { |
Tobias Bosch | b27c8f2 | 2019-07-19 06:53:07 -0700 | [diff] [blame] | 193 | sysroot := "" |
| 194 | if !builder.cfg.isHostWrapper { |
| 195 | sysroot = processSysrootFlag(builder) |
| 196 | } |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 197 | builder.addPreUserArgs(builder.cfg.gccFlags...) |
Tobias Bosch | b27c8f2 | 2019-07-19 06:53:07 -0700 | [diff] [blame] | 198 | if !builder.cfg.isHostWrapper { |
| 199 | calcCommonPreUserArgs(builder) |
| 200 | } |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 201 | processGccFlags(builder) |
Tobias Bosch | b27c8f2 | 2019-07-19 06:53:07 -0700 | [diff] [blame] | 202 | if !builder.cfg.isHostWrapper { |
| 203 | allowCCache := true |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 204 | if err := processGomaCCacheFlags(sysroot, allowCCache, builder); err != nil { |
| 205 | return nil, err |
| 206 | } |
Tobias Bosch | b27c8f2 | 2019-07-19 06:53:07 -0700 | [diff] [blame] | 207 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 208 | return builder.build(), nil |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 209 | } |
| 210 | |
| 211 | func calcCommonPreUserArgs(builder *commandBuilder) { |
| 212 | builder.addPreUserArgs(builder.cfg.commonFlags...) |
Tobias Bosch | 8cb363f | 2019-10-17 07:44:13 -0700 | [diff] [blame] | 213 | if !builder.cfg.isHostWrapper { |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 214 | processPieFlags(builder) |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 215 | processThumbCodeFlags(builder) |
Tobias Bosch | 1cd5f84 | 2019-08-20 10:05:33 -0700 | [diff] [blame] | 216 | processStackProtectorFlags(builder) |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 217 | processX86Flags(builder) |
| 218 | } |
Tobias Bosch | 8cb363f | 2019-10-17 07:44:13 -0700 | [diff] [blame] | 219 | processSanitizerFlags(builder) |
Tobias Bosch | d868417 | 2019-07-08 10:59:14 -0700 | [diff] [blame] | 220 | } |
| 221 | |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 222 | func processGomaCCacheFlags(sysroot string, allowCCache bool, builder *commandBuilder) (err error) { |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 223 | gomaccUsed := false |
| 224 | if !builder.cfg.isHostWrapper { |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 225 | gomaccUsed, err = processGomaCccFlags(builder) |
| 226 | if err != nil { |
| 227 | return err |
| 228 | } |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 229 | } |
Tobias Bosch | 198a3c9 | 2019-07-17 04:22:34 -0700 | [diff] [blame] | 230 | if !gomaccUsed && allowCCache { |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 231 | processCCacheFlag(sysroot, builder) |
| 232 | } |
Tobias Bosch | c58f8d5 | 2019-09-30 10:37:42 -0700 | [diff] [blame] | 233 | return nil |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 234 | } |
| 235 | |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 236 | func getAbsWrapperPath(env env, wrapperCmd *command) (string, error) { |
Tobias Bosch | 5847281 | 2019-07-11 04:24:52 -0700 | [diff] [blame] | 237 | wrapperPath := getAbsCmdPath(env, wrapperCmd) |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 238 | evaledCmdPath, err := filepath.EvalSymlinks(wrapperPath) |
| 239 | if err != nil { |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 240 | return "", wrapErrorwithSourceLocf(err, "failed to evaluate symlinks for %s", wrapperPath) |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 241 | } |
Tobias Bosch | 31dec2c | 2019-07-18 07:34:03 -0700 | [diff] [blame] | 242 | return evaledCmdPath, nil |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 243 | } |
| 244 | |
Tobias Bosch | 900dbc9 | 2019-06-24 09:31:39 -0700 | [diff] [blame] | 245 | func printCompilerError(writer io.Writer, compilerErr error) { |
| 246 | if _, ok := compilerErr.(userError); ok { |
| 247 | fmt.Fprintf(writer, "%s\n", compilerErr) |
| 248 | } else { |
| 249 | fmt.Fprintf(writer, |
| 250 | "Internal error. Please report to chromeos-toolchain@google.com.\n%s\n", |
| 251 | compilerErr) |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 252 | } |
Tobias Bosch | ef8f969 | 2019-06-10 15:50:33 -0700 | [diff] [blame] | 253 | } |
Tobias Bosch | 6f59a66 | 2019-08-20 15:37:11 -0700 | [diff] [blame] | 254 | |
| 255 | func teeStdinIfNeeded(env env, inputCmd *command, dest io.Writer) io.Reader { |
| 256 | // We can't use io.TeeReader unconditionally, as that would block |
| 257 | // calls to exec.Cmd.Run(), even if the underlying process has already |
| 258 | // terminated. See https://github.com/golang/go/issues/7990 for more details. |
| 259 | lastArg := "" |
| 260 | for _, arg := range inputCmd.Args { |
| 261 | if arg == "-" && lastArg != "-o" { |
| 262 | return io.TeeReader(env.stdin(), dest) |
| 263 | } |
| 264 | lastArg = arg |
| 265 | } |
| 266 | return env.stdin() |
| 267 | } |