Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: sitescripts/extensions/bin/createNightlies.py

Issue 29538644: Issue 5642 - Fix Windows Store devbuild upload routine (Closed)
Patch Set: Remove redundant code and simplify formatting Created Sept. 8, 2017, 11:11 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # This file is part of the Adblock Plus web scripts, 1 # This file is part of the Adblock Plus web scripts,
2 # Copyright (C) 2006-present eyeo GmbH 2 # Copyright (C) 2006-present eyeo GmbH
3 # 3 #
4 # Adblock Plus is free software: you can redistribute it and/or modify 4 # Adblock Plus is free software: you can redistribute it and/or modify
5 # it under the terms of the GNU General Public License version 3 as 5 # it under the terms of the GNU General Public License version 3 as
6 # published by the Free Software Foundation. 6 # published by the Free Software Foundation.
7 # 7 #
8 # Adblock Plus is distributed in the hope that it will be useful, 8 # Adblock Plus is distributed in the hope that it will be useful,
9 # but WITHOUT ANY WARRANTY; without even the implied warranty of 9 # but WITHOUT ANY WARRANTY; without even the implied warranty of
10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 10 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
(...skipping 600 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 # Create submission 611 # Create submission
612 # https://msdn.microsoft.com/en-us/windows/uwp/monetize/create-an-app-su bmission 612 # https://msdn.microsoft.com/en-us/windows/uwp/monetize/create-an-app-su bmission
613 request = urllib2.Request(submissions_path, '', headers) 613 request = urllib2.Request(submissions_path, '', headers)
614 request.get_method = lambda: 'POST' 614 request.get_method = lambda: 'POST'
615 with contextlib.closing(opener.open(request)) as response: 615 with contextlib.closing(opener.open(request)) as response:
616 submission = json.load(response) 616 submission = json.load(response)
617 617
618 submission_id = submission['id'] 618 submission_id = submission['id']
619 file_upload_url = submission['fileUploadUrl'] 619 file_upload_url = submission['fileUploadUrl']
620 620
621 # Update submission
622 submission['applicationPackages'][0]['fileStatus'] = 'PendingDelete'
623 submission['applicationPackages'].append(
624 {
Sebastian Noack 2017/09/08 17:30:31 FWIW, it would be OK to put the parenthesis an bra
625 'fileStatus': 'PendingUpload',
626 'fileName': os.path.basename(self.path)
tlucas 2017/09/08 11:26:38 nit: Please add a comma to the last line of a mult
627 }
628 )
629
621 new_submission_path = '{}/{}'.format(submissions_path, 630 new_submission_path = '{}/{}'.format(submissions_path,
622 submission_id) 631 submission_id)
632 new_submission = json.dumps(submission)
623 633
624 request = urllib2.Request(new_submission_path, None, headers) 634 request = urllib2.Request(new_submission_path, new_submission, headers)
635 request.get_method = lambda: 'PUT'
625 opener.open(request).close() 636 opener.open(request).close()
626 637
627 self.upload_appx_file_to_windows_store(file_upload_url) 638 self.upload_appx_file_to_windows_store(file_upload_url)
628 639
629 # Commit submission 640 # Commit submission
630 # https://msdn.microsoft.com/en-us/windows/uwp/monetize/commit-an-app-su bmission 641 # https://msdn.microsoft.com/en-us/windows/uwp/monetize/commit-an-app-su bmission
631 commit_path = '{}/commit'.format(new_submission_path) 642 commit_path = '{}/commit'.format(new_submission_path)
632 request = urllib2.Request(commit_path, '', headers) 643 request = urllib2.Request(commit_path, '', headers)
633 request.get_method = lambda: 'POST' 644 request.get_method = lambda: 'POST'
634 with contextlib.closing(opener.open(request)) as response: 645 with contextlib.closing(opener.open(request)) as response:
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
724 except Exception as ex: 735 except Exception as ex:
725 logging.error('The build for %s failed:', repo) 736 logging.error('The build for %s failed:', repo)
726 logging.exception(ex) 737 logging.exception(ex)
727 738
728 file = open(nightlyConfigFile, 'wb') 739 file = open(nightlyConfigFile, 'wb')
729 nightlyConfig.write(file) 740 nightlyConfig.write(file)
730 741
731 742
732 if __name__ == '__main__': 743 if __name__ == '__main__':
733 main() 744 main()
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld