Re: Recent changes for the unsafe directory handling - TINY improvement requested in error output
- From
Bagas Sanjaya <bagasdotme@gmail.com>
- Date
- Apr 20, 2022, 06:16 UTC
- Message-ID
- <e689bb30-dc7f-6e3d-a8d5-864e00de59d4@gmail.com>
- In-Reply-To
- <ec588ff7-b300-bc3a-b51c-67d190b27e9b@neulinger.org>
On 4/19/22 18:26, Nathan Neulinger wrote:
Show 9 quoted lines
> However, if you do a 'git diff' - such as what you might naturally do right before attempting a 'git commit' - you do NOT get any useful response: > > root@SERVER:/local/APPNAME/bin# git diff > warning: Not a git repository. Use --no-index to compare two paths outside a working tree > usage: git diff --no-index [<options>] <path> <path> > ...snip-long-diff-help... > > Please consider a small improvement to have the git diff help output (if displayed due to the repo exclusion warning) - ALSO display the 'git config' suggestion at the end of the output, instead of me having to remember - oh, go issue status to get the command hint to go set the exclusion. Would also be useful to have the git diff output do the same 'unsafe repository' message about ownership. >
I assumed that you have /local/APPNAME/bin shared by users (maybe via NFS or something else), right?
I agreed with your suggestion. Also, I think usage help for git diff in this case just add noise.
-- An old man doll... just what I always wanted! - Clara