Bug 10778

Summary:	Adopt skipStep() throughout the codebase
Product:	[Infrastructure] AutoBuilder	Reporter:	Joshua Lock <joshuagloe>
Component:	autobuilder	Assignee:	Joshua Lock <joshuagloe>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	Medium	CC:	infras.ab.watcher, Infras.watcher, pidge
Version:	2.2
Target Milestone:	2.3 M2
Hardware:	x86
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)

Description Joshua Lock 2016-12-08 15:25:36 UTC

We introduced a method on the BitbakeShellCommand base class for unifying the way in which a buildstep is skipped, we should adopt this pattern throughout the codebase but shouldn't do so until after M1.

Comment 1 Joshua Lock 2016-12-20 15:13:48 UTC

There are relatively few spots left where adopting using this method makes sense, but a patch has been written to do so.

Comment 2 Joshua Lock 2016-12-21 17:04:53 UTC

skipStep() used everywhere it makes sense, most recently:

http://git.yoctoproject.org/clean/cgit.cgi/yocto-autobuilder/commit/?id=f70c7bf903ab62a7b2db540b2d33881166c383bd