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

Side by Side Diff: build/android/gyp/pack_relocations.py

Issue 2336173003: Fix android depfiles to always list GN's outputs[0] (Closed)
Patch Set: fix cronet_package Created 4 years, 3 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 unified diff | Download patch
« no previous file with comments | « build/android/gyp/locale_pak_resources.py ('k') | build/android/gyp/util/build_utils.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # 2 #
3 # Copyright 2014 The Chromium Authors. All rights reserved. 3 # Copyright 2014 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 """Pack relocations in a library (or copy unchanged). 7 """Pack relocations in a library (or copy unchanged).
8 8
9 If --enable-packing and --configuration-name=='Release', invoke the 9 If --enable-packing and --configuration-name=='Release', invoke the
10 relocation_packer tool to pack the .rel.dyn or .rela.dyn section in the given 10 relocation_packer tool to pack the .rel.dyn or .rela.dyn section in the given
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 if enable_packing: 84 if enable_packing:
85 PackLibraryRelocations(options.android_pack_relocations, 85 PackLibraryRelocations(options.android_pack_relocations,
86 library_path, 86 library_path,
87 output_path) 87 output_path)
88 else: 88 else:
89 CopyLibraryUnchanged(library_path, output_path) 89 CopyLibraryUnchanged(library_path, output_path)
90 90
91 if options.filelistjson: 91 if options.filelistjson:
92 build_utils.WriteJson({ 'files': output_paths }, options.filelistjson) 92 build_utils.WriteJson({ 'files': output_paths }, options.filelistjson)
93 output_paths.append(options.filelistjson)
93 94
94 if options.depfile: 95 if options.depfile:
95 build_utils.WriteDepfile( 96 build_utils.WriteDepfile(options.depfile, output_paths[-1], libraries)
96 options.depfile,
97 libraries + build_utils.GetPythonDependencies())
98 97
99 if options.stamp: 98 if options.stamp:
100 build_utils.Touch(options.stamp) 99 build_utils.Touch(options.stamp)
101 100
102 return 0 101 return 0
103 102
104 103
105 if __name__ == '__main__': 104 if __name__ == '__main__':
106 sys.exit(main(sys.argv[1:])) 105 sys.exit(main(sys.argv[1:]))
OLDNEW
« no previous file with comments | « build/android/gyp/locale_pak_resources.py ('k') | build/android/gyp/util/build_utils.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698