Re: [PATCH] user.default: New config to prevent using the default values for user.*

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Santi Béjar, Thu, Mar 06, 2008 22:45:43 +0100:
> On Wed, Mar 5, 2008 at 9:44 PM, Alex Riesen <raa.lkml@xxxxxxxxx> wrote:
> > Santi Béjar, Wed, Mar 05, 2008 20:18:04 +0100:
> >
> >  > @@ -171,7 +171,7 @@ static const char au_env[] = "GIT_AUTHOR_NAME";
> >  >  static const char co_env[] = "GIT_COMMITTER_NAME";
> >  >  static const char *env_hint =
> >  >  "\n"
> >  > -"*** Your name cannot be determined from your system services (gecos).\n"
> >  > +"*** Your name cannot be determined.\n"
> >
> >  Why not?
> 
> Is this important? Or in another way, is this useful? The important
> thing is how you can fix it. But others think otherwise I'll change it
> to explain the reason.

It is precise explanation of what happened. It could be a hint to fix
gecos field to someone. It is considered useful not to hide
information, even if you have no idea of how useful it is. Others may
be smarter than you.

It could be shortened, thopugh. Maybe remove the asterisks, and
replace "from your system services (gecos)" with a shorter (but
precise) information.

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux