Skip to content

fix slackware package detection#67

Merged
ThatOneCalculator merged 1 commit into
ThatOneCalculator:mainfrom
rrotter:slackware
Jun 6, 2025
Merged

fix slackware package detection#67
ThatOneCalculator merged 1 commit into
ThatOneCalculator:mainfrom
rrotter:slackware

Conversation

@rrotter
Copy link
Copy Markdown
Contributor

@rrotter rrotter commented Jun 5, 2025

Fix invalid case statement match to make slackpkg detection work

Couldn't get a slackware iso to boot on my computer, but did manage to get a docker image of slackware to run to confirm this is correct. Resolves #65.

bash-5.1$ ./nerdfetch  # before

      ___     testuser@slackware
     (.. \      Slackware 15.0 x86_64
     (<> |      5.10.0
    //  \ \   󰍛  13764/128817 MiB (10%)
   ( |  | /|  󰏔  368 (bin)
  _/\ __)/_)  󰅶  3 days, 7 hours, 26 mins
  \/-____\/     ██████████████████
bash-5.1$ ./nerdfetch-patched  # after

      ___     testuser@slackware
     (.. \      Slackware 15.0 x86_64
     (<> |      5.10.0
    //  \ \   󰍛  13787/128817 MiB (10%)
   ( |  | /|  󰏔  60 (slackpkg)
  _/\ __)/_)  󰅶  3 days, 7 hours, 26 mins
  \/-____\/     ██████████████████
bash-5.1$ diff nerdfetch nerdfetch-patched
218c218
< /usr/sbin/slackpkg) packages="$(ls /var/log/packages | wc -l)" ;;
---
> slackpkg) packages="$(ls /var/log/packages | wc -l)" ;;
bash-5.1$ echo $PATH
/usr/local/bin:/bin:/usr/bin

@ThatOneCalculator
Copy link
Copy Markdown
Owner

Ty again!!

@ThatOneCalculator ThatOneCalculator merged commit 129baf2 into ThatOneCalculator:main Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

slackpkg detection (probably) doesn't work

2 participants