Issues were detected by static analyzer:
[#1](https://github.com/rpm-software-management/rpm/blob/master/lib/rpmfi.c#L738)
[#2](https://github.com/rpm-software-management/rpm/blob/master/lib/rpmfi.c#L1951)
[#3](https://github.com/rpm-software-management/rpm/blob/master/lib/rpmfi.c#L1956)
@DemiMarie commented on this pull request.
> + print("'"..string.gsub(p, "'", "'''").."'".." ") end}
+%sources %{lua: for i, s in ipairs(sources) do \
+ print("'"..string.gsub(s, "'", "'''").."'".." ") end}
```suggestion
print("'"..string.gsub(p, "'", "'''").."' ") end}
@wuyuoss pushed 2 commits.
8a6be594669227ce3b6836b4adc09dbcb7ab552c formatting
a1e44c4791eef8cc3e76501d4cf1f0d371bd7392 Merge branch 'pkcs11' of
https://github.com/wuyuoss/rpm into pkcs11
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
We (Aleksander Adamowski) recently made a change to Kernel fsverity-utils to
[Implement PKCS#11 opaque keys support through OpenSSL pkcs11
engine](https://git.kernel.org/pub/scm/linux/kernel/git/ebiggers/fsverity-utils.git/commit/?id=66b1d8a276cb3836ac275cb9f3f6517a07462737)
and the change is
by enclosing them in single quotes ().
Replace sigle quotes within with \ as needed for sh
Resolves: #1445
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/1781
-- Commit Summary --
*
See #1781
--
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/1644#issuecomment-923858687___
Rpm-maint mailing list
Yes, definitely a duplicate. Closing here - but working on the original issue.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
Closed #1644.
--
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/1644#event-5334505537___
Rpm-maint mailing list
Not 100% sure if this no machine business is 100% correct. But it
looks like a better defense against `strip` failing than relying on the file
name alone.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/1780
-- Commit