allow skipping extras_requires from (Build)Requires

sometimes one wants to generate a "minimal" RPM, with no extra
dependencies, which this should help with.

Author: evgeni

pyp2rpm/bin.py

@@ -127,6 +127,10 @@ def get_scl_help_record(self, ctx):
               help='Enable / disable metadata extraction from virtualenv '
               '(default: enabled).',
               default=True)
+@click.option('--include-extras / --no-include-extras',
+              help='Enable / disable including extras_require as dependencies '
+              '(default: enabled).',
+              default=True)
 @click.option('--autonc/ --no-autonc',
               help='Enable / disable using automatic provides with '
               'a standardized name in dependencies declaration ('
@@ -167,7 +171,7 @@ def get_scl_help_record(self, ctx):
               default=None,
               metavar='FILE_NAME')
 @click.argument('package', nargs=1)
-def main(package, v, d, s, r, proxy, srpm, p, b, o, t, venv, autonc, sclize,
+def main(package, v, d, s, r, proxy, srpm, p, b, o, t, venv, include_extras, autonc, sclize,
          **scl_kwargs):
     """Convert PyPI package to RPM specfile or SRPM.
 
@@ -203,7 +207,8 @@ def main(package, v, d, s, r, proxy, srpm, p, b, o, t, venv, autonc, sclize,
                           rpm_name=r,
                           proxy=proxy,
                           venv=venv,
-                          autonc=autonc)
+                          autonc=autonc,
+                          include_extras=include_extras)
 
     logger.debug(
         'Convertor: {0} created. Trying to convert.'.format(convertor))

pyp2rpm/command/extract_dist.py

@@ -25,12 +25,14 @@ def __init__(self, *args, **kwargs):
             self.metadata[attr] = to_list(getattr(self.distribution, attr, []))
 
         try:
+            self.metadata['setup_requires_extras'] = []
+            self.metadata['install_requires_extras'] = []
             for k, v in getattr(
                     self.distribution, 'extras_require', {}).items():
                 if k in ['test, docs', 'doc', 'dev']:
-                    attr = 'setup_requires'
+                    attr = 'setup_requires_extras'
                 else:
-                    attr = 'install_requires'
+                    attr = 'install_requires_extras'
                 self.metadata[attr] += to_list(v)
         except (AttributeError, ValueError):
             # extras require are skipped in case of wrong data format

pyp2rpm/convertor.py

@@ -45,7 +45,8 @@ def __init__(self, package=None, version=None,
                  distro=settings.DEFAULT_DISTRO,
                  base_python_version=settings.DEFAULT_PYTHON_VERSION,
                  python_versions=[],
-                 rpm_name=None, proxy=None, venv=True, autonc=False):
+                 rpm_name=None, proxy=None, venv=True, autonc=False,
+                 include_extras=True):
         self.package = package
         self.version = version
         self.save_dir = save_dir
@@ -59,6 +60,7 @@ def __init__(self, package=None, version=None,
         self.proxy = proxy
         self.venv = venv
         self.autonc = autonc
+        self.include_extras = include_extras
         self.pypi = True
         suffix = os.path.splitext(self.package)[1]
         if (os.path.exists(self.package)
@@ -272,6 +274,7 @@ def metadata_extractor(self):
                 self.version,
                 self.rpm_name,
                 self.venv,
+                self.include_extras,
                 base_python_version)
 
         return self._metadata_extractor

pyp2rpm/metadata_extractors.py

@@ -178,6 +178,7 @@ class LocalMetadataExtractor(object):
 
     def __init__(self, local_file, name, name_convertor, version,
                  rpm_name=None, venv=True,
+                 include_extras_require=True,
                  base_python_version=None,
                  metadata_extension=False):
         self.local_file = local_file
@@ -187,6 +188,7 @@ def __init__(self, local_file, name, name_convertor, version,
         self.version = version
         self.rpm_name = rpm_name
         self.venv = venv
+        self.include_extras_require = include_extras_require
         self.base_python_version = base_python_version
         self.metadata_extension = metadata_extension
         self.unsupported_version = None
@@ -382,6 +384,10 @@ def runtime_deps(self):  # install_requires
             list of runtime dependencies of the package
         """
         install_requires = self.metadata['install_requires']
+
+        if self.include_extras_require:
+            install_requires += self.metadata['install_requires_extras']
+
         if self.metadata[
                 'entry_points'] and 'setuptools' not in install_requires:
             install_requires.append('setuptools')  # entrypoints
@@ -406,6 +412,10 @@ def build_deps(self):  # setup_requires [tests_require, install_requires]
 
         if 'setuptools' not in build_requires:
             build_requires.append('setuptools')
+
+        if self.include_extras_require:
+            build_requires += self.metadata['setup_requires_extras']
+
         return sorted(self.name_convert_deps_list(deps_from_pyp_format(
             build_requires, runtime=False)))