Daniel Jacques
							
						 
						
							 
							
							
							
								
							
								31f3e63cbc 
								
							
								 
							
						 
						
							
							
								
								Revert "Revert "[tryserver] Remove unused methods.""  
							
							 
							
							... 
							
							
							
							This reverts commit cc27ecb005 .
Reason for revert: Published this method downstream to internal repo.
Original change's description:
> Revert "[tryserver] Remove unused methods."
> 
> This reverts commit 133ac1ab8d .
> 
> Reason for revert: Turns out these are used by internal recipes.
> 
> Original change's description:
> > [tryserver] Remove unused methods.
> > 
> > R=agable@chromium.org, dnj@chromium.org, hinoka@chromium.org 
> > 
> > Bug:
> > Change-Id: I82a11f31c8c1c4c4a2b461090e5aee637f8821c2
> > Reviewed-on: https://chromium-review.googlesource.com/569019 
> > Reviewed-by: Nodir Turakulov <nodir@chromium.org>
> > Reviewed-by: Aaron Gable <agable@chromium.org>
> > Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
> 
> TBR=iannucci@chromium.org,hinoka@chromium.org,agable@chromium.org,dnj@chromium.org,nodir@chromium.org
> 
> Change-Id: Ib1d4192520a36f649f1f9b31e2928027667311d4
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/570988 
> Reviewed-by: Daniel Jacques <dnj@chromium.org>
> Commit-Queue: Daniel Jacques <dnj@chromium.org>
TBR=iannucci@chromium.org,hinoka@chromium.org,agable@chromium.org,dnj@chromium.org,nodir@chromium.org
Change-Id: Id7ac3555d40162e4204ceac5e96c2e3864c67aba
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/570781 
Reviewed-by: Daniel Jacques <dnj@chromium.org>
Commit-Queue: Daniel Jacques <dnj@chromium.org> 
							
						 
						
							8 years ago  
						
					 
				
					
						
							
							
								 
								Daniel Jacques
							
						 
						
							 
							
							
							
								
							
								cc27ecb005 
								
							
								 
							
						 
						
							
							
								
								Revert "[tryserver] Remove unused methods."  
							
							 
							
							... 
							
							
							
							This reverts commit 133ac1ab8d .
Reason for revert: Turns out these are used by internal recipes.
Original change's description:
> [tryserver] Remove unused methods.
> 
> R=agable@chromium.org, dnj@chromium.org, hinoka@chromium.org 
> 
> Bug:
> Change-Id: I82a11f31c8c1c4c4a2b461090e5aee637f8821c2
> Reviewed-on: https://chromium-review.googlesource.com/569019 
> Reviewed-by: Nodir Turakulov <nodir@chromium.org>
> Reviewed-by: Aaron Gable <agable@chromium.org>
> Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
TBR=iannucci@chromium.org,hinoka@chromium.org,agable@chromium.org,dnj@chromium.org,nodir@chromium.org
Change-Id: Ib1d4192520a36f649f1f9b31e2928027667311d4
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/570988 
Reviewed-by: Daniel Jacques <dnj@chromium.org>
Commit-Queue: Daniel Jacques <dnj@chromium.org> 
							
						 
						
							8 years ago  
						
					 
				
					
						
							
							
								 
								Robert Iannucci
							
						 
						
							 
							
							
							
								
							
								133ac1ab8d 
								
							
								 
							
						 
						
							
							
								
								[tryserver] Remove unused methods.  
							
							 
							
							... 
							
							
							
							R=agable@chromium.org, dnj@chromium.org, hinoka@chromium.org 
Bug:
Change-Id: I82a11f31c8c1c4c4a2b461090e5aee637f8821c2
Reviewed-on: https://chromium-review.googlesource.com/569019 
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org> 
							
						 
						
							8 years ago  
						
					 
				
					
						
							
							
								 
								Dan Jacques
							
						 
						
							 
							
							
							
								
							
								209a681f9c 
								
							
								 
							
						 
						
							
							
								
								[git] Remove hard-coded "depot_tools" Git.  
							
							 
							
							... 
							
							
							
							Several tools, including the "git" recipe module, hard-code a
