Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Unified Diff: deps/third_party/valgrind/scripts/build-valgrind-for-chromium.sh

Issue 536082: Remove un-needed Valgrind binaries and strip some of the remaining.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/
Patch Set: Created 10 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: deps/third_party/valgrind/scripts/build-valgrind-for-chromium.sh
===================================================================
--- deps/third_party/valgrind/scripts/build-valgrind-for-chromium.sh (revision 36404)
+++ deps/third_party/valgrind/scripts/build-valgrind-for-chromium.sh (working copy)
@@ -157,6 +157,26 @@
maybe_build_gdb_for_mac "$OUTPUT_DIR"
+ # Delete un-needed stuff from the $OUTPUT_DIR
+ # TODO(timurrrr): probably, we should just don't build the unused tools
+ cd $OUTPUT_DIR
+ rm -rf include
+ rm -rf lib/pkgconfig lib/*.a
+ rm bin/*cg_* bin/callgrind*
+ cd lib/valgrind
+ rm -rf *.a \
+ *drd* \
+ *exp-* \
+ *none* \
+ *lackey* \
+ *massif* \
+ *helgrind* \
+ *callgrind* \
+ *cachegrind*
+
+ # Strip all binaries except "vgpreload" (we do need their symbols).
+ strip `ls -1 | grep "tsan\|memcheck" | grep -v "dSYM" | grep -v "vgpreload"`
+
# We're done
cd $THISDIR
# }}}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698