On Wed, Jan 24, 2007 at 07:50:21PM +0100, Frans Pop wrote: > Thanks for your suggested patch. There were some minor problems with it, > so we have modified it slightly. I've committed the following patch in > our SVN repository: [snip] > Main changes are: > - the way protocol is determined
OK. I wasn't sure what shell features were OK to use, and I saw sed(1) used elsewhere. > - that the variable exported now depends on the protocol (http/ftp_proxy) Hehe, indeed. :-) > Could you test this patch before we upload it? > You can easily test it by just editing (using nano) the 60local file in > /usr/lib/apt-setup/generators before apt-setup is run. When I tried this, apt-setup writes this to syslog: apt-setup: warning: /usr/lib/apt-setup/generators/60local returned error code 1; discarding output 'sh -n' returns cleanly on my modified 60local, and I triple-checked the modifications I made, since I had to type them by hand. Perhaps something in this patch isn't playing nicely with 'set -e'? FWIW, here's my proxy preseed configuration: d-i mirror/http/proxy string http://172.16.5.162:8080/
pgpfeGExAme1K.pgp
Description: PGP signature
