#11982 closed enhancement (fixed)
Remove dependency on ConfigObj — at Version 3
Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
---|---|---|---|
Priority: | normal | Milestone: | 1.1.5 |
Component: | general | Version: | |
Severity: | normal | Keywords: | authzpolicy configobj config |
Cc: | Branch: | ||
Release Notes: |
Removed dependency on |
||
API Changes: |
Added |
||
Internal Changes: |
Description
It seems like the dependency on ConfigObj
in tracopt.perm.authz_policy
may be unnecessary. After extracting a class from the trac.config
module that provides a thin wrapper over ConfigParser
, we can use that class instead of ConfigObj
.
Change History (3)
comment:1 by , 9 years ago
Milestone: | next-dev-1.1.x → 1.1.5 |
---|---|
Owner: | set to |
Status: | new → assigned |
comment:2 by , 9 years ago
Revised changes in log:rjollos.git:t11982-remove-configobj.1. Tested so far on Mac OSX with Python 2.6 and Python 2.7.
comment:3 by , 9 years ago
API Changes: | modified (diff) |
---|---|
Release Notes: | modified (diff) |
Resolution: | → fixed |
Status: | assigned → closed |
Tested also with Python 2.6 and 2.7 on Windows 7. Committed to trunk in [13993:13995]. More work on config
module is forthcoming in #6551, and yet-to-be-created tickets.
I'll update the 1.1 documentation soon to remove ConfigObj
from the optional dependencies.
Proposed changes in log:rjollos.git:t11982-remove-configobj:
ConfigParser
doesn't throw an exception due to duplicate sections or options.ComponentManager
orOption
classes. The class with knowledge ofComponentManager
andOption
would utilize this extracted class to read and write the environment configuration. The extracted class could be utilized by theauthz_policy
module so that we have the capability to write authz files as well as read them.