[Rpm-maint] [rpm-software-management/rpm] Support optionally creating backup files with %autopatch or %autosetup (#110)

Panu Matilainen notifications at github.com
Wed Jan 11 14:16:04 UTC 2017


pmatilai requested changes on this pull request.



I'm actually in favor of enabling backups by default. However this patch clearly hasn't been tested, nor reviewed, at all. It refers to a non-existent patches_num variable so any attempt to use %autosetup with this patch ends up in:

`error: lua script failed: [string "<lua>"]:4: attempt to index a nil value (global 'patches_num')`




-- 
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/110#pullrequestreview-16144876
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20170111/5f2ac667/attachment.html>


More information about the Rpm-maint mailing list