Bug 265797 - small modification in "usr.sbin/bsdinstall/scripts/fetchmissingdists"
Summary: small modification in "usr.sbin/bsdinstall/scripts/fetchmissingdists"
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: conf (show other bugs)
Version: CURRENT
Hardware: amd64 Any
: --- Affects Some People
Assignee: Brad Davis
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-12 15:23 UTC by ghislain
Modified: 2022-09-14 17:09 UTC (History)
2 users (show)

See Also:


Attachments
small modification in "usr.sbin/bsdinstall/scripts/fetchmissingdists" (453 bytes, patch)
2022-08-12 15:23 UTC, ghislain
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description ghislain 2022-08-12 15:23:31 UTC
Created attachment 235867 [details]
small modification in "usr.sbin/bsdinstall/scripts/fetchmissingdists"

The 'error' function in the script 
"usr.sbin/bsdinstall/scripts/fetchmissingdists" has this statement:


bsddialog --backtitle "FreeBSD Installer" --title "Error" --msgbox "$1" 0 0



The suggested change is proposing this statement :

bsddialog --backtitle "$OSNAME Installer" --title "Error" --msgbox "$1" 0 0


the $OSNAME variable declared in the Makefile contained in the bsdinstall directory centrally controls the name of the Operating System so this is consistent with other scripts in the bsdinstall directory.
Comment 1 Alfonso S. Siciliano freebsd_committer freebsd_triage 2022-09-05 22:01:54 UTC
Review: https://reviews.freebsd.org/D36235

The Makefile controls OSNAME in the tools written in C, partedit, distextract and distfetch. The OSNAME in the scripts is controlled by bsdconfig "/usr/share/bsdconfig/common.subr".
Comment 2 ghislain 2022-09-13 03:33:53 UTC
(In reply to Alfonso S. Siciliano from comment #1)


Thank you for the thread. The review ( https://reviews.freebsd.org/D36235 ) does contain the same modifications I proposed. 

In addition, thanks for the pointer. I'm still hard at work to get a better understanding of how this beastie works. slowly but surely!
Comment 3 commit-hook freebsd_committer freebsd_triage 2022-09-14 17:07:16 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/src/commit/?id=2913e785f057f54f7ee825fe4f9fe3c039c0f78f

commit 2913e785f057f54f7ee825fe4f9fe3c039c0f78f
Author:     Brad Davis <brd@FreeBSD.org>
AuthorDate: 2022-09-14 17:04:07 +0000
Commit:     Brad Davis <brd@FreeBSD.org>
CommitDate: 2022-09-14 17:06:11 +0000

    bsdinstall: fix a couple stragglers in whitelabeling the scripts

    PR:             265797
    Reviewed by:    allanjude, asiciliano
    Sponsored by:   Rubicon Communications, LLC ("Netgate")
    Differential Revision:  https://reviews.freebsd.org/D36235

 usr.sbin/bsdinstall/scripts/checksum          | 2 +-
 usr.sbin/bsdinstall/scripts/fetchmissingdists | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)
Comment 4 Brad Davis freebsd_committer freebsd_triage 2022-09-14 17:09:23 UTC
Good catch, sorry I didn't see this until after I opened the review.