Comment 1:

Sorry, didn't change full/import/path before clicking submit. Of course it should be
code.google.com/p/go.tools/cmd/goimports

Comment 2:

Labels changed: added repo-tools, release-go1.3maybe.

Comment 3:

Can you provide an example minimal play.golang.org program that shows differing results?
I failed to make a repro quickly.

Comment 4:

http://play.golang.org/p/cvG0QUhg0H
Check Imports then click Format. Now uncheck it and click Format again.

Comment 5:

Oh, goimports is forgetting to put a blank line in between the standard library and the
3rd-party imports in this case.

Comment 6:

We're not shipping goimports in the binary release so not tied to a specific release.

Labels changed: added release-none, removed release-go1.3maybe.

Status changed to Accepted.

Comment 7:

The problem is in imports.go:
https://code.google.com/p/go/source/browse/imports/imports.go?repo=tools&r=1911e4f0ca34#241
  var impLine = regexp.MustCompile(`^\s+(?:\w+\s+)?"(.+)"`)
That regex fails to match dot imports (it works fine for _ and renamed imports, since \w
captures that, but not dot).
It can be fixed by changing the regex to:
  var impLine = regexp.MustCompile(`^\s+(?:[\w\.]+\s+)?"(.+)"`)
In my testing, that fixes this issue. Being regex, it may introduce other issues (and
it's unfortunate to need to resort to regex to make Go source modifications), so I'll
let someone else figure out if this is acceptable.
I could make a CL for it, but I already have one in flight, so I'll wait until that
moves forward, or after 1.3 is out.

Comment 8:

Sorry, your other CL is now submitted. Could you send a CL & test for this one as well?

Comment 9:

Thanks a lot; will do later today!

Comment 10:

This issue was fixed by https://golang.org/cl/99400043. Can someone close it
please?

Comment 11:

Status changed to Fixed.