As this is only generating a list of all directories in the file list and their
parents it would probably be faster to first strip the filenames and run sort
-u before adding the parent dirs. Not sure if this is really worth the effort,
though.
--
You are receiving this because you are
@ignatenkobrain I started a repo to flesh out some of the requirement
conversions from python dist tools to rpm. It doesn't mangle versions right
now, but maybe you could open an issue there and describe what should happen?
https://github.com/gordonmessmer/pyreq2rpm/
--
You are receiving
Merged #982 into master.
--
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/982#event-2901943796___
Rpm-maint mailing list
@Conan-Kudo Seems only right that I fix what I broke. :)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@gordonmessmer You beat me to it. Thanks for this!
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Conan-Kudo approved this pull request.
--
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/982#pullrequestreview-335495931___
> python3.7dist(molecule) == 3.0a5.dev15+gec8c85e9.d20191124
It would be very cool if we could actually mangle this to a proper RPM
notation, but this is definitely outside of the scope of this PR.
--
You are receiving this because you are subscribed to this thread.
Reply to this email
@gordonmessmer pushed 1 commit.
d8d1da567caeb7ff7e1f90a2a80beba21aa2d996 scripts/pythondistdeps: Only print
rich dep list when required to.
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
gordonmessmer commented on this pull request.
> @@ -269,7 +269,10 @@
spec_list.append('{n} >= {v} with {n} <
{vnext}'.format(n=name, v=spec[1], vnext=next_ver))
else:
spec_list.append('{} {} {}'.format(name, spec[0], spec[1]))
-
```
[gordon@vagabond:~/git/rpm]$ find /home/gordon/git/molecule-root/ | python3
~/git/rpm/scripts/pythondistdeps.py -R
python(abi) == 3.7
python3.7dist(ansible) >= 2.5
(python3.7dist(ansible-lint) >= 4.1.1a2 with python3.7dist(ansible-lint) < 5)
python3.7dist(cerberus) >= 1.3.1
ignatenkobrain commented on this pull request.
> @@ -269,7 +269,10 @@
spec_list.append('{n} >= {v} with {n} <
{vnext}'.format(n=name, v=spec[1], vnext=next_ver))
else:
spec_list.append('{} {} {}'.format(name, spec[0], spec[1]))
-
This should resolve #981
--
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/982#issuecomment-568068686___
Rpm-maint mailing list
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/982
-- Commit Summary --
* scripts/pythondistdeps: Only print rich dep list when required to.
-- File Changes --
M scripts/pythondistdeps.py (5)
-- Patch Links --
@hroncok Yep. But I'm pretty sure you can't do this exact thing on Requires
either...
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
oh, this is for provides?
--
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/issues/981#issuecomment-568048318___
Rpm-maint mailing list
> but also you're not allowed to do rich Provides at all
… which would be so amazing, wouldn't it? :)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Yes, but also you're not allowed to do rich Provides at all
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
When the list contains just one item, the parenthesis should not be there,
correct?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
cc: @gordonmessmer, @hroncok
--
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/issues/981#issuecomment-568039425___
Rpm-maint mailing
When bootstrapping RPM 4.15.1 with pythondistdeps fixes backported in
OpenMandriva Cooker, I've encountered the following bug:
```
BUILDSTDERR: error: No rich dependencies allowed for this type:
(python3.8dist(createrepo-c) == 0.15.4)
BUILDSTDERR: error: No rich dependencies allowed for this
Updated the code. It's now much faster and also implements prefix search.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
@mlschroe pushed 1 commit.
66c055f43b184ab42dc02a219660f866bee48b63 Add support for reading BDB without
the library
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
22 matches
Mail list logo