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
torsava commented on this pull request.
> 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.
I've changed it to `\d+\.\d` so we don't get a Python Y2K equivalent. :)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
ffesti commented on this pull request.
> 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.
Any chance this commit can be backported to current Fedora branches?
`rpm.addSign` and `rpm.delSign` in the python extension are completely broken
without it (see https://bugzilla.redhat.com/show_bug.cgi?id=1393659).
--
You are receiving this because you are subscribed to this thread.
Reply
torsava commented on this pull request.
> 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.
Merged #154.
--
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#event-963210836___
Rpm-maint mailing list
Thank *you!*
--
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#issuecomment-280033726___
Rpm-maint mailing list
