Simplifying the plugin interface

Author: Pierre-Luc Tessier Gagné

can/interface.py

@@ -4,7 +4,7 @@
 import importlib
 
 from can.broadcastmanager import CyclicSendTaskABC, MultiRateCyclicSendTaskABC
-from can.plugin import get_pluginmanager
+from pkg_resources import iter_entry_points
 from can.util import load_config
 
 # interface_name => (module, classname)
@@ -25,6 +25,12 @@
 }
 
 
+BACKENDS.update({
+    interface.name: (interface.module_name, interface.attrs[0])
+    for interface in iter_entry_points('python_can.interface')
+})
+
+
 class Bus(object):
     """
     Instantiates a CAN Bus of the given `bustype`, falls back to reading a
@@ -59,12 +65,7 @@ def __new__(cls, other, channel=None, *args, **kwargs):
 
         # Import the correct Bus backend
         try:
-            interfaces_hook = get_pluginmanager().hook
-            plugin = interfaces_hook.pythoncan_interface(interface=interface)
-            if plugin:
-                (module_name, class_name) = plugin[0]
-            else:
-                (module_name, class_name) = BACKENDS[interface]
+            (module_name, class_name) = BACKENDS[interface]
         except KeyError:
             raise NotImplementedError("CAN interface '{}' not supported".format(interface))
 

can/interfaces/__init__.py

@@ -2,8 +2,14 @@
 """
 Interfaces contain low level implementations that interact with CAN hardware.
 """
+from pkg_resources import iter_entry_points
 
 VALID_INTERFACES = set(['kvaser', 'serial', 'pcan', 'socketcan_native',
                         'socketcan_ctypes', 'socketcan', 'usb2can', 'ixxat',
                         'nican', 'iscan', 'vector', 'virtual', 'neovi',
                         'slcan'])
+
+
+VALID_INTERFACES.update(set([
+    interface.name for interface in iter_entry_points('python_can.interface')
+]))

can/interfaces/hookspecs.py

@@ -29,7 +29,6 @@
     # Tests can be run using `python setup.py test`
     test_suite="nose.collector",
     tests_require=['mock', 'nose', 'pyserial'],
-    install_requires=['pluggy'],
     extras_require={
         'serial': ['pyserial'],
         'neovi': ['python-ics'],