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

Side by Side Diff: chrome/chrome_browser_extensions.gypi

Issue 10914296: Fix missing dependency from app_host to chrome_version_resources (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/installer/launcher_support/chrome_launcher_support.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'targets': [ 6 'targets': [
7 { 7 {
8 'target_name': 'browser_extensions', 8 'target_name': 'browser_extensions',
9 'type': 'static_library', 9 'type': 'static_library',
10 'variables': { 'enable_wexit_time_destructors': 1, }, 10 'variables': { 'enable_wexit_time_destructors': 1, },
(...skipping 816 matching lines...) Expand 10 before | Expand all | Expand 10 after
827 'include_dirs': [ 827 'include_dirs': [
828 '..', 828 '..',
829 ], 829 ],
830 'direct_dependent_settings': { 830 'direct_dependent_settings': {
831 'include_dirs': [ 831 'include_dirs': [
832 '..', 832 '..',
833 ], 833 ],
834 }, 834 },
835 'dependencies': [ 835 'dependencies': [
836 '<(DEPTH)/base/base.gyp:base', 836 '<(DEPTH)/base/base.gyp:base',
837 '<(DEPTH)/chrome/chrome.gyp:chrome_version_resources',
erikwright (departed) 2012/09/14 20:06:33 Sorry, should have uploaded an initial patchset eq
837 '<(DEPTH)/chrome/chrome.gyp:launcher_support', 838 '<(DEPTH)/chrome/chrome.gyp:launcher_support',
838 '<(DEPTH)/google_update/google_update.gyp:google_update', 839 '<(DEPTH)/google_update/google_update.gyp:google_update',
839 ], 840 ],
840 'sources': [ 841 'sources': [
841 'browser/extensions/app_host/app_host.rc', 842 'browser/extensions/app_host/app_host.rc',
842 'browser/extensions/app_host/app_host_main.cc', 843 'browser/extensions/app_host/app_host_main.cc',
843 'browser/extensions/app_host/binaries_installer.cc', 844 'browser/extensions/app_host/binaries_installer.cc',
844 'browser/extensions/app_host/binaries_installer.h', 845 'browser/extensions/app_host/binaries_installer.h',
845 'browser/extensions/app_host/binaries_installer_internal.cc', 846 'browser/extensions/app_host/binaries_installer_internal.cc',
846 'browser/extensions/app_host/binaries_installer_internal.h', 847 'browser/extensions/app_host/binaries_installer_internal.h',
848 'browser/extensions/app_host/update.cc',
849 'browser/extensions/app_host/update.h',
850 '<(SHARED_INTERMEDIATE_DIR)/chrome_version/app_host_exe_version.rc',
847 ], 851 ],
848 }, 852 },
849 ], 853 ],
850 }, ], # 'OS=="win"' 854 }, ], # 'OS=="win"'
851 ], # 'conditions' 855 ], # 'conditions'
852 } 856 }
853 857
OLDNEW
« no previous file with comments | « chrome/chrome.gyp ('k') | chrome/installer/launcher_support/chrome_launcher_support.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698