Page MenuHomeFreeBSD

Ensure logging retained when BSDINSTALL_LOG is not changed by preamble
Needs ReviewPublic

Authored by stevek on Jun 18 2020, 4:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 11:00 AM
Unknown Object (File)
Sat, Apr 20, 8:12 AM
Unknown Object (File)
Feb 28 2024, 5:38 AM
Unknown Object (File)
Feb 28 2024, 5:34 AM
Unknown Object (File)
Dec 20 2023, 6:33 AM
Unknown Object (File)
Sep 13 2023, 8:35 PM
Unknown Object (File)
Aug 23 2023, 8:38 PM
Unknown Object (File)
Jul 8 2023, 1:06 AM
Subscribers

Details

Reviewers
nwhitehorn
dteske
Summary

Before the preamble script is sourced, initialize BSDINSTALL_LOG with the
file in $debugFile, if it is not already set.

Without this change, the bsdinstall script would assume the preamble set
BSDINSTALL_LOG to empty and the comparison with $debugFile will fail,
causing the log to be re-initialized to /dev/null.

Test Plan

Found during use of scripted bsdinstall.
Without the change, all debug log output after the preamble is lost.
With the change, the log output is seen in the log file, as expected.
Setting BSDINSTALL_LOG in the preamble causes the bsdinstall script to
re-initialize the log to point to the appropriate place.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 31802
Build 29363: arc lint + arc unit

Event Timeline

Thank for submitting. Give me a chance to review, but I do believe this looks like a good candidate for commit. Thank you