[Rpm-maint] [rpm-software-management/rpm] Short option for SCM setting. (#564)

Panu Matilainen notifications at github.com
Thu Oct 25 10:35:22 UTC 2018


Okay so I do see some real-world uses for specifying temporary scm from the cli, such as not wanting backups by default but needing to rediff, so you might want to specify "gendiff" on an occasion. Or not wanting the overhead of git or other real scm for regular builds but wanting to use git for bisecting between patches. 

My biggest problems with this are
1) the apparent promise of scm interchangeability when they are not, I think this needs a disclaimer in the man page
2) use of a single letter option for what is quite a rare use

If it's changed to --scm=<SCM> and the use (and limitations) explained in the manual, I'd be okay merging it.

-- 
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/564#issuecomment-433001037
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20181025/30db1cb9/attachment.html>


More information about the Rpm-maint mailing list