Previously, the $PAGER was opened even with no contents.
Applies patch to no longer do this.
PR: 194547,208497
Differential D18913
freebsd-update: open $PAGER only if necessary aryeeteygerald_rogers.com on Jan 21 2019, 5:55 PM. Authored by Tags None Referenced Files
Details
Diff Detail
Event TimelineComment Actions I like this change in principle, but I think you could simplify the code a little bit (see my comments in line).
Comment Actions One last change request -- could you please use ${PAGER} instead of $PAGER while there? The change looks otherwise fine to me. |