[Python-Dev] Why does httplib import from test_support?

Barry Warsaw barry at python.org
Tue Aug 19 21:05:26 CEST 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Aug 19, 2008, at 2:31 PM, Brett Cannon wrote:

> On Tue, Aug 19, 2008 at 8:46 AM, Nick Coghlan <ncoghlan at gmail.com>  
> wrote:
>> Brett Cannon wrote:
>>> On Mon, Aug 18, 2008 at 11:55 AM, Fred Drake <fdrake at acm.org> wrote:
>>>> On Aug 18, 2008, at 2:43 PM, Antoine Pitrou wrote:
>>>>> If it's about silencing warnings, then how about putting it in the
>>>>> warnings
>>>>> module?
>>>>
>>>> That sounds good to me, and would be very reasonable.  This would  
>>>> make a
>>>> nice context manager.
>>>>
>>>
>>> That works for me as well; just move catch_warning() to 'warnings'  
>>> and
>>> leave test.test_support as-is. If we did that, though, I would  
>>> want to
>>> make 'record' False by default so that it had more reasonable  
>>> defaults
>>> for non-testing purposes.
>>
>> Moving this single context manager over to warnings definitely makes
>> more sense to me than moving the whole test_support module.
>>
>> It would also be easy enough to add a "catch_warning" factory  
>> function
>> in test_support that switched the default back to record=True so we
>> didn't have to change all of the invocations in the test suite.
>>
>
> Right; just have a simple wrapper in test.test_support that had a
> different default.
>
> I will get to this when I can, although I am sure Barry won't be too
> upset if this misses b3.

Nope.
- -Barry

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Darwin)

iQCVAwUBSKsZd3EjvBPtnXfVAQLGRAQAhVwJLkwi/+ZlW7x53sKyyMEf+pJj7UTK
l0MjS/JFXlDmW5uBg9VnKbSrdMnbQPwHIhA1Xl20Pz3qV4mPTdrD6+pVuwyBQJj6
rakN6wtjvYMq3S3cDIZIabbaqx05XT5+mqWA+2fIc7/mzHboORtJmfAlE/Tjr4/P
TR7TgSqryzQ=
=TCYz
-----END PGP SIGNATURE-----


More information about the Python-Dev mailing list