neeverett added the comment:

I think the original author adds the relative path "%APPDATA%..." with the 
intention to keep it effective even when the set path of APPDATA changes. 
However that compulsively changes the type of PATH var to REG_EXPAND_SZ.

It is reasonable that the type of PATH var should not be changed. So I think 
when the type is REG_SZ, not adding the %APPDATA% path and instead adding the 
absolute path of it should be a good idea.

I just modified my PR and fixed it this way, along with other problems that 
Eryk Sun mentioned.

----------

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

Reply via email to