@dmnks commented on this pull request.


> @@ -275,6 +275,8 @@ static keystore *getKeystore(rpmts ts)
            ts->keystore = new keystore_fs();
        } else if (rstreq(krtype, "rpmdb")) {
            ts->keystore = new keystore_rpmdb();
+       } else if (rstreq(krtype, "openpgp")) {
+           ts->keystore = new keystore_openpgp_cert_d();

Nevermind the above; I've realized that `openpgp` is way too generic, this 
whole signature business is about OpenPGP (at least `keystore_rpmdb` is, too) 
so just keep the name as is, it's good.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3437#discussion_r1836662583
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3437/review/[email protected]>
_______________________________________________
Rpm-maint mailing list
[email protected]
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to