Skip to content
This repository has been archived by the owner on Sep 6, 2018. It is now read-only.

go test -race fails #100

Open
tmc opened this issue Aug 6, 2013 · 5 comments
Open

go test -race fails #100

tmc opened this issue Aug 6, 2013 · 5 comments
Assignees
Milestone

Comments

@tmc
Copy link

tmc commented Aug 6, 2013

I haven't looked into whether not these are legitimate races or test-only races but they should be cleaned up so go test -race is useful.

@benbjohnson
Copy link
Contributor

@traviscline Thanks for submitting the bug. The race detector was passing for a while but we must have gotten lazy. :)

I'll fix the issues and change Travis CI to use the race detector on every build.

@ghost ghost assigned benbjohnson Aug 6, 2013
@xiang90
Copy link
Contributor

xiang90 commented Aug 6, 2013

@benbjohnson @traviscline
Most the race is caused by debugln, and some other minor issues. We will fix it as thing is getting stable.

@erikstmartin
Copy link

Looks like these races still exist. Is this something that's looking to be addressed? Should I look into some of these and submit pull-requests?

@benbjohnson
Copy link
Contributor

@erikstmartin I still want to address those races but I haven't had the time so far. If you want to send some PRs that'd be great. Otherwise I'll try to fix them in the near future.

@xiang90
Copy link
Contributor

xiang90 commented Oct 22, 2013

@benbjohnson I will take care about this very soon.

@xiang90 xiang90 mentioned this issue Oct 22, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants