Is there some reason you couldn't just ensure all the proxy variables are in the env whitelist, so they end up in the metadata? This patch seems extremely hackish to me.
On Wed, Jul 14, 2010 at 2:35 PM, Bernhard Reutner-Fischer < [email protected]> wrote: > iff they are not in the config > > Signed-off-by: Bernhard Reutner-Fischer <[email protected]> > --- > lib/bb/fetch/__init__.py | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > diff --git a/lib/bb/fetch/__init__.py b/lib/bb/fetch/__init__.py > index 276ff46..c4d860e 100644 > --- a/lib/bb/fetch/__init__.py > +++ b/lib/bb/fetch/__init__.py > @@ -397,12 +397,16 @@ def runfetchcmd(cmd, d, quiet = False): > # rather than host provided > # Also include some other variables. > # FIXME: Should really include all export varaiables? > - exportvars = ['PATH', 'GIT_PROXY_COMMAND', 'GIT_PROXY_HOST', > 'GIT_PROXY_PORT', 'GIT_CONFIG', 'http_proxy', 'ftp_proxy', 'SSH_AUTH_SOCK', > 'SSH_AGENT_PID', 'HOME'] > + exportvars = ['PATH', 'GIT_PROXY_COMMAND', 'GIT_PROXY_HOST', > 'GIT_PROXY_PORT', 'GIT_CONFIG', 'http_proxy', 'ftp_proxy', 'no_proxy', > 'SSH_AUTH_SOCK', 'SSH_AGENT_PID', 'HOME'] > > for var in exportvars: > val = data.getVar(var, d, True) > if val: > cmd = 'export ' + var + '=\"%s\"; %s' % (val, cmd) > + else if var.endswith('_proxy'): > + val = os.getenv(var) > + if val: > + cmd = 'export ' + var + '=\"%s\"; %s' % (val, cmd) > > bb.msg.debug(1, bb.msg.domain.Fetcher, "Running %s" % cmd) > > -- > 1.7.1 > > _______________________________________________ > Bitbake-dev mailing list > [email protected] > https://lists.berlios.de/mailman/listinfo/bitbake-dev > -- Christopher Larson clarson at kergoth dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics
_______________________________________________ Bitbake-dev mailing list [email protected] https://lists.berlios.de/mailman/listinfo/bitbake-dev
