Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

FreeBSD Host Model Patch #1588

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

FreeBSD Host Model Patch #1588

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e407d5b
freebsd host model fix
dvogit Oct 24, 2020
ddd8dad
updated freebsd host fix to latest commit
dvogit Oct 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion neofetch
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1316,7 +1316,11 @@ get_model() {
;;

BSD|MINIX)
model=$(sysctl -n hw.vendor hw.product)
if [[ $(uname -s) = "FreeBSD" ]]; then
model=$(kenv smbios.system.version)
else
model=$(sysctl -n hw.vendor hw.product)
fi
Comment on lines +1319 to +1323
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look at other code in neofetch to see how to detect OS/Distro name. There are variables which can be used.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look at other code in neofetch to see how to detect OS/Distro name. There are variables which can be used.

You could have just changed it to $kernel_name for them... or at least tell them that it should be $kernel_name. Not every contributor have time to read through like 13k lines of bash code

;;

Windows)
Expand Down