Bug 10679

Summary: Drop unneeded expansion parameter to getVar and getVarFlag
Product: [Build System, Metadata & Runtime] BitBake Reporter: Richard Purdie <richard.purdie>
Component: bitbakeAssignee: Joshua Lock <joshuagloe>
Status: RESOLVED FIXED QA Contact: David Lopez Barriba <davidx.lopez.barriba>
Severity: enhancement    
Priority: Medium+ CC: jose.perez.carranza, poky.bs.watcher, poky.watcher
Version: 2.3   
Target Milestone: 2.3 M1   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: Yes (doc changes required)

Description Richard Purdie 2016-11-16 13:02:06 UTC
There are patches pending which make expansion the default for getVar/getVarFlag. Once merged we should remove the expansion parameter from most of the users of getVar and getVarFlag. There are sed expressions in the git history which can be used as a template to do this.
Comment 1 Robert Yang 2016-11-18 01:47:51 UTC
Josh will work on it.
Comment 2 Joshua Lock 2016-11-22 22:32:35 UTC
I have patches queued for this change using regex search/replace with the following regex:

getVar ?\(( ?[^,()]*), True\)
getVar($1)

getVarFlag ?\(( ?[^,()]*, ?[^,()]*), True\)
getVarFlag($1)