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

Unified Diff: tools/relocation_packer/relocation_packer.gyp

Issue 310483003: Add a host tool to pack R_ARM_RELATIVE relocations in libchrome.<ver>.so. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 7 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: tools/relocation_packer/relocation_packer.gyp
diff --git a/tools/relocation_packer/relocation_packer.gyp b/tools/relocation_packer/relocation_packer.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..ca932a6e362fdf56f15e289eed813c7489e9b765
--- /dev/null
+++ b/tools/relocation_packer/relocation_packer.gyp
@@ -0,0 +1,65 @@
+# Copyright (c) 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+{
+ 'targets': [
+ {
+ 'target_name': 'pack_relocations',
rmcilroy 2014/06/02 15:16:35 nit - lib_relocation_packer ?
simonb (inactive) 2014/06/04 16:40:35 Done.
+ 'toolsets': ['host'],
+ 'type': 'static_library',
+ 'dependencies': [
+ '../../third_party/elfutils/elfutils.gyp:libelf',
+ ],
+ 'include_dirs': [
+ 'src',
rmcilroy 2014/06/02 15:16:35 Is this needed? Doesn't seem to be required in th
simonb (inactive) 2014/06/04 16:40:35 Done.
+ ],
+ 'sources': [
+ 'src/relocation_packer_debug.cc',
+ 'src/relocation_packer_elf_file.cc',
+ 'src/relocation_packer_leb128.cc',
+ 'src/relocation_packer_packer.cc',
+ 'src/relocation_packer_rle.cc',
rmcilroy 2014/06/02 15:16:35 optional nit - maybe avoid the "relocation_packer_
simonb (inactive) 2014/06/04 16:40:35 Done.
+ ],
+ },
+ {
+ 'target_name': 'relocation_packer',
+ 'toolsets': ['host'],
+ 'type': 'executable',
+ 'dependencies': [
+ '../../third_party/elfutils/elfutils.gyp:libelf',
+ 'pack_relocations',
+ ],
+ 'include_dirs': [
+ 'src',
+ ],
+ 'sources': [
+ 'src/relocation_packer_main.cc',
+ ],
+ 'link_settings': {
+ 'libraries': [
+ '-lelf',
rmcilroy 2014/06/02 15:16:35 Do you need this? I though depending on the libel
simonb (inactive) 2014/06/04 16:40:35 Done.
+ ],
+ },
+ },
+ {
+ 'target_name': 'relocation_packer_unittests',
+ 'toolsets': ['host'],
+ 'type': 'executable',
+ 'dependencies': [
+ '../../testing/gtest.gyp:gtest',
+ 'pack_relocations',
+ ],
+ 'include_dirs': [
+ '../..',
+ ],
+ 'sources': [
+ 'src/relocation_packer_elf_file_unittest.cc',
+ 'src/relocation_packer_leb128_unittest.cc',
+ 'src/relocation_packer_packer_unittest.cc',
+ 'src/relocation_packer_rle_unittest.cc',
+ 'src/run_all_unittests.cc',
+ ],
+ },
+ ],
+}

Powered by Google App Engine
This is Rietveld 408576698