Re: [PATCH] git-rebase.sh: Change .dotest directory to .git-dotest

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

 



On Tue, 2008-02-05 at 22:41 +0000, Johannes Schindelin wrote:
> Hi,
> 
> [Jari, keep me Cc'ed.  I missed your mail, because you didn't.]
> 
> On Tue, 5 Feb 2008, Linus Torvalds wrote:
> 
> > It all makes perfect sense. Or rather, it made sense way back when. I 
> > agree that it's just totally insane these days, and ".dotest/" should be 
> > renamed to something like ".git/split/" or something.
> 
> But please, please, please not without a proper plan to keep people 
> informed!  I.e. a switch-over plan with deprecation and all.  (IOW what I 
> _already_ wrote to Jari.)
> 
> On at least one machine (not the current one, because I was too lazy), I 
> have a git alias to call when the patch does not apply, to call another 
> program in turn which made it easier for me to integrate a non-applying 
> patch into the current working directory.

Aha, that's the script we're talking about :)

> And guess what: this script accesses .dotest/.  Yes, I know, it was 
> _prone_ to move.
> 
> But I am lucky, I read the git list regularly, I know what truck is going 
> to hit me if I do not change that alias.

I think there's a correlation here: anybody who's meddling with git
implementation details (yes, I'm talking about .dotest here) is probably
also subscribed to this list :)  Seriously though, in git there is often
no clear line between implementation details and supported features, so
it's way to easy to claim everything is set in stone and that the world
will break if we change it.  Especially if you've written a script that
happens to reach a little to far into the git guts.

Kristian

-
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