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

Side by Side Diff: projects.py

Issue 145293006: Sort issues by their age in the commit queue. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/commit-queue
Patch Set: Addressing the comments Created 6 years, 10 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
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 """Define the supported projects.""" 4 """Define the supported projects."""
5 5
6 import json 6 import json
7 import logging 7 import logging
8 import os 8 import os
9 import re 9 import re
10 import sys 10 import sys
(...skipping 145 matching lines...) Expand 10 before | Expand all | Expand 10 after
156 'blink', 156 'blink',
157 user, 157 user,
158 None, 158 None,
159 'svn://svn.chromium.org/blink/trunk', 159 'svn://svn.chromium.org/blink/trunk',
160 []) 160 [])
161 context_obj = context.Context( 161 context_obj = context.Context(
162 rietveld_obj, 162 rietveld_obj,
163 local_checkout, 163 local_checkout,
164 async_push.AsyncPush( 164 async_push.AsyncPush(
165 'https://chromium-status.appspot.com/cq', 165 'https://chromium-status.appspot.com/cq',
166 _chromium_status_pwd(root_dir))) 166 _chromium_status_pwd(root_dir)),
167 user)
167 168
168 project_bases = [ 169 project_bases = [
169 '^%s/trunk(|/.*)$' % re.escape(base) for base in BLINK_SVN_BASES] 170 '^%s/trunk(|/.*)$' % re.escape(base) for base in BLINK_SVN_BASES]
170 project_bases.append( 171 project_bases.append(
171 r'^https?\:\/\/chromium.googlesource.com\/chromium\/blink(?:\.git)?%s$' % 172 r'^https?\:\/\/chromium.googlesource.com\/chromium\/blink(?:\.git)?%s$' %
172 BRANCH_MATCH) 173 BRANCH_MATCH)
173 verifiers_no_patch = [ 174 verifiers_no_patch = [
174 project_base.ProjectBaseUrlVerifier(project_bases), 175 project_base.ProjectBaseUrlVerifier(project_bases),
175 reviewer_lgtm.ReviewerLgtmVerifier( 176 reviewer_lgtm.ReviewerLgtmVerifier(
176 _get_chromium_committers(), 177 _get_chromium_committers(),
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
238 'chromium', 239 'chromium',
239 user, 240 user,
240 None, 241 None,
241 'svn://svn.chromium.org/chrome/trunk/src', 242 'svn://svn.chromium.org/chrome/trunk/src',
242 [chromium_copyright.process]) 243 [chromium_copyright.process])
243 context_obj = context.Context( 244 context_obj = context.Context(
244 rietveld_obj, 245 rietveld_obj,
245 local_checkout, 246 local_checkout,
246 async_push.AsyncPush( 247 async_push.AsyncPush(
247 'https://chromium-status.appspot.com/cq', 248 'https://chromium-status.appspot.com/cq',
248 _chromium_status_pwd(root_dir))) 249 _chromium_status_pwd(root_dir)),
250 user)
249 251
250 project_bases = [ 252 project_bases = [
251 '^%s/trunk/src(|/.*)$' % re.escape(base) for base in CHROME_SVN_BASES] 253 '^%s/trunk/src(|/.*)$' % re.escape(base) for base in CHROME_SVN_BASES]
252 254
253 aliases = ( 255 aliases = (
254 # Old path. 256 # Old path.
255 'git.chromium.org/git/chromium', 257 'git.chromium.org/git/chromium',
256 # New path. 258 # New path.
257 'git.chromium.org/chromium/src', 259 'git.chromium.org/chromium/src',
258 'chromium.googlesource.com/chromium/src', 260 'chromium.googlesource.com/chromium/src',
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 'skia', 512 'skia',
511 user, 513 user,
512 None, 514 None,
513 'https' + naked_url) 515 'https' + naked_url)
514 context_obj = context.Context( 516 context_obj = context.Context(
515 rietveld_obj, 517 rietveld_obj,
516 local_checkout, 518 local_checkout,
517 async_push.AsyncPush( 519 async_push.AsyncPush(
518 'https://skia-tree-status.appspot.com/cq', 520 'https://skia-tree-status.appspot.com/cq',
519 _skia_status_pwd(root_dir)), 521 _skia_status_pwd(root_dir)),
522 user,
520 server_hooks_missing=True) 523 server_hooks_missing=True)
521 524
522 project_bases = [ 525 project_bases = [
523 '^%s(|/.*)$' % re.escape(base + naked_url) for base in ('http', 'https') 526 '^%s(|/.*)$' % re.escape(base + naked_url) for base in ('http', 'https')
524 ] 527 ]
525 project_bases.append( 528 project_bases.append(
526 r'^https?\:\/\/skia.googlesource.com\/skia(?:\.git)?%s$' % 529 r'^https?\:\/\/skia.googlesource.com\/skia(?:\.git)?%s$' %
527 BRANCH_MATCH) 530 BRANCH_MATCH)
528 verifiers_no_patch = [ 531 verifiers_no_patch = [
529 project_base.ProjectBaseUrlVerifier(project_bases), 532 project_base.ProjectBaseUrlVerifier(project_bases),
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
576 root_dir, 579 root_dir,
577 'nacl', 580 'nacl',
578 user, 581 user,
579 None, 582 None,
580 'svn://svn.chromium.org/native_client/' + offset) 583 'svn://svn.chromium.org/native_client/' + offset)
581 context_obj = context.Context( 584 context_obj = context.Context(
582 rietveld_obj, 585 rietveld_obj,
583 local_checkout, 586 local_checkout,
584 async_push.AsyncPush( 587 async_push.AsyncPush(
585 'https://nativeclient-status.appspot.com/cq', 588 'https://nativeclient-status.appspot.com/cq',
586 _chromium_status_pwd(root_dir))) 589 _chromium_status_pwd(root_dir)),
590 user)
587 591
588 host_aliases = SVN_HOST_ALIASES + [ 592 host_aliases = SVN_HOST_ALIASES + [
589 'http://src.chromium.org', 'https://src.chromium.org'] 593 'http://src.chromium.org', 'https://src.chromium.org']
590 svn_bases = [i + '/native_client' for i in host_aliases] 594 svn_bases = [i + '/native_client' for i in host_aliases]
591 project_bases = [ 595 project_bases = [
592 '^%s/%s(|/.*)$' % (re.escape(base), offset) for base in svn_bases 596 '^%s/%s(|/.*)$' % (re.escape(base), offset) for base in svn_bases
593 ] 597 ]
594 aliases = ( 598 aliases = (
595 'git.chromium.org/native_client/src/native_client', 599 'git.chromium.org/native_client/src/native_client',
596 'chromium.googlesource.com/native_client/src/native_client', 600 'chromium.googlesource.com/native_client/src/native_client',
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
646 root_dir, 650 root_dir,
647 'gyp', 651 'gyp',
648 user, 652 user,
649 None, 653 None,
650 'https' + naked_url) 654 'https' + naked_url)
651 context_obj = context.Context( 655 context_obj = context.Context(
652 rietveld_obj, 656 rietveld_obj,
653 local_checkout, 657 local_checkout,
654 async_push.AsyncPush( 658 async_push.AsyncPush(
655 'https://chromium-status.appspot.com/cq/receiver', 659 'https://chromium-status.appspot.com/cq/receiver',
656 _chromium_status_pwd(root_dir))) 660 _chromium_status_pwd(root_dir)),
661 user)
657 662
658 project_bases = [ 663 project_bases = [
659 '^%s(|/.*)$' % re.escape(base + naked_url) for base in ('http', 'https') 664 '^%s(|/.*)$' % re.escape(base + naked_url) for base in ('http', 'https')
660 ] 665 ]
661 verifiers_no_patch = [ 666 verifiers_no_patch = [
662 project_base.ProjectBaseUrlVerifier(project_bases), 667 project_base.ProjectBaseUrlVerifier(project_bases),
663 reviewer_lgtm.ReviewerLgtmVerifier( 668 reviewer_lgtm.ReviewerLgtmVerifier(
664 _get_gyp_committers(), 669 _get_gyp_committers(),
665 [re.escape(user)]), 670 [re.escape(user)]),
666 ] 671 ]
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
738 os.path.basename(path), 743 os.path.basename(path),
739 user, 744 user,
740 None, 745 None,
741 'svn://svn.chromium.org/chrome/trunk/' + path, 746 'svn://svn.chromium.org/chrome/trunk/' + path,
742 [chromium_copyright.process]) 747 [chromium_copyright.process])
743 context_obj = context.Context( 748 context_obj = context.Context(
744 rietveld_obj, 749 rietveld_obj,
745 local_checkout, 750 local_checkout,
746 async_push.AsyncPush( 751 async_push.AsyncPush(
747 'https://chromium-status.appspot.com/cq', 752 'https://chromium-status.appspot.com/cq',
748 _chromium_status_pwd(root_dir))) 753 _chromium_status_pwd(root_dir)),
754 user)
749 755
750 verifiers_no_patch = [ 756 verifiers_no_patch = [
751 project_base.ProjectBaseUrlVerifier(project_bases), 757 project_base.ProjectBaseUrlVerifier(project_bases),
752 reviewer_lgtm.ReviewerLgtmVerifier( 758 reviewer_lgtm.ReviewerLgtmVerifier(
753 _get_chromium_committers(), 759 _get_chromium_committers(),
754 [re.escape(user)]), 760 [re.escape(user)]),
755 ] 761 ]
756 verifiers = [ 762 verifiers = [
757 presubmit_check.PresubmitCheckVerifier(context_obj, timeout=900), 763 presubmit_check.PresubmitCheckVerifier(context_obj, timeout=900),
758 ] 764 ]
759 765
760 return pending_manager.PendingManager( 766 return pending_manager.PendingManager(
761 context_obj, 767 context_obj,
762 verifiers_no_patch, 768 verifiers_no_patch,
763 verifiers) 769 verifiers)
764 770
765 771
766 def supported_projects(): 772 def supported_projects():
767 """List the projects that can be managed by the commit queue.""" 773 """List the projects that can be managed by the commit queue."""
768 return sorted( 774 return sorted(
769 x[5:] for x in dir(sys.modules[__name__]) if x.startswith('_gen_')) 775 x[5:] for x in dir(sys.modules[__name__]) if x.startswith('_gen_'))
770 776
771 777
772 def load_project(project, user, root_dir, rietveld_obj, no_try): 778 def load_project(project, user, root_dir, rietveld_obj, no_try):
773 """Loads the specified project.""" 779 """Loads the specified project."""
774 assert os.path.isabs(root_dir) 780 assert os.path.isabs(root_dir)
775 return getattr(sys.modules[__name__], '_gen_' + project)( 781 return getattr(sys.modules[__name__], '_gen_' + project)(
776 user, root_dir, rietveld_obj, no_try) 782 user, root_dir, rietveld_obj, no_try)
OLDNEW
« pending_manager.py ('K') | « pending_manager.py ('k') | tests/commit_queue_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698