Macchina-CLI / macchina

A system information frontend with an emphasis on performance.
https://crates.io/crates/macchina
MIT License
1.24k stars 47 forks source link

-o isn't working properly #299

Closed Magniquick closed 6 months ago

Magniquick commented 6 months ago

Describe the bug -o is outputting (error: Found argument 'Kernel' which wasn't expected, or isn't valid in this context)

To Reproduce Steps to reproduce the behavior:

  1. run ❯ macchina --theme Beryllium -o Host Kernel Distribution Packages DesktopEnvironment Memory ProcessorLoad Shell Terminal WindowManager Battery
  2. errors out with:
    error: Found argument 'Kernel' which wasn't expected, or isn't valid in this context
    Usage: macchina [OPTIONS]
    For more information try '--help'

Expected behavior proper output

Screenshots

Screenshot 2023-12-30 at 8 14 00 PM

macchina --doctor output

❯ macchina --doctor
Let's check your system for errors... Here's a summary:

We've collected 18 readouts, including 2 failed read(s) and 1 read(s) which resulted in a warning.
Readout "LocalIP" failed with message: Unable to get local IPv4 address.
Readout "Battery" failed with message: Dictionary does not contain information about the battery. Are you using a third-party battery?

1 of the 3 unsuccessful read(s) resulted in a warning:
Readout "Distribution" threw a warning with message: Since you're on macOS, there is no distribution to be read from the system.

System Information You don't have to provide this information if you're not comfortable doing so, but it'll help us solve the issue a lot faster.

grtcdr commented 6 months ago

Fixed, thanks for reporting!

Magniquick commented 6 months ago

@grtcdr Fixed, thanks for reporting!

Just built from source, still seems to be broken

Screenshot 2023-12-31 at 2 13 10 PM
grtcdr commented 6 months ago

That's really weird, it seems to work for me (casing is different however, this issue should be addressed separately).

2023-12-31T12:45:12,684528323+01:00

Magniquick commented 6 months ago

That's really weird, it seems to work for me (casing is different however, this issue should be addressed separately).

Comma seperation along with leaving no spaces work, thanks !

grtcdr commented 6 months ago

I forgot to mention that multiple arguments to a flag require a delimiter. Glad to hear it's working.