diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-11-10 09:52:44 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-11-24 15:50:26 +0000 |
commit | fecb07755ca6aec978cac19d5cb17fdc888b0057 (patch) | |
tree | 27f41de22af46881008bfbd70e964969ccfcda4e /meta/classes/terminal.bbclass | |
parent | 9167f200ebc4a280c2d60dc9649f0d3be953cb26 (diff) | |
download | poky-fecb07755ca6aec978cac19d5cb17fdc888b0057.tar.gz |
classes: Ensure pass setVar/setVarFlag strings, not integers
This doesn't cause any issues right now but it make sense to standardise
on consistently using strings in the data store.
(From OE-Core rev: 99203fbe5ad470ef65cff93cec9d7f332883b5ee)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/terminal.bbclass')
-rw-r--r-- | meta/classes/terminal.bbclass | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/meta/classes/terminal.bbclass b/meta/classes/terminal.bbclass index e577c6d594..fd87cfe3d8 100644 --- a/meta/classes/terminal.bbclass +++ b/meta/classes/terminal.bbclass | |||
@@ -15,7 +15,7 @@ def emit_terminal_func(command, envdata, d): | |||
15 | cmd_func = 'do_terminal' | 15 | cmd_func = 'do_terminal' |
16 | 16 | ||
17 | envdata.setVar(cmd_func, 'exec ' + command) | 17 | envdata.setVar(cmd_func, 'exec ' + command) |
18 | envdata.setVarFlag(cmd_func, 'func', 1) | 18 | envdata.setVarFlag(cmd_func, 'func', '1') |
19 | 19 | ||
20 | runfmt = d.getVar('BB_RUNFMT', True) or "run.{func}.{pid}" | 20 | runfmt = d.getVar('BB_RUNFMT', True) or "run.{func}.{pid}" |
21 | runfile = runfmt.format(func=cmd_func, task=cmd_func, taskfunc=cmd_func, pid=os.getpid()) | 21 | runfile = runfmt.format(func=cmd_func, task=cmd_func, taskfunc=cmd_func, pid=os.getpid()) |
@@ -39,14 +39,14 @@ def oe_terminal(command, title, d): | |||
39 | 39 | ||
40 | for v in os.environ: | 40 | for v in os.environ: |
41 | envdata.setVar(v, os.environ[v]) | 41 | envdata.setVar(v, os.environ[v]) |
42 | envdata.setVarFlag(v, 'export', 1) | 42 | envdata.setVarFlag(v, 'export', '1') |
43 | 43 | ||
44 | for export in oe.data.typed_value('OE_TERMINAL_EXPORTS', d): | 44 | for export in oe.data.typed_value('OE_TERMINAL_EXPORTS', d): |
45 | value = d.getVar(export, True) | 45 | value = d.getVar(export, True) |
46 | if value is not None: | 46 | if value is not None: |
47 | os.environ[export] = str(value) | 47 | os.environ[export] = str(value) |
48 | envdata.setVar(export, str(value)) | 48 | envdata.setVar(export, str(value)) |
49 | envdata.setVarFlag(export, 'export', 1) | 49 | envdata.setVarFlag(export, 'export', '1') |
50 | if export == "PSEUDO_DISABLED": | 50 | if export == "PSEUDO_DISABLED": |
51 | if "PSEUDO_UNLOAD" in os.environ: | 51 | if "PSEUDO_UNLOAD" in os.environ: |
52 | del os.environ["PSEUDO_UNLOAD"] | 52 | del os.environ["PSEUDO_UNLOAD"] |
@@ -62,7 +62,7 @@ def oe_terminal(command, title, d): | |||
62 | if value is not None: | 62 | if value is not None: |
63 | os.environ[key] = str(value) | 63 | os.environ[key] = str(value) |
64 | envdata.setVar(key, str(value)) | 64 | envdata.setVar(key, str(value)) |
65 | envdata.setVarFlag(key, 'export', 1) | 65 | envdata.setVarFlag(key, 'export', '1') |
66 | 66 | ||
67 | # A complex PS1 might need more escaping of chars. | 67 | # A complex PS1 might need more escaping of chars. |
68 | # Lets not export PS1 instead. | 68 | # Lets not export PS1 instead. |