Allow lazy evalution of some of the Extension's attributes

Related: #12

setup.py

@@ -11,45 +11,88 @@
 
 version = '0.12'
 
-def pkgconfig(pkg):
-    def _str2list(pkgstr, onlystr):
-        res = []
-        for l in pkgstr.split(" "):
-            if l.find(onlystr) == 0:
-                res.append(l.replace(onlystr, "", 1))
-        return res
-
-    (res, cflags) = commands.getstatusoutput('pkg-config --cflags-only-other %s' % pkg)
-    if res != 0:
-        print('Failed to query pkg-config --cflags-only-other %s' % pkg)
-        sys.exit(1)
-
-    (res, includes) = commands.getstatusoutput('pkg-config --cflags-only-I %s' % pkg)
-    if res != 0:
-        print('Failed to query pkg-config --cflags-only-I %s' % pkg)
-        sys.exit(1)
-
-    (res, libs) = commands.getstatusoutput('pkg-config --libs-only-l %s' % pkg)
-    if res != 0:
-        print('Failed to query pkg-config --libs-only-l %s' % pkg)
-        sys.exit(1)
-
-    (res, libdirs) = commands.getstatusoutput('pkg-config --libs-only-L %s' % pkg)
-    if res != 0:
-        print('Failed to query pkg-config --libs-only-L %s' % pkg)
-        sys.exit(1)
-
-
-    # Clean up the results and return what we've extracted from pkg-config
-    return {'cflags': cflags,
-            'include': _str2list(includes, '-I'),
-            'libs': _str2list(libs, '-l'),
-            'libdirs': _str2list(libdirs, '-L')
-            }
-
-
-libnl = pkgconfig('libnl-3.0')
-libnl['libs'].append('nl-route-3')
+
+class PkgConfigExtension(Extension):
+    '''Extension with lazy properties taken from pkg-config'''
+    def __init__(self, *args, **kwargs):
+        pkg = kwargs['pkg']
+        del kwargs['pkg']
+        Extension.__init__(self, *args, **kwargs)
+        try:
+            # on Python 2 we need to delete those now
+            del self.include_dirs
+            del self.library_dirs
+            del self.libraries
+        except AttributeError:
+            # on Python 3, that's not needed or possible
+            pass
+        self._pkg = pkg
+        self._pkgconfig_result = None
+
+    @property
+    def pkgconfig(self):
+        if self._pkgconfig_result is None:
+            self._pkgconfig_result = self._pkgconfig(self._pkg)
+        return self._pkgconfig_result
+
+    def _pkgconfig(self, pkg):
+        def _str2list(pkgstr, onlystr):
+            res = []
+            for l in pkgstr.split(" "):
+                if l.find(onlystr) == 0:
+                    res.append(l.replace(onlystr, "", 1))
+            return res
+
+        (res, cflags) = commands.getstatusoutput('pkg-config --cflags-only-other %s' % pkg)
+        if res != 0:
+            print('Failed to query pkg-config --cflags-only-other %s' % pkg)
+            sys.exit(1)
+
+        (res, includes) = commands.getstatusoutput('pkg-config --cflags-only-I %s' % pkg)
+        if res != 0:
+            print('Failed to query pkg-config --cflags-only-I %s' % pkg)
+            sys.exit(1)
+
+        (res, libs) = commands.getstatusoutput('pkg-config --libs-only-l %s' % pkg)
+        if res != 0:
+            print('Failed to query pkg-config --libs-only-l %s' % pkg)
+            sys.exit(1)
+
+        (res, libdirs) = commands.getstatusoutput('pkg-config --libs-only-L %s' % pkg)
+        if res != 0:
+            print('Failed to query pkg-config --libs-only-L %s' % pkg)
+            sys.exit(1)
+
+        # Clean up the results and return what we've extracted from pkg-config
+        return {'cflags': cflags,
+                'include': _str2list(includes, '-I'),
+                'libs': _str2list(libs, '-l'),
+                'libdirs': _str2list(libdirs, '-L')
+                }
+
+    @property
+    def include_dirs(self):
+        return self.pkgconfig['include']
+
+    @property
+    def library_dirs(self):
+        return self.pkgconfig['libdirs']
+
+    @property
+    def libraries(self):
+        return self.pkgconfig['libs'] + ['nl-route-3']
+
+    @include_dirs.setter
+    def include_dirs(self, value):
+        pass
+
+    @library_dirs.setter
+    def library_dirs(self, value):
+        pass
+
+    @libraries.setter
+    def libraries(self, value):
+        pass
 
 # don't reformat this line, Makefile parses it
 setup(name='ethtool',
@@ -59,7 +102,7 @@ def _str2list(pkgstr, onlystr):
       author_email='davids@redhat.com',
       url='https://github.com/fedora-python/python-ethtool',
       ext_modules=[
-        Extension(
+        PkgConfigExtension(
             'ethtool',
             sources = [
                 'python-ethtool/ethtool.c',
@@ -68,10 +111,8 @@ def _str2list(pkgstr, onlystr):
                 'python-ethtool/netlink.c',
                 'python-ethtool/netlink-address.c'],
             extra_compile_args=['-fno-strict-aliasing'],
-            include_dirs = libnl['include'],
-            library_dirs = libnl['libdirs'],
-            libraries = libnl['libs'],
-            define_macros = [('VERSION', '"%s"' % version)]
-            )
-        ]
+            define_macros = [('VERSION', '"%s"' % version)],
+            pkg = 'libnl-3.0',
+        )
+      ]
 )