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

Side by Side Diff: native_client_sdk/src/build_tools/build_sdk.py

Issue 11228013: [NaCl SDK] Refactor sdk_update*. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix build_updater Created 8 years, 1 month 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 | « native_client_sdk/PRESUBMIT.py ('k') | native_client_sdk/src/build_tools/build_updater.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 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Entry point for both build and try bots 6 """Entry point for both build and try bots
7 7
8 This script is invoked from XXX, usually without arguments 8 This script is invoked from XXX, usually without arguments
9 to package an SDK. It automatically determines whether 9 to package an SDK. It automatically determines whether
10 this SDK is for mac, win, linux. 10 this SDK is for mac, win, linux.
(...skipping 768 matching lines...) Expand 10 before | Expand all | Expand 10 after
779 manifest_name = 'naclsdk_manifest2.json' 779 manifest_name = 'naclsdk_manifest2.json'
780 with open(os.path.join(SERVER_DIR, manifest_name), 'wb') as \ 780 with open(os.path.join(SERVER_DIR, manifest_name), 'wb') as \
781 manifest_stream: 781 manifest_stream:
782 manifest_stream.write(manifest.GetDataAsString()) 782 manifest_stream.write(manifest.GetDataAsString())
783 783
784 # use newly built sdk updater to pull this bundle 784 # use newly built sdk updater to pull this bundle
785 buildbot_common.BuildStep('Update from local server') 785 buildbot_common.BuildStep('Update from local server')
786 naclsdk_sh = os.path.join(OUT_DIR, 'nacl_sdk', 'naclsdk') 786 naclsdk_sh = os.path.join(OUT_DIR, 'nacl_sdk', 'naclsdk')
787 if platform == 'win': 787 if platform == 'win':
788 naclsdk_sh += '.bat' 788 naclsdk_sh += '.bat'
789 buildbot_common.Run([naclsdk_sh, '-U', 789 buildbot_common.Run([naclsdk_sh, 'update', 'pepper_' + pepper_ver,
790 server.GetURL(manifest_name), 'update', 'pepper_' + pepper_ver]) 790 '-U', server.GetURL(manifest_name), '-v'])
791 791
792 # Return the new pepper directory as the one inside the downloaded SDK. 792 # Return the new pepper directory as the one inside the downloaded SDK.
793 return os.path.join(OUT_DIR, 'nacl_sdk', 'pepper_' + pepper_ver) 793 return os.path.join(OUT_DIR, 'nacl_sdk', 'pepper_' + pepper_ver)
794 794
795 # kill server 795 # kill server
796 finally: 796 finally:
797 if server: 797 if server:
798 server.Shutdown() 798 server.Shutdown()
799 799
800 800
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
1011 # Archive on non-trybots. 1011 # Archive on non-trybots.
1012 if options.archive or buildbot_common.IsSDKBuilder(): 1012 if options.archive or buildbot_common.IsSDKBuilder():
1013 BuildStepArchiveBundle(pepper_ver, clnumber, tarfile) 1013 BuildStepArchiveBundle(pepper_ver, clnumber, tarfile)
1014 BuildStepArchiveSDKTools() 1014 BuildStepArchiveSDKTools()
1015 1015
1016 return 0 1016 return 0
1017 1017
1018 1018
1019 if __name__ == '__main__': 1019 if __name__ == '__main__':
1020 sys.exit(main(sys.argv)) 1020 sys.exit(main(sys.argv))
OLDNEW
« no previous file with comments | « native_client_sdk/PRESUBMIT.py ('k') | native_client_sdk/src/build_tools/build_updater.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698