diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-02-05 16:02:03 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-02-06 13:13:01 +0000 |
commit | 6fecb1a3e02a91b4945f1d3e5faf72e9c129e833 (patch) | |
tree | 75552a348fa5a3bdc7425a0095560e7cdc9e782c /bitbake | |
parent | 1f192a7ade00b3e1dac8af603e9b48d254100fb6 (diff) | |
download | poky-6fecb1a3e02a91b4945f1d3e5faf72e9c129e833.tar.gz |
bitbake: cooker/utils: Drop custom 'interactive' variables list and store environment in BB_ORIGENV instead
Maintining hardcoded lists of user environmental variables is hacky,
replace these with the savedenv datastore. Allow access to that
through the BB_ORIGENV variable.
(Bitbake rev: 0a99563a4ea270594fd9a61da46f9387fb79dc66)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake')
-rw-r--r-- | bitbake/lib/bb/cooker.py | 1 | ||||
-rw-r--r-- | bitbake/lib/bb/utils.py | 27 |
2 files changed, 2 insertions, 26 deletions
diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py index 34fbfb0701..f70a04f460 100644 --- a/bitbake/lib/bb/cooker.py +++ b/bitbake/lib/bb/cooker.py | |||
@@ -185,6 +185,7 @@ class BBCooker: | |||
185 | 185 | ||
186 | filtered_keys = bb.utils.approved_variables() | 186 | filtered_keys = bb.utils.approved_variables() |
187 | bb.data.inheritFromOS(self.configuration.data, self.savedenv, filtered_keys) | 187 | bb.data.inheritFromOS(self.configuration.data, self.savedenv, filtered_keys) |
188 | self.configuration.data.setVar("BB_ORIGENV", self.savedenv) | ||
188 | 189 | ||
189 | def enableDataTracking(self): | 190 | def enableDataTracking(self): |
190 | self.configuration.data.enableTracking() | 191 | self.configuration.data.enableTracking() |
diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py index 83159a6465..484fb2dc76 100644 --- a/bitbake/lib/bb/utils.py +++ b/bitbake/lib/bb/utils.py | |||
@@ -458,27 +458,6 @@ def preserved_envvars_exported(): | |||
458 | 'USER', | 458 | 'USER', |
459 | ] | 459 | ] |
460 | 460 | ||
461 | def preserved_envvars_exported_interactive(): | ||
462 | """Variables which are taken from the environment and placed in and exported | ||
463 | from the metadata, for interactive tasks""" | ||
464 | return [ | ||
465 | 'COLORTERM', | ||
466 | 'DBUS_SESSION_BUS_ADDRESS', | ||
467 | 'DESKTOP_SESSION', | ||
468 | 'DESKTOP_STARTUP_ID', | ||
469 | 'DISPLAY', | ||
470 | 'GNOME_KEYRING_PID', | ||
471 | 'GNOME_KEYRING_SOCKET', | ||
472 | 'GPG_AGENT_INFO', | ||
473 | 'GTK_RC_FILES', | ||
474 | 'SESSION_MANAGER', | ||
475 | 'KRB5CCNAME', | ||
476 | 'SSH_AUTH_SOCK', | ||
477 | 'XAUTHORITY', | ||
478 | 'XDG_DATA_DIRS', | ||
479 | 'XDG_SESSION_COOKIE', | ||
480 | ] | ||
481 | |||
482 | def preserved_envvars(): | 461 | def preserved_envvars(): |
483 | """Variables which are taken from the environment and placed in the metadata""" | 462 | """Variables which are taken from the environment and placed in the metadata""" |
484 | v = [ | 463 | v = [ |
@@ -487,7 +466,7 @@ def preserved_envvars(): | |||
487 | 'BB_ENV_WHITELIST', | 466 | 'BB_ENV_WHITELIST', |
488 | 'BB_ENV_EXTRAWHITE', | 467 | 'BB_ENV_EXTRAWHITE', |
489 | ] | 468 | ] |
490 | return v + preserved_envvars_exported() + preserved_envvars_exported_interactive() | 469 | return v + preserved_envvars_exported() |
491 | 470 | ||
492 | def filter_environment(good_vars): | 471 | def filter_environment(good_vars): |
493 | """ | 472 | """ |
@@ -509,10 +488,6 @@ def filter_environment(good_vars): | |||
509 | 488 | ||
510 | return removed_vars | 489 | return removed_vars |
511 | 490 | ||
512 | def create_interactive_env(d): | ||
513 | for k in preserved_envvars_exported_interactive(): | ||
514 | os.setenv(k, d.getVar(k, True)) | ||
515 | |||
516 | def approved_variables(): | 491 | def approved_variables(): |
517 | """ | 492 | """ |
518 | Determine and return the list of whitelisted variables which are approved | 493 | Determine and return the list of whitelisted variables which are approved |