freebsd-update: open $PAGER only if necessary
ClosedPublic
Actions

Authored by aryeeteygerald_rogers.com on Jan 21 2019, 5:55 PM.

Details

Reviewers

emaste
delphij

Summary

Previously, the $PAGER was opened even with no contents.
Applies patch to no longer do this.

PR: 194547,208497

Test Plan

Verified expected behaviour in interactive sh shell

Lint

Lint Skipped

Unit

Tests Skipped

I like this change in principle, but I think you could simplify the code a little bit (see my comments in line).

usr.sbin/freebsd-update/freebsd-update.sh
1953	Could you please refactor this to something like: cat - modifiedfiles <<- EOF \| ${PAGER} The following files are affected by updates, but no changes have ... EOF instead?

This revision now requires changes to proceed.Jan 22 2019, 5:26 AM

This is indeed much nicer.

Remove unnecessary diff junk.

One last change request -- could you please use ${PAGER} instead of $PAGER while there? The change looks otherwise fine to me.

This revision is now accepted and ready to land.Jan 24 2019, 5:51 AM

Forgot the review link; committed in rS343407

Path

Size

usr.sbin/

freebsd-update/

43 lines