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

Unified Diff: native_client_sdk/src/build_tools/make_rules.py

Issue 10827363: Add Linux Host builds to NaCl SDK (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 4 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: native_client_sdk/src/build_tools/make_rules.py
===================================================================
--- native_client_sdk/src/build_tools/make_rules.py (revision 151748)
+++ native_client_sdk/src/build_tools/make_rules.py (working copy)
@@ -109,11 +109,6 @@
'Release': '<TAB>$(<LINK>) /DLL /OUT:$@ $(<PROJ>_LDFLAGS) /LIBPATH:$(NACL_SDK_ROOT)/lib/win_x86_32_host/Release $^ <LIBLIST> $(WIN_LDFLAGS)'
}
-WIN_LAUNCH_RULES = """
-HOST_ARGS:=--register-pepper-plugins=$(abspath win/<proj>.dll);application/x-nacl
-LAUNCH_HOST: CHECK_FOR_CHROME all
-<TAB>$(CHROME_PATH) $(HOST_ARGS) "localhost:5103/index_win.html"
-"""
#
# Lib rules for various platforms.
@@ -170,9 +165,9 @@
'DEFINE': '-D%s',
'INCLUDE': '-I%s',
'LIBRARY': '-l%s',
- 'MAIN': '<tc>/<config>/lib<proj>_<ARCH>.so',
- 'NMFMAIN': '<tc>/<config>/lib<proj>_<ARCH>.so',
- 'SO': '<tc>/<config>/lib<proj>_<ARCH>.so',
+ 'MAIN': '<tc>/<config>/lib<proj>.so',
+ 'NMFMAIN': '<tc>/<config>/lib<proj>.so',
+ 'SO': '<tc>/<config>/lib<proj>.so',
'LIB': '$(NACL_SDK_ROOT)/lib/linux_<ARCH>_host/<config>/lib<proj>.a',
}
@@ -203,16 +198,11 @@
#
# Various Architectures
#
-LINUX_32 = {
- '<arch>': '32',
- '<ARCH>': 'x86_32',
- '<MACH>': '-m32',
+LINUX = {
+ '<arch>': '',
+ '<ARCH>': '',
+ '<MACH>': '',
}
-LINUX_64 = {
- '<arch>': '64',
- '<ARCH>': 'x86_64',
- '<MACH>': '-m64',
-}
NACL_X86_32 = {
'<arch>': '32',
'<ARCH>': 'x86_32',
@@ -281,7 +271,7 @@
'TOOL': WIN_TOOL
},
'linux' : {
- 'ARCHES': [LINUX_32, LINUX_64],
+ 'ARCHES': [LINUX],
'DEFS': LINUX_DEFAULTS,
'CC': SO_CC_RULES,
'CXX': SO_CC_RULES,
@@ -367,6 +357,13 @@
return '%s_%s_%s_%s_O' % (self.project.upper(), self.tc.upper(),
self.cfg.upper(), self.arch['<ARCH>'])
+ def GetPepperPlugin(self):
+ plugin = self.Replace(BUILD_RULES[self.tc]['TOOL']['MAIN'])
+ text ='PPAPI_<CONFIG>:=$(abspath %s)' % plugin
binji 2012/08/16 00:17:52 nit: remove extra space before =, add space after
noelallen1 2012/08/16 17:58:30 Done.
+ text +=';application/x-ppapi-%s\n' % self.vars['<config>'].lower()
binji 2012/08/16 00:17:52 nit: add space after +=
noelallen1 2012/08/16 17:58:30 Done.
+ return self.Replace(text)
+
+
def SetArch(self, arch):
self.arch = arch
for key in arch:

Powered by Google App Engine
This is Rietveld 408576698