[Merge] lp:~smoser/ubuntu-dev-tools/better-proxy-support into lp:ubuntu-dev-tools

[Scott Moser]

Scott Moser has proposed merging lp:~smoser/ubuntu-dev-tools/better-proxy-support into lp:ubuntu-dev-tools.

Requested reviews:
  Ubuntu Development Team (ubuntu-dev)
Related bugs:
  Bug #881654 in ubuntu-dev-tools (Ubuntu): "mk-sbuild needs better http proxy support"
  https://bugs.launchpad.net/ubuntu/+source/ubuntu-dev-tools/+bug/881654

For more details, see:
https://code.launchpad.net/~smoser/ubuntu-dev-tools/better-proxy-support/+merge/80386

mk-sbuild: better support apt http proxy (LP: #881654)

This improves the support for apt http proxy.
It determines the value for 'http_proxy' by, in increasing importance:
  * setting in apt.conf (per apt-config)
  * environment variable 'http_proxy'
  * command line flag '--debootstrap-proxy'

Then, if that is set, it makes use of it in
  * initial call to debootstrap (environment variable passed through sudo)
  * 'finish.sh' and subsequent invocation by writing config to
    /etc/apt/config.d/

Thus, the value here is 'sticky' after the debootstrap invocation.

Some may not want that to be sticky, so I could see a need for
something like '--non-sticky-http-proxy' flag.  Thoughts on that?

-- 
https://code.launchpad.net/~smoser/ubuntu-dev-tools/better-proxy-support/+merge/80386
Your team Ubuntu Development Team is requested to review the proposed merge of lp:~smoser/ubuntu-dev-tools/better-proxy-support into lp:ubuntu-dev-tools.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 4180 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20111025/c0412fd3/attachment.diff>