search

gscarr / winsvc

Automatically exported from code.google.com/p/winsvc
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

go test failed on win7/64 #5

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
go version
go version devel +23e48d4f3066 Wed Sep 03 23:10:15 2014 -0400 windows/amd64

go test ./...
?       code.google.com/p/winsvc/debug  [no test files]
ok      code.google.com/p/winsvc/eventlog       0.257s
?       code.google.com/p/winsvc/example        [no test files]
--- FAIL: TestMyService (0.00s)
        mgr_test.go:37: service myservice already exists
FAIL
FAIL    code.google.com/p/winsvc/mgr    0.194s
?       code.google.com/p/winsvc/registry       [no test files]
--- FAIL: TestExample (1.30s)
        svc_test.go:87: Start(myservice) failed: The service did not respond to the start or control request in a timely fashion.
FAIL
FAIL    code.google.com/p/winsvc/svc    1.499s
?       code.google.com/p/winsvc/winapi [no test files]

Original issue reported on code.google.com by chaishus...@gmail.com on 5 Sep 2014 at 6:27

GoogleCodeExporter commented 9 years ago 
I will investigate.

Alex

Original comment by alex.bra...@gmail.com on 5 Sep 2014 at 10:56

GoogleCodeExporter commented 9 years ago 
I can see couple of problems, including one in new syscall.NewCallback. 
Hopefully I can fix them all. Thank you for raising issue with me.

Alex

Original comment by alex.bra...@gmail.com on 11 Sep 2014 at 2:46

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision 137aafe2081d.

TBR=brainman
R=alex.brainman
CC=brainman
https://codereview.appspot.com/141410043

Original comment by alex.bra...@gmail.com on 11 Sep 2014 at 2:47

GoogleCodeExporter commented 9 years ago 
Here https://codereview.appspot.com/138330046/ is the fix. Would you like to 
review it?

Alex

Original comment by alex.bra...@gmail.com on 15 Sep 2014 at 7:31

GoogleCodeExporter commented 9 years ago 
I am a newibe to windows service, i can't do the review work.
Thanks for your trust.

All tests passed!
Thanks!

Original comment by chaishus...@gmail.com on 15 Sep 2014 at 8:06

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision ac1c6963c5b8.

Original comment by alex.bra...@gmail.com on 16 Sep 2014 at 12:39