On 2014/03/10 01:20:01, axw wrote: > https://codereview.appspot.com/73310043/diff/1/container/lxc/lxc.go > File container/lxc/lxc.go (right):
https://codereview.appspot.com/73310043/diff/1/container/lxc/lxc.go#newcode49 > container/lxc/lxc.go:49: var FsCommandOutput = (*exec.Cmd).CombinedOutput > Do you really want CombinedOutput? Just Output, I think.
https://codereview.appspot.com/73310043/diff/1/container/lxc/lxc.go#newcode52 > container/lxc/lxc.go:52: cmd := exec.Command("df", "--output=fstype", > LxcContainerDir) > If you use Output instead, you can store Stderr in a bytes.Buffer and add it to > the error for context.
After discussion, LGTM, since you won't get mixed stdout/stderr.
https://codereview.appspot.com/73310043/
« Back to merge proposal
On 2014/03/10 01:20:01, axw wrote: /codereview. appspot. com/73310043/ diff/1/ container/ lxc/lxc. go lxc/lxc. go (right):
> https:/
> File container/
https:/ /codereview. appspot. com/73310043/ diff/1/ container/ lxc/lxc. go#newcode49 lxc/lxc. go:49: var FsCommandOutput = Cmd).CombinedOu tput
> container/
(*exec.
> Do you really want CombinedOutput? Just Output, I think.
https:/ /codereview. appspot. com/73310043/ diff/1/ container/ lxc/lxc. go#newcode52 lxc/lxc. go:52: cmd := exec.Command("df", "--output=fstype",
> container/
> LxcContainerDir)
> If you use Output instead, you can store Stderr in a bytes.Buffer and
add it to
> the error for context.
After discussion, LGTM, since you won't get mixed stdout/stderr.
https:/ /codereview. appspot. com/73310043/