I'm going to wait to submit this until the tree is in a happier state.
We have 3 different build failures now, and Windows hasn't gotten past the
FileMode CL, which I cross-compiled (some of) but never tested on Windows.
On Thu, Jan 19, 2012 at 3:58 PM, <r@golang.org> wrote:
> LGTM
> assuming the deletions are just missing from codereview
>
>
>
http://codereview.appspot.com/**5536076/<http://codereview.appspot.com/5536076/>
>
Nevermind, windows x64 just went "ok", and darwin is cool too. Linux
multicast stuff can be fixed later.
On Thu, Jan 19, 2012 at 4:02 PM, Brad Fitzpatrick <bradfitz@golang.org>wrote:
> I'm going to wait to submit this until the tree is in a happier state.
>
> We have 3 different build failures now, and Windows hasn't gotten past the
> FileMode CL, which I cross-compiled (some of) but never tested on Windows.
>
>
> On Thu, Jan 19, 2012 at 3:58 PM, <r@golang.org> wrote:
>
>> LGTM
>> assuming the deletions are just missing from codereview
>>
>>
>>
http://codereview.appspot.com/**5536076/<http://codereview.appspot.com/5536076/>
>>
>
>
*** Submitted as http://code.google.com/p/go/source/detail?r=25879029ed5b *** database/sql: move from exp/sql R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5536076
Issue 5536076: code review 5536076: database/sql: move from exp/sql
(Closed)
Created 13 years ago by bradfitz
Modified 13 years ago
Reviewers:
Base URL:
Comments: 0