-
Notifications
You must be signed in to change notification settings - Fork 850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix compile error on macos #191
Conversation
Signed-off-by: Yang Keao <[email protected]>
779a900
to
13fd900
Compare
Codecov Report
@@ Coverage Diff @@
## master #191 +/- ##
=======================================
Coverage 39.47% 39.47%
=======================================
Files 17 17
Lines 608 608
=======================================
Hits 240 240
Misses 337 337
Partials 31 31 Continue to review full report at Codecov.
|
Signed-off-by: Yang Keao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rest LGTM
pkg/chaosdaemon/netem_darwin.go
Outdated
@@ -0,0 +1,32 @@ | |||
// Copyright 2019 PingCAP, Inc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/2019/2020/g
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Signed-off-by: Yang Keao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* fix compile error on macos Signed-off-by: Yang Keao <[email protected]>
Signed-off-by: Yang Keao [email protected]
What problem does this PR solve?
This PR solves compile error on mac os. And hope to fix #167
What is changed and how does it work?
Now
handle.QdiscAdd
,handle.QdiscDel
andhandler.QdiscList
are only used in_linux.go
files.Check List
Tests
Wait for manual test from a macos user @cwen0 . New bugs may occure during test on macos.
Code changes