code review 4697041: go/build: make Nuke comment say what it does

go/build: make Nuke comment say what it does

[brainman, 2011-07-12 02:56:20]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[brainman, 2011-07-12 02:56:53]

I'm not certain on semantics of Nuke. Perhaps comment is correct, and we need to
change Nuke code to really delete Intermediates.

Alex

[rsc, 2011-07-12 03:00:03]

I think the old comment was right except
for s/Clean/Nuke/.  Note that it calls Clean
at the end.


On Mon, Jul 11, 2011 at 19:56,  <alex.brainman@gmail.com> wrote:
> I'm not certain on semantics of Nuke. Perhaps comment is correct, and we
> need to change Nuke code to really delete Intermediates.
>
> Alex
>
> http://codereview.appspot.com/4697041/
>

[brainman, 2011-07-12 03:06:30]

On 2011/07/12 03:00:03, rsc wrote:
> ...  Note that it calls Clean
> at the end.
> 

Of course it does. I should have washed my eyes this morning <g>.

PTAL.

[rsc, 2011-07-12 03:06:52]

LGTM

[brainman, 2011-07-12 03:08:28]

*** Submitted as http://code.google.com/p/go/source/detail?r=6b9301cc45ce ***

go/build: make Nuke comment say what it does

R=rsc
CC=golang-dev
http://codereview.appspot.com/4697041