Skip to content

Get MSAN, ASAN working for C#46

Merged
nicolasnoble merged 8 commits intogrpc:masterfrom
ctiller:testing
Jan 15, 2015
Merged

Get MSAN, ASAN working for C#46
nicolasnoble merged 8 commits intogrpc:masterfrom
ctiller:testing

Conversation

@ctiller
Copy link
Copy Markdown
Member

@ctiller ctiller commented Jan 15, 2015

No description provided.

MSAN, TSAN, ASAN need to see all the code. Valgrind, MSAN want -DPURIFY.
These libraries need to see all of the source code, so using system
installed libraries is out of the question.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That shouldn't work. We're missing a ZLIB_DEP somewhere down. (darn, I wish github would let me comment on arbitrary lines...)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something odd is going on with zlib - help accepted :)

On Thu Jan 15 2015 at 8:44:41 AM Nicolas Noble notifications@github.com
wrote:

In templates/Makefile.template
#46 (diff):

@@ -189,7 +213,7 @@ endif

ifeq ($(HAS_SYSTEM_ZLIB),false)
ifeq ($(HAS_EMBEDDED_ZLIB),true)
-ZLIB_DEP = third_party/zlib/libz.a
+ZLIB_DEP = libs/$(CONFIG)/zlib/libz.a

That shouldn't work. We're missing a ZLIB_DEP somewhere down. (darn, I
wish github would let me comment on arbitrary lines...)


Reply to this email directly or view it on GitHub
https://github.com/google/grpc/pull/46/files#r23023364.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries, I'll share a commit to fix this shortly.

@murgatroid99
Copy link
Copy Markdown
Member

@nicolasnoble This article may help.

@ctiller
Copy link
Copy Markdown
Member Author

ctiller commented Jan 15, 2015

Nico's change gets zlib building right.

@ctiller
Copy link
Copy Markdown
Member Author

ctiller commented Jan 15, 2015

Hang on: this doesn't build.

@ctiller
Copy link
Copy Markdown
Member Author

ctiller commented Jan 15, 2015

make: *** No rule to make target libs/msan/zlib/libz.alibs/msan/libgpr.so.0.8.0.0', needed bylibs/msan/libgrpc.so.0.8.0.0'. Stop.

@ctiller
Copy link
Copy Markdown
Member Author

ctiller commented Jan 15, 2015

Found the bug

nicolasnoble added a commit that referenced this pull request Jan 15, 2015
Get MSAN, ASAN working for C
@nicolasnoble nicolasnoble merged commit a82516b into grpc:master Jan 15, 2015
@ctiller ctiller deleted the testing branch January 16, 2015 23:22
stanley-cheung referenced this pull request in stanley-cheung/grpc Aug 27, 2015
apolcyn referenced this pull request in apolcyn/grpc Jul 25, 2016
jtattermusch referenced this pull request in jtattermusch/grpc Jan 18, 2017
…esome-dance-tune-y'all

Fix Python errors
@lock lock bot locked as resolved and limited conversation to collaborators Feb 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants