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

Unified Diff: third_party/instrumented_libraries/fix_rpaths

Issue 50423003: Adds a flag "use_instrumented_libraries" and corresponding target with 2 simple libs (Closed) Base URL: https://src.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 2 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
Index: third_party/instrumented_libraries/fix_rpaths
===================================================================
--- third_party/instrumented_libraries/fix_rpaths (revision 0)
+++ third_party/instrumented_libraries/fix_rpaths (revision 0)
@@ -0,0 +1,16 @@
+#!/bin/bash
+#
+# Changes all RPATHs from XORIGIN to $ORIGIN
Alexander Potapenko 2013/10/29 13:36:11 "all RPATHs in a given directory"
alextaran1 2013/10/30 11:16:29 Done.
+
+# Fixes rpath from XORIGIN to $ORIGIN in a single file $1
Alexander Potapenko 2013/10/29 13:36:11 Period at the end of the comment here and everywhe
alextaran1 2013/10/30 11:16:29 Done.
+function fix_rpath {
+ chrpath -r $(chrpath $1 | cut -d " " -f 2 | sed s/XORIGIN/\$ORIGIN/g \
+ | sed s/RPATH=//g) $1
+}
+
+for i in $(find $1 | grep "\.so$"); do
+ fix_rpath $i > /dev/null
Alexander Potapenko 2013/10/29 13:36:11 Why suppress the output? Does chrpath print too mu
alextaran1 2013/10/30 11:16:29 It prints 2 lines for each file: old rpath and new
+done
+
+# Mark that rpaths are fixed
+touch $1/rpaths.fixed.txt
Property changes on: third_party/instrumented_libraries/fix_rpaths
___________________________________________________________________
Added: svn:executable
+ *

Powered by Google App Engine
This is Rietveld 408576698