Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mcman version vs mcman --version #47

Closed
tippfehlr opened this issue Jan 15, 2024 · 0 comments
Closed

mcman version vs mcman --version #47

tippfehlr opened this issue Jan 15, 2024 · 0 comments

Comments

@tippfehlr
Copy link

tippfehlr commented Jan 15, 2024

I tried to run ncman --version, just to see it fail.

Just for the sake of correctness, I would propose:

  1. mcman --version printing just the current version
  2. mcman self-update updating mcman. However, that shouldn’t work when installed via some package manager, so I would propose to check if the executable is /usr/bin/mcman, and otherwise continue to self-update (just like rustup: rustup self update is disabled when installed via package manager).

EDIT: just had a look in the code and saw that mcman version doesn’t update mcman.
Cargo handles cargo --version and cargo version the same; I think it would be nice to do the same.

TheAlan404 pushed a commit that referenced this issue Jan 31, 2024
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

No branches or pull requests

2 participants