[Rpm-maint] [rpm-software-management/rpm] Add a new option --all-per-file to rpmdeps (#220)

Peter Kjellerstedt notifications at github.com
Thu May 18 15:04:46 UTC 2017


This is an alternative solution to the part in https://github.com/rpm-software-management/rpm/pull/216 that was rejected.

Rather than changing the behavior of the existing `--rpmfcdebug` option, this adds a new option `--all-per-file`, which outputs the same information as `--rpmfcdebug` does, but to stdout. 

The reason why this is needed and why not to just use `rpmdeps --rpmfcdebug ... 2>&1` is that if there is an error, the error message would then end up on stdout as well and never be shown to the user who then has no idea why the command failed (assuming that stdout is caught by some other process that uses the output from `rpmdeps` as its input).

I am not very happy about the option name, `--all-per-file`, but it was the best I could come up with to indicate that it outputs all dependencies per input file. Feel free to suggest alternatives.

You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Add a new option --all-per-file to rpmdeps

-- File Changes --

    M tools/rpmdeps.c (7)

-- Patch Links --

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

-- 
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/pull/220
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20170518/b5a9a22d/attachment.html>


More information about the Rpm-maint mailing list