pmatilai approved this pull request.
I thought I actually removed the ts as an argument to rpmalCreate() back
somewhen as part of overall efforts to minimize what has access to the full
transaction set, but seems that's not the case. Anyway, I can live with that,
it does of course simplify the
Merged #1081 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/1081#event-3064699495___
Rpm-maint mailing list
@mlschroe pushed 1 commit.
4c43d263ee1e01b5f3a9822e8bc24aff17cc0461 Use the transaction set as argument
in rpmalCreate()
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
This pull request **introduces 1 alert** when merging
d9ffe24cd45a16a9c2ea258d40fbf2e967760d2c into
153c5c219844f0f294862c9043b20f4d24f7fa69 - [view on
LGTM.com](https://lgtm.com/projects/g/rpm-software-management/rpm/rev/pr-74d712b30f7e113426127e28a80b606b4871f505)
**new alerts:**
* 1 for
@mlschroe pushed 1 commit.
d9ffe24cd45a16a9c2ea258d40fbf2e967760d2c Use the transaction set as argument
in rpmalCreate()
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
This gets rid of a couple of id-str-id roundtrips and also makes the
dependency hashes in rpmtsCheck() use pool ids instead of strings.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/1081
-- Commit Summary --
* Add