[Rpm-maint] [rpm-software-management/rpm] [rpmplugin] decouple rpmplugin return code from rpmRC (PR #1909)

chantra notifications at github.com
Fri Feb 4 06:28:55 UTC 2022


In #1470, it was mentioned that currently the plugin API is too coupled
to rpmRC code and would benefit from [getting its own return codes](https://github.com/rpm-software-management/rpm/pull/1470#discussion_r551890395) and eventually
become a [public
API](https://github.com/rpm-software-management/rpm/pull/1470#issuecomment-1020165240).

This diff defines a new emum `rpmPluginRC` and its accompanying
`RPMPLUGINRC_{OK,NOTFOUND,FAIL}`, which are currently being used by
plugins, and adjust the existing plugin to use it.

CI build: https://gist.github.com/chantra/c8f61770b133b66c24cfbd5c2ed60a69
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/1909

-- Commit Summary --

  * [rpmplugin] decouple rpmplugin return code from rpmRC

-- File Changes --

    M lib/rpmplugin.h (25)
    M lib/rpmplugins.c (96)
    M lib/rpmplugins.h (48)
    M lib/rpmscript.c (2)
    M lib/rpmtypes.h (9)
    M lib/transaction.c (4)
    M plugins/audit.c (4)
    M plugins/dbus_announce.c (28)
    M plugins/fapolicyd.c (28)
    M plugins/fsverity.c (24)
    M plugins/ima.c (10)
    M plugins/prioreset.c (4)
    M plugins/selinux.c (44)
    M plugins/syslog.c (20)
    M plugins/systemd_inhibit.c (16)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/1909.patch
https://github.com/rpm-software-management/rpm/pull/1909.diff

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

Message ID: <rpm-software-management/rpm/pull/1909 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20220203/73433fa5/attachment-0001.html>


More information about the Rpm-maint mailing list