[Rpm-maint] [rpm-software-management/rpm] Sanitize signing operation output (PR #3424)

Panu Matilainen notifications at github.com
Mon Nov 4 11:54:16 UTC 2024


Only output filenames on addsign/delsign success on verbose mode, no news is good news. Previusly we emitted non-sensical messages like this:

   /some/path/my-1.rpm:
   /some/path/my-2.rpm:

...presumably in anticipation of error messages, but it's saner to have error messages carry the path where relevant.

Loosely related to the multiple signatures stuff - this was driving me crazy when writing tests.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Run all signing tests with runroot since we now can
  * Sanitize rpmsign --addsign/--delsign output

-- File Changes --

    M sign/rpmgensig.cc (14)
    M tests/atlocal.in (1)
    M tests/rpmsigdig.at (66)

-- Patch Links --

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

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

Message ID: <rpm-software-management/rpm/pull/3424 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20241104/8b98eb3c/attachment.html>


More information about the Rpm-maint mailing list