[yum-cvs] yum/yum plugins.py,1.8,1.9

[Menno Smits]

Update of /home/groups/yum/cvs/yum/yum
In directory login:/tmp/cvs-serv9596

Modified Files:
	plugins.py 
Log Message:
Fixed subtle problem that meant config slot check was not working in Python 2.2.
Updated TODO items.


Index: plugins.py
===================================================================
RCS file: /home/groups/yum/cvs/yum/yum/plugins.py,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- plugins.py	15 Apr 2005 04:02:20 -0000	1.8
+++ plugins.py	21 Apr 2005 21:58:40 -0000	1.9
@@ -22,21 +22,10 @@
 import config 
 import Errors
 
-# TODO: better documentation of how the whole thing works (esp. addition of
-# config options)
-#       - use epydoc for this?
-#       - document from user perspective in yum man page
-#       - PLUGINS.txt for developers
+# TODO: cmdline option to disable plugins "--noplugins" (for support problems)
+#   - document
 
-# TODO: test the API by implementing some crack from bugzilla
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=181 
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=270
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=310
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=431
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=88 (?)
-#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=396 (DONE)
-
-# TODO: expose progress bar 
+# TODO: expose progress bar interface
 
 # TODO: allow plugins to define new repository types
 
@@ -51,10 +40,6 @@
 
 # TODO: multiversion plugin support
 
-# TODO: cmdline options to disable/enable plugins (all or specific)
-
-# TODO: cmdline option to override/specify additional plugin directories
-
 # TODO: config vars marked as PLUG_OPT_WHERE_ALL should inherit defaults from
 #   the [main] setting if the user doesn't specify them
 
@@ -70,6 +55,18 @@
 # TODO: detect conflicts between builtin yum options and registered plugin
 # options (will require refactoring of config.py)
 
+# TODO: Developer docs:
+#       - PLUGINS.(txt|html?)
+#       - use epydoc as API begins to stablise
+
+# TODO: test the API by implementing some crack from bugzilla
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=181 
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=270
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=310
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=431
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=88 (?)
+#         - http://devel.linux.duke.edu/bugzilla/show_bug.cgi?id=396 (DONE)
+
 
 # The API_VERSION constant defines the current plugin API version. It is used
 # to decided whether or not plugins can be loaded. It is compared against the
@@ -123,7 +120,7 @@
             return
        
         # Determine handler class to use
-        if slotname in ('config'):
+        if slotname == 'config':
             conduitcls = ConfigPluginConduit
         elif slotname == 'init':
             conduitcls = InitPluginConduit