Junio C Hamano <gitster@xxxxxxxxx> writes: > Actually, the situation is now even worse than I originally thought > especially with Jeff's pager.<cmd> patch on 'master' recently. For > example, you can screw yourself quite badly by forcing diff-files used in > the scripts you run to page, defeating --exit-code option. Which means > > (1) It hurts? Don't do it then; but > > (2) Then why are we even allowing to configure the plumbing to page? > > Should we maintain a table of commands that we allow paging to be > customized, and ignore pager.<cmd> for commands that are not in the list? > > Which codepath should issue error messages when the user tries to break > the system by saying "pager.diff-files = true"? Another possibility is to set up an extra process whose sole purpose is to wait for the main process that feeds the pager pipe and relay its exit status to the outside world. But I do not think we would want to go there... -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
