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 terry.reedy
Recipients louielu, terry.reedy, vstinner
Date 2017-07-07.04:32:38
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1499401958.64.0.510842337751.issue30869@psf.upfronthosting.co.za>
In-reply-to
Content
I would have liked the mistake in #30780 to have been noticed, but I don't think the patch would have caught it.  What I failed to restore was an in memory date structure that is part of the config module.   The patch seems to be concerned only with the filesystem, which is something different.

On IDLE startup, config tries 1. to find a home directory, 2. within that a .idlerc directory, and 3. within that user config files.  If 1. fails, it used getcwd() as home.  Either way, if 2. fails, config tries to make .idlerc.  Failure here is currently fatal, though I would like to eventually change that.  So I presume creation succeeds at least once on each buildbot.  Creating .idlerc is intentional behavior, not a mistake, and in the absence of buildbot owners complaining, I don't think it should be made a test failure.  Creating non-empty user config files during a test (on user machines) would be a mistake, and I have made sure that tests do not touch them.  Checking for user config changes within .idlerc would be okay,

Victor, it is up to you as regrtest expert to review both the implied policy change and the effect of the code change.
History
Date User Action Args
2017-07-07 04:32:38terry.reedysetrecipients: + terry.reedy, vstinner, louielu
2017-07-07 04:32:38terry.reedysetmessageid: <1499401958.64.0.510842337751.issue30869@psf.upfronthosting.co.za>
2017-07-07 04:32:38terry.reedylinkissue30869 messages
2017-07-07 04:32:38terry.reedycreate