This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eli.bendersky
Recipients Arfrever, brett.cannon, eli.bendersky, eric.snow, ezio.melotti, pitrou
Date 2013-02-06.13:19:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAF-Rda_kVDdGXN-uEqbQ2RXCE6sJuXuMLWFO6Jcb_gXvwXSGFQ@mail.gmail.com>
In-reply-to <1360153032.0.0.428892259279.issue17037@psf.upfronthosting.co.za>
Content
On Wed, Feb 6, 2013 at 4:17 AM, Ezio Melotti <report@bugs.python.org> wrote:

>
> Ezio Melotti added the comment:
>
> I'm still a bit skeptic about this.
>
> To summarize my position, I am:
> +0 about adding something like this to test.support and use it for new
> tests;
> -1 about mass-rewriting the existing tests to use this (in particular for
> test_json);
> +1 about simplifying the API of import_fresh_module so that it returns
> both the modules and doesn't require the fresh/blocked args;
>

I share Ezio's sentiment regarding the new proposal(s). As for
import_fresh_module, if it's only ever used to do what we describe here,
then I propose to rename it to something more descriptive and indeed make
it just return both modules. If it's being used for other stuff, we can add
another function and rewrite all the tests to use that.

py_module, c_module = import_python_module_and_accelerator('module',
'_module')
History
Date User Action Args
2013-02-06 13:19:18eli.benderskysetrecipients: + eli.bendersky, brett.cannon, pitrou, ezio.melotti, Arfrever, eric.snow
2013-02-06 13:19:18eli.benderskylinkissue17037 messages
2013-02-06 13:19:18eli.benderskycreate