-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[Go] remove "-lpthread" flag from "compile_fuzzer" and fix comments (#2714). #3657
Conversation
Also fixed gonids and go-dns builds. |
Hey @Dor1s, TravisBuddy Request Identifier: 0e9a4610-7fae-11ea-8584-e9567e2fd09d |
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.
Thanks!
projects/golang/build.sh
Outdated
# Copy the fuzzer binary | ||
cp fuzzer-$fuzzer $OUT | ||
# Copy the fuzzer binary | ||
cp fuzzer-$fuzzer $OUT |
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.
nit: i think these two cps can be removed by adding $OUT/ as prefix in the two commands.
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.
done!
} | ||
|
||
compile_fuzzer /root/go/src/github.com/google/gonids/ FuzzParseRule fuzz_parserule | ||
compile_fuzzer github.com/google/gonids FuzzParseRule fuzz_parserule |
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.
You might want to remove RUN go get github.com/google/gonids from Dockerfile.
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.
Why? I think that commands checks out the project to fuzz
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.
My bad!, skip.
Hey @Dor1s, TravisBuddy Request Identifier: 37d41fc0-8002-11ea-a591-33c228b1c119 |
Looks like it isn't needed anymore, or maybe it was never needed and I mistakenly added it in the first place.
Also fixed
gonids
andgo-dns
builds, they had import errors.