[Rpm-maint] [rpm-software-management/rpm] Fix don't use 'rpm2archive foo.rpm | tar -x' (PR #2244)
yangchenguang
notifications at github.com
Tue Nov 22 03:17:07 UTC 2022
> This doesn't actually address #2208. It only enables the omission of `-`, and that only works if you also supply another argument (`-n` in the above examples), otherwise rpm2archive fails. Also, a man page update would be in order if we change such behavior.
>
> What we need is to _always_ print to stdout, not a file. But even then, I'm not sure how to keep backward compatibility in case somebody is used to the current behavior (see my comment in the issue).
>
> Thanks for the effort nevertheless. If you plan to do a proper fix (with the above in mind), please open a new PR, I'll close this one now to keep our backlog tidy.
Sorry, I didn't understand your point of view. The modification is to add the expected usage method on the basis of the original one and keep it compatible. Now it is planned to abandon the original method or not deal with other parameter types.For example:
`rpm2archive CUnit-help-2.1.3-22.uel20.noarch.rpm | tar -xz`
I am willing to submit a new PR for this problem. Thank you for your reply
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2244#issuecomment-1322975876
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/2244/c1322975876 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20221121/3ffdf9aa/attachment.html>
More information about the Rpm-maint
mailing list