On Mar 26, 2017 11:25 AM, "Christophe Gouiran" <bechris13...@gmail.com> wrote:
Please find as attached file another patch which: 1. First test for a real terminal before spawning the pager command. 2. No more paginate json command. I see that most of you complain about this proposed feature. It was only a proposition, if it is not accepted I won't care. However, it would be possible to make both world happy: 1. Those who don't like this feature simply don't set a pager-command setting. 2. Those who want to use it set a pager-command setting. And, IMHO adding or removing a '|' at the end of command names is not a maintenance nightmare. I'm sorry if I've offended. My point was that one code path that works with all future commands or changes to commands is not bad, even if it is not something I will personally use. Needing to tweak code paths unrelated to DVCS functionality to maintain this feature going forward seems less than ideal. I do commend you for stepping forward to implement it. I just disagree with the idea of making it command specific.
_______________________________________________ fossil-users mailing list fossil-users@lists.fossil-scm.org http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users