[Rpm-maint] [rpm-software-management/rpm] Fix pythondistdeps.py --provides for Python wheels (#154)

Florian Festi notifications at github.com
Wed Feb 15 12:07:46 UTC 2017


ffesti requested changes on this pull request.

Other than the bit sloppy RE this looks good.

>          if not dist.py_version:
-            warn("Version for {!r} has not been found".format(dist), RuntimeWarning)
-            continue
+            # Try to parse the Python version from the path the metadata
+            # resides at (e.g. /usr/lib/pythonX.Y/site-packages/...)
+            import re
+            res = re.search("/python(?P<pyver>[0-9.]{3})/", path_item)

Can you please use a more precise RE pattern . E.g.
\\d\\.\\d or [0-9]\\.[0-9] instead of 
 [0-9.]{3}

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/154#pullrequestreview-21972479
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20170215/a92e2583/attachment.html>


More information about the Rpm-maint mailing list