389-ds-base

#50986 Issue 50337 - Replace exec() with setattr()
Closed by spichugi. Opened by spichugi.
spichugi/389-ds-base i50337  into  master

Download 50986.patch

Description: _constants.py uses exec() a lot to define module
global variables. That's rather slow and not very elegant.

Get the current module object with sys.modules[name] and
then use setattr() instead.

https://pagure.io/389-ds-base/issue/50337

Reviewed by: ?

Ack

rebased onto 7a6bbc1df29892863aa9367a242264c302cacf5f

Pull-Request has been merged by spichugi

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4039

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.