@pmatilai commented on this pull request.


> @@ -290,3 +291,61 @@ int rpmcliVerifySignatures(rpmts ts, ARGV_const_t argv)
     rpmKeyringFree(keyring);
     return res;
 }
+
+int cliMatchPubkeys(rpmts ts, ARGV_const_t args, int callback(rpmPubkey, 
void*), void * userdata)

A low-cost workaround is to drop RPM_GNUC_INTERNAL from rpmIsValidHex(). It'll 
show up in the library ABI but is basically uncallable because it doesn't have 
a public header, and pretty harmless as such. We have quite a few such items, 
for similar reasons.

It'll just need an extra directive for rpmkeys in the cmake file to allow it to 
include from lib/, see rpmlua/rpmgraph for examples on that.

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

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

Reply via email to