Terry J. Reedy added the comment:

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.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue30869>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com


Reply via email to