chattr version/verbose options

chattr version/verbose options

[Lavrentiev, Anton (NIH/NLM/NCBI) [C] via cygwin]

Hi,

Thank you for adding the chattr/lsattr utilities to Cygwin base, I think they can come very handy.

May I have a small suggestion, please:

To use -V/--version (that lsattr does already) consistently for both, and -v/--verbose in chattr (and update the docs too).

-V is conventionally used for the version reporting and -v is usually for verbose logging, so the current -V/--verbose and -v/--version pairs in chattr are a bit off, IMHO.

Thanks for considering this,
Anton Lavrentiev


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

Re: chattr version/verbose options

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 709 bytes --]

On Jan 18 16:12, Lavrentiev, Anton (NIH/NLM/NCBI) [C] via cygwin wrote:
> Hi,
> 
> Thank you for adding the chattr/lsattr utilities to Cygwin base, I think they can come very handy.
> 
> May I have a small suggestion, please:
> 
> To use -V/--version (that lsattr does already) consistently for both, and -v/--verbose in chattr (and update the docs too).
> 
> -V is conventionally used for the version reporting and -v is usually for verbose logging, so the current -V/--verbose and -v/--version pairs in chattr are a bit off, IMHO.

I modeled this after the upstream version.  Compare with
https://linux.die.net/man/1/chattr

¯\_(ツ)_/¯
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: chattr version/verbose options

[Erik Soderquist]

On Fri, Jan 18, 2019 at 11:12 AM Lavrentiev, Anton (NIH/NLM/NCBI) [C] wrote:
>
> Hi,
>
> Thank you for adding the chattr/lsattr utilities to Cygwin base, I think they can come very handy.
>
> May I have a small suggestion, please:
>
> To use -V/--version (that lsattr does already) consistently for both, and -v/--verbose in chattr (and update the docs too).
>
> -V is conventionally used for the version reporting and -v is usually for verbose logging, so the current -V/--verbose and -v/--version pairs in chattr are a bit off, IMHO.


While I agree having the cases reversed from what is typical feels
"off", looking at the package doc, those look to be the package's
choices upstream of us, and so effectively out of our control.  Cygwin
strives to be as accurate to how things work in Linux as is reasonably
possible, and so making Cygwin specific changes to what any given
package uses for its cli options when not absolutely necessary would
go directly against that goal.

Such a request should be addressed to the upstream package maintainers.

-- Erik


--
"I do not think any of us are truly sane, Caleb. Not even you. Courage
is not sanity. Being willing to die for someone else is not sanity."
... "Love is not sane, nor is faith." ... "If sanity lacks those
things, Caleb, I want no part of it."

-- Alexandria Terri in "Weaving the Wyvern" by Alexis Desiree Thorne

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple