jenkins-bot has submitted this change. (
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/652550 )
Change subject: [bugfix] Re-enable reading user-config.py from site package
......................................................................
[bugfix] Re-enable reading user-config.py from site package
reading user-config.py was accidently disabled if pwikibot was
installed as a site package.
- always read user-config.py if the file exists
Bug: T270941
Change-Id: If3dbea73c730d0f5871afc482be242c317e8d9cb
---
M pywikibot/config2.py
1 file changed, 15 insertions(+), 20 deletions(-)
Approvals:
JJMC89: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/config2.py b/pywikibot/config2.py
index 0f8395f..4c45f89 100644
--- a/pywikibot/config2.py
+++ b/pywikibot/config2.py
@@ -989,26 +989,21 @@
# we can detect modified config items easily.
_exec_globals = copy.deepcopy(_public_globals)
-# Get the user files
-if __no_user_config:
- if __no_user_config != '2':
- warning('Skipping loading of user-config.py.')
-else:
- _filename = os.path.join(base_dir, 'user-config.py')
- if os.path.exists(_filename):
- _filestatus = os.stat(_filename)
- _filemode = _filestatus[0]
- _fileuid = _filestatus[4]
- if OSWIN32 or _fileuid in [os.getuid(), 0]:
- if OSWIN32 or _filemode & 0o02 == 0:
- with open(_filename, 'rb') as f:
- exec(compile(f.read(), _filename, 'exec'), _exec_globals)
- else:
- warning("Skipped '%(fn)s': writeable by others."
- % {'fn': _filename})
- else:
- warning("Skipped '%(fn)s': owned by someone else."
- % {'fn': _filename})
+# Always try to get the user files
+_filename = os.path.join(base_dir, 'user-config.py')
+if os.path.exists(_filename):
+ _filestatus = os.stat(_filename)
+ _filemode = _filestatus[0]
+ _fileuid = _filestatus[4]
+ if not OSWIN32 and _fileuid not in [os.getuid(), 0]:
+ warning('Skipped {fn!r}: owned by someone else.'.format(fn=_filename))
+ elif OSWIN32 or _filemode & 0o02 == 0:
+ with open(_filename, 'rb') as f:
+ exec(compile(f.read(), _filename, 'exec'), _exec_globals)
+ else:
+ warning('Skipped {fn!r}: writeable by others.'.format(fn=_filename))
+elif __no_user_config and __no_user_config != '2':
+ warning('user-config.py cannot be loaded.')
class _DifferentTypeError(UserWarning, TypeError):
--
To view, visit
https://gerrit.wikimedia.org/r/c/pywikibot/core/+/652550
To unsubscribe, or for help writing mail filters, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: If3dbea73c730d0f5871afc482be242c317e8d9cb
Gerrit-Change-Number: 652550
Gerrit-PatchSet: 4
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: JJMC89 <JJMC89.Wikimedia(a)gmail.com>
Gerrit-Reviewer: Mpaa <mpaa.wiki(a)gmail.com>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged