KittyKatt / screenFetch

Fetches system/theme information in terminal for Linux desktop screenshots.
GNU General Public License v3.0
3.84k stars 454 forks source link

macOS Disk Size #685

Closed unixb0y closed 1 year ago

unixb0y commented 4 years ago

I'm submitting a ... (check one with "x")

[x] bug report
[ ] new distro request

Bug report My disk usage is around 190GB, but shows as 11G / 500G.

> screenfetch -v
...
:: Finding current disk usage...found as '11G / 500G (4%%)'
...

If I execute the lines of the scrip that should be executed manually, it works fine though:

> totaldisk=$(df -H /System/Volumes/Data 2>/dev/null | tail -1)
> awk '{print $3}' <<< "${totaldisk}"
189G
FrankKair commented 3 years ago

+1 I'm experiencing the same issue

KittyKatt commented 3 years ago

Hmmmm. Unsure why that's happening. I'll need to check it out locally and see what's happening. Thanks for the report!

unixb0y commented 3 years ago

@KittyKatt awesome thanks!

vu1Art1st commented 1 year ago

@unixb0y @FrankKair I recently encountered this problem and I don’t know if you still need a solution. To solve the bug, My change was: [Original code] totaldisk=$(df -H / 2>/dev/null | tail -1) [Changed code] totaldisk=$(df -H /System/Volumes/Data 2>/dev/null | tail -1)

unixb0y commented 1 year ago

@unixb0y @FrankKair I recently encountered this problem and I don’t know if you still need a solution. To solve the bug, My change was: [Original code] totaldisk=$(df -H / 2>/dev/null | tail -1) [Changed code] totaldisk=$(df -H /System/Volumes/Data 2>/dev/null | tail -1)

Awesome, this seems to work! I just realized that it's already part of screenfetch-dev in the main branch of this repo. So I just copied all its contents into my screenfetch bash script.

Seems like a new release is due, no? Been a few years since the last one.

vu1Art1st commented 1 year ago

@unixb0y @FrankKair I recently encountered this problem and I don’t know if you still need a solution. To solve the bug, My change was: [Original code] totaldisk=$(df -H / 2>/dev/null | tail -1) [Changed code] totaldisk=$(df -H /System/Volumes/Data 2>/dev/null | tail -1)

Awesome, this seems to work! I just realized that it's already part of screenfetch-dev in the main branch of this repo. So I just copied all its contents into my screenfetch bash script.

Seems like a new release is due, no? Been a few years since the last one.

Yeah, it seems like a new release is due, and maybe you can take a look at issue #750? Those people also helped me fix a bug when I used it on Ventura 13.3.1 (Intel i9).

unixb0y commented 1 year ago

FIXED IN MAIN BRANCH VERSION. NOT IN LATEST (3.9.1, 4 years old....) RELEASE THOUGH.