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

Issue 265001: Make programs in src/tools/linux build via the automake build system (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by Ted Mielczarek
Modified:
13 years, 1 month ago
Reviewers:
jimb
CC:
google-breakpad-dev_googlegroups.com
Visibility:
Public.

Description

I got tired of the fiddly Makefile build for the Linux dump_syms tool, so I went
ahead and autoconfiscated it along with its unit tests, and the other things in
src/tools/linux. There are a few fiddly code changes in here, mostly because I
lumped all the dump_syms unit tests into one binary, and we had a few tests that
used the same test fixture name in different ways. This builds and passes all
tests on my Ubuntu system.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M Makefile.am View 4 chunks +81 lines, -1 line 0 comments Download
M Makefile.in View 21 chunks +865 lines, -12 lines 0 comments Download
M configure View 8 chunks +230 lines, -6 lines 0 comments Download
M configure.ac View 2 chunks +21 lines, -2 lines 0 comments Download
M src/common/dwarf_cu_to_module_unittest.cc View 4 chunks +16 lines, -16 lines 0 comments Download
M src/common/dwarf_line_to_module_unittest.cc View 3 chunks +7 lines, -7 lines 0 comments Download
M src/common/stabs_reader.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/config.h.in View 1 chunk +30 lines, -0 lines 0 comments Download
R src/tools/linux/dump_syms/Makefile View 1 chunk +0 lines, -410 lines 0 comments Download
R src/tools/linux/symupload/Makefile View 1 chunk +0 lines, -24 lines 0 comments Download

Messages

Total messages: 3
Ted Mielczarek
13 years, 1 month ago #1
jimb
"Fiddly"? That's my best GNU Make style! [very hurt] I'll take a look at this ...
13 years, 1 month ago #2
jimb
13 years, 1 month ago #3
LGTM
Sign in to reply to this message.

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