MNT: Change config home path logic#10705
Conversation
|
As I wrote on #10661, I wonder if we shouldn't replace this logic entirely to just use
It appears Also worth noting that as of Python 3.8:
That change was motivated by this issue. |
9e295e2 to
dee80de
Compare
dee80de to
059a434
Compare
|
@embray , I tried to implement your suggestion in the second commit but there is a chance I completely misunderstood, so please review. Does the doc at https://docs.astropy.org/en/latest/config/index.html need updating too? |
|
@pllim Sorry for the late reply. The last week has been kind of crazy for me; I will be around more after this week. Looking at the documentation, it looks like it conflicts with my previous recommendation that Windows users not use I would argue we change that recommendation, but actually in retrospect there is not currently any other way to specify a different astropy config path via the environment except for setting Perhaps we should take a cue here from matplotlib's |
|
I'm otherwise 👍 on this as-is. |
|
@embray , thanks for opening the meta issue, it was really illuminating. If you are okay with this, would you mind approving and merging? If you need write access, please let Kelle know. |
|
Cool, looks like I have access to merge. Just had to approve my review. |
|
Thanks! 😄 |
Description
This pull request is to address the problems Windows users encounter because
configlooks inHOMESHARE(either inaccessible or simply missing) too early. Now, it only looks forHOMESHAREif all the other options are exhausted.I thought about removing it altogether but maybe some people do use
HOMESHAREto store config; who knows. 🤷If this is too controversial, we can move the milestone down to 5.0.
Fixes #10661
Fixes #8731