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

Unified Diff: scripts/common.sh

Issue 6813106: Remove an unused out-of-date function (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/deps/third_party/valgrind/
Patch Set: Created 9 years, 8 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: scripts/common.sh
===================================================================
--- scripts/common.sh (revision 81227)
+++ scripts/common.sh (working copy)
@@ -34,58 +34,6 @@
cd $SOURCE_DIR/valgrind
}
-checkout_and_patch_valgrind() {
- # $1 = Valgrind revision
- # $2 = VEX revision
- # $3 = source dir
- # Checkout Valgrind, apply our patches to Valgrind.
- # The source will be put in $VG_SRC_DIR/valgrind-source
- mkdir -p "$VG_SRC_DIR"
- cd "$VG_SRC_DIR"
- VG_REV="$1"
- VEX_REV="$2"
- SOURCE_DIR="$3"
- SVN_URI="svn://svn.valgrind.org/valgrind/trunk"
-
- test -d "$SOURCE_DIR" && rm -rf "$SOURCE_DIR"
- mkdir "$SOURCE_DIR"
-
- # Check out latest version that following patches known to apply against
- svn co -r "$VG_REV" "$SVN_URI" "$SOURCE_DIR"
-
- cd "$SOURCE_DIR"
- # Make sure svn gets the right version of the external VEX repo, too
- svn up -r "$VEX_REV" VEX/
-
- # Apply common patches to Valgrind {{{1
- # Work around bug https://bugs.kde.org/show_bug.cgi?id=162848
- # "fork() not handled properly"
- if system_is_snow_leopard
- then
- patch -p0 -i "${THISDIR}/fork.10.6.patch"
- else
- patch -p0 -i "${THISDIR}/fork.patch"
- fi
-
- # Work around bug https://bugs.kde.org/show_bug.cgi?id=164485
- # "VG_N_SEGNAMES and VG_N_SEGMENTS are (still) too small"
- patch -p0 -i "${THISDIR}/limits.patch"
-
- # Add feature bug https://bugs.kde.org/show_bug.cgi?id=205000
- # "Need library load address in log files"
- patch -p0 -i "${THISDIR}/xml-loadadr.patch"
-
- # Fix/work around https://bugs.kde.org/show_bug.cgi?id=210481
- # which prevented valgrind from handling v8 on 64 bits
- patch -p0 -i "${THISDIR}/vbug210481.patch"
-
- # Add intercepts for tcmalloc memory functions.
- # The corresponding feature request for Valgrind is at
- # https://bugs.kde.org/show_bug.cgi?id=219156.
- patch -p0 -i "${THISDIR}/intercept_tcmalloc.patch"
- # }}}
-}
-
build_valgrind() {
# Build Valgrind from sources in the current directory.
# $1 = output directory name for binaries
« 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