Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1263)

Issue 612002: Linux: Fix builds on systems without PR_SET_PTRACER in linux/prctl.h. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 2 months ago by Lei Zhang (chromium)
Modified:
11 years, 2 months ago
Reviewers:
thakis
CC:
google-breakpad-dev_googlegroups.com
Base URL:
http://google-breakpad.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Linux: Fix builds on systems without PR_SET_PTRACER in linux/prctl.h.

R=thakis@chromium.org

Committed: https://code.google.com/p/google-breakpad/source/detail?r=1199

Patch Set 1 #

Total comments: 2

Patch Set 2 : fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc View 1 2 chunks +5 lines, -1 line 0 comments Download

Messages

Total messages: 5
Lei Zhang (chromium)
11 years, 2 months ago #1
thakis
https://breakpad.appspot.com/612002/diff/1/src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc File src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc (right): https://breakpad.appspot.com/612002/diff/1/src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc#newcode76 src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc:76: #ifndef __ANDROID__ Can this be removed then?
11 years, 2 months ago #2
thakis
https://breakpad.appspot.com/612002/diff/1/src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc File src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc (right): https://breakpad.appspot.com/612002/diff/1/src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc#newcode76 src/client/linux/minidump_writer/linux_ptrace_dumper_unittest.cc:76: #ifndef __ANDROID__ On 2013/07/02 02:07:21, thakis wrote: > Can ...
11 years, 2 months ago #3
Lei Zhang (chromium)
I'm just going to play it safe and leave it out.
11 years, 2 months ago #4
Lei Zhang (chromium)
11 years, 2 months ago #5
Message was sent while issue was closed.
Committed patchset #2 manually as r1199 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1004:630ec63f810e-tainted