Reviewers: mp+83695_code.launchpad.net, niemeyer,
Message: Done.
On 2011/11/29 13:11:45, niemeyer wrote: > LGTM
> Just one detail:
> https://codereview.appspot.com/5445048/diff/1/zk_test.go > File zk_test.go (right):
> https://codereview.appspot.com/5445048/diff/1/zk_test.go#newcode1 > zk_test.go:1: package zk_test > The package name in all tests should be renamed to "zookeeper_test" as well.
Description: updated the package names for the tests as well, as per Gustavo's comments.
https://code.launchpad.net/~mathieu-lonjaret/gozk/zookeeper/+merge/83695
(do not edit description out of merge proposal)
Please review this at https://codereview.appspot.com/5445048/
Affected files: M Makefile M retry_test.go M server.go M suite_test.go M zk.go M zk_test.go
Index: Makefile === <email address hidden> > <email address hidden> === modified file 'Makefile' --- Makefile 2011-09-24 14:00:23 +0000 +++ Makefile 2011-11-28 21:30:01 +0000 @@ -2,7 +2,7 @@
all: package
-TARG=launchpad.net/gozk/zk +TARG=launchpad.net/gozk/zookeeper
GOFILES=\ server.go\
Index: retry_test.go === <email address hidden> > <email address hidden> === modified file 'retry_test.go' --- retry_test.go 2011-09-24 14:00:23 +0000 +++ retry_test.go 2011-11-29 13:55:45 +0000 @@ -1,8 +1,8 @@ -package zk_test +package zookeeper_test
import ( . "launchpad.net/gocheck" - "launchpad.net/gozk/zk" + zk "launchpad.net/gozk/zookeeper" "os" )
Index: server.go === <email address hidden> > <email address hidden> === modified file 'server.go' --- server.go 2011-09-24 14:00:23 +0000 +++ server.go 2011-11-28 21:30:01 +0000 @@ -1,4 +1,4 @@ -package zk +package zookeeper
import ( "bufio"
Index: suite_test.go === <email address hidden> > <email address hidden> === modified file 'suite_test.go' --- suite_test.go 2011-09-24 14:00:23 +0000 +++ suite_test.go 2011-11-29 13:55:45 +0000 @@ -1,11 +1,11 @@ -package zk_test +package zookeeper_test
import ( . "launchpad.net/gocheck" "bufio" "exec" "fmt" - "launchpad.net/gozk/zk" + zk "launchpad.net/gozk/zookeeper" "os" "testing" "time"
Index: zk.go === <email address hidden> > <email address hidden> === modified file 'zk.go' --- zk.go 2011-09-24 14:00:23 +0000 +++ zk.go 2011-11-28 21:30:01 +0000 @@ -6,7 +6,7 @@ // // Written by Gustavo Niemeyer <email address hidden> // -package zk +package zookeeper
/* #cgo CFLAGS: -I/usr/include/c-client-src
Index: zk_test.go === <email address hidden> > <email address hidden> === modified file 'zk_test.go' --- zk_test.go 2011-09-24 14:00:23 +0000 +++ zk_test.go 2011-11-29 13:55:45 +0000 @@ -1,8 +1,8 @@ -package zk_test +package zookeeper_test
import ( . "launchpad.net/gocheck" - "launchpad.net/gozk/zk" + zk "launchpad.net/gozk/zookeeper" "time" )
« Back to merge proposal
Reviewers: mp+83695_ code.launchpad. net, niemeyer,
Message:
Done.
On 2011/11/29 13:11:45, niemeyer wrote:
> LGTM
> Just one detail:
> https:/ /codereview. appspot. com/5445048/ diff/1/ zk_test. go
> File zk_test.go (right):
> https:/ /codereview. appspot. com/5445048/ diff/1/ zk_test. go#newcode1
> zk_test.go:1: package zk_test
> The package name in all tests should be renamed to "zookeeper_test" as
well.
Description:
updated the package names for the tests as well, as per Gustavo's
comments.
https:/ /code.launchpad .net/~mathieu- lonjaret/ gozk/zookeeper/ +merge/ 83695
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/5445048/
Affected files:
M Makefile
M retry_test.go
M server.go
M suite_test.go
M zk.go
M zk_test.go
Index: Makefile
=== <email address hidden> >
<email address hidden>
=== modified file 'Makefile'
--- Makefile 2011-09-24 14:00:23 +0000
+++ Makefile 2011-11-28 21:30:01 +0000
@@ -2,7 +2,7 @@
all: package
-TARG=launchpad .net/gozk/ zk .net/gozk/ zookeeper
+TARG=launchpad
GOFILES=\
server.go\
Index: retry_test.go
=== <email address hidden> >
<email address hidden>
=== modified file 'retry_test.go'
--- retry_test.go 2011-09-24 14:00:23 +0000
+++ retry_test.go 2011-11-29 13:55:45 +0000
@@ -1,8 +1,8 @@
-package zk_test
+package zookeeper_test
import ( net/gocheck" net/gozk/ zk" net/gozk/ zookeeper"
. "launchpad.
- "launchpad.
+ zk "launchpad.
"os"
)
Index: server.go
=== <email address hidden> >
<email address hidden>
=== modified file 'server.go'
--- server.go 2011-09-24 14:00:23 +0000
+++ server.go 2011-11-28 21:30:01 +0000
@@ -1,4 +1,4 @@
-package zk
+package zookeeper
import (
"bufio"
Index: suite_test.go
=== <email address hidden> >
<email address hidden>
=== modified file 'suite_test.go'
--- suite_test.go 2011-09-24 14:00:23 +0000
+++ suite_test.go 2011-11-29 13:55:45 +0000
@@ -1,11 +1,11 @@
-package zk_test
+package zookeeper_test
import ( net/gocheck" net/gozk/ zk" net/gozk/ zookeeper"
. "launchpad.
"bufio"
"exec"
"fmt"
- "launchpad.
+ zk "launchpad.
"os"
"testing"
"time"
Index: zk.go
=== <email address hidden> >
<email address hidden>
=== modified file 'zk.go'
--- zk.go 2011-09-24 14:00:23 +0000
+++ zk.go 2011-11-28 21:30:01 +0000
@@ -6,7 +6,7 @@
//
// Written by Gustavo Niemeyer <email address hidden>
//
-package zk
+package zookeeper
/* include/ c-client- src
#cgo CFLAGS: -I/usr/
Index: zk_test.go
=== <email address hidden> >
<email address hidden>
=== modified file 'zk_test.go'
--- zk_test.go 2011-09-24 14:00:23 +0000
+++ zk_test.go 2011-11-29 13:55:45 +0000
@@ -1,8 +1,8 @@
-package zk_test
+package zookeeper_test
import ( net/gocheck" net/gozk/ zk" net/gozk/ zookeeper"
. "launchpad.
- "launchpad.
+ zk "launchpad.
"time"
)