checkout-relative "git.bat" path. Git is a feature that is provided by
the system, both to tooling and recipes:
1) For users, "depot_tools" must be on PATH, and during setup it will
   have installed "git.bat", ensuring that Git tooling is available in
   PATH.
2) For bots, the system is responsible for providing "git.bat" on PATH.
   This is typically done at "/b/depot_tools/git.bat", which is sync'd
   through the "update_scripts" step.
By formally treating Git as a system resource, we absolve Windows bots
and users from manually installing a depot_tools-local Git, bringing
them in line with other platforms.
BUG=chromium:590806
TEST=local
Change-Id: I93e89855cdd330a2ba7a8cfb8117a1789d1ab54e
Reviewed-on: https://chromium-review.googlesource.com/568694 
Commit-Queue: Daniel Jacques <dnj@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org> 
							
						 
						
							8 years ago  
						
					 
				
					
						
							
							
								 
								Paweł Hajdan, Jr
							
						 
						
							 
							
							
							
								
							
								24025d3e71 
								
							
								 
							
						 
						
							
							
								
								tryserver: add support for gerrit footers  
							
							 
							
							... 
							
							
							
							Bug: 644609
Change-Id: Ied7439d688b89f90b9705968927521b5060c5fb4
Reviewed-on: https://chromium-review.googlesource.com/565564 
Commit-Queue: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org> 
							
						 
						
							8 years ago  
						
					 
				
					
						
							
							
								 
								Robert Iannucci
							
						 
						
							 
							
							
							
								
							
								56038f517d 
								
							
								 
							
						 
						
							
							
								
								rename example.py -> examples/full.py for all modules  
							
							 
							
							... 
							
							
							
							R=dnj@chromium.org, nodir@chromium.org, phajdan.jr@chromium.org 
Bug:
Change-Id: I47d8f9311b6ee71583fb989888a1e4d28b5495df
Reviewed-on: https://chromium-review.googlesource.com/520731 
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org> 
							
						 
						
							9 years ago  
						
					 
				
					
						
							
							
								 
								Robert Iannucci
							
						 
						
							 
							
							
							
								
							
								5702f0baef 
								
							
								 
							
						 
						
							
							
								
								[recipes] convert all recipe_modules to use new context module.  
							
							 
							
							... 
							
							
							
							R=recipe-roller@chromium.org 
Bug:
Change-Id: I84c19965b779f1010c00cd8c857b7533eec7413f
Reviewed-on: https://chromium-review.googlesource.com/502929 
Reviewed-by: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org> 
							
						 
						
							9 years ago  
						
					 
				
					
						
							
							
								 
								Paweł Hajdan, Jr
							
						 
						
							 
							
							
							
								
							
								3978290b21 
								
							
								 
							
						 
						
							
							
								
								Enable strict coverage for all modules  
							
							 
							
							... 
							
							
							
							Bug: chromium:693058
Change-Id: I6e00f73141722dc47f8da16e21dadd1d4e26a14f
Reviewed-on: https://chromium-review.googlesource.com/475911 
Commit-Queue: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org> 
							
						 
						
							9 years ago  
						
					 
				
					
						
							
							
								 
								Paweł Hajdan, Jr
							
						 
						
							 
							
							
							
								
							
								f14709556d 
								
							
								 
							
						 
						
							
							
								
								Move recipes to their own subdirectory  
							
							 
							
							... 
							
							
							
							This will help remove confusion when depot_tools' recipes.py
ends up in PATH.
BUG=699120
Change-Id: Id4c21b0cc6bb022ea2c21145abe76bebb0a8d9c1
Reviewed-on: https://chromium-review.googlesource.com/458430 
Commit-Queue: Paweł Hajdan Jr. <phajdan.jr@chromium.org>
Reviewed-by: Robbie Iannucci <iannucci@chromium.org> 
							
						 
						
							9 years ago