On 07/04/2011 10:06 AM, Holger Levsen wrote:
Hi Scott,

thanks for your bug report with patch. If you want to be even more awesome
than you already are, you could include debian/changelog entries in your
patches too :-)

Um ... OK. I will have to learn how to do that. I will add to my "Learn About ..." list :-)
On Mittwoch, 29. Juni 2011, Scott Schaefer wrote:
Attached patch causes piuparts to add appropriate line to apt.conf in
chroot to cause apt to use current HTTP_PROXY variable from OS
environment.
it also seems to contain unwanted noise about force-confdef :/


I didn't see that in my (limited) testing. I will re-test (after mod described below) and look specifically for anything about confdef. In meantime, do you recall precisely what it reports ?
It PROBABLY should also read current apt config and duplicate any
Acquire::http::proxy value from there.  However, there are at least three
issues with that:
I think the ability to specify the proxy like this is well sufficient. If a
user wants to configure apt further, she can always do this via a custom
script :-)


I already found 'apt-config' (which addresses all three of my "issues"). I have it on my list to execute it and pull any Acquire::Proxy value from there if HTTP_PROXY is not in os.environ. So I'll send revised patch in next 3-4 days.
cheers,
        Holger




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to