# suggestion for git rebase -i 7 messages from 2007-07-28 to 2007-07-28. Participants: Seth Falcon, Johannes Schindelin, Jeff King. Thread: https://gitlist.dev/t/9268 ## Seth Falcon, 2007-07-28 04:58 Subject: suggestion for git rebase -i Message-ID: URL: https://gitlist.dev/e/m26445t98z.fsf%40ziti.fhcrc.org ``` The tempfile used by git rebase -i is called "todo". For those using something like emacsclient, this is a bit too generic for the name of a file in a pre-existing editor session. I realize this is a small nit, but how about renaming the file to "git-rebase-todo"? + seth -- Seth Falcon | Computational Biology | Fred Hutchinson Cancer Research Center BioC: http://bioconductor.org/ Blog: http://userprimary.net/user/ ``` ## Johannes Schindelin, 2007-07-28 08:43 Subject: Re: suggestion for git rebase -i Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.64.0707280941380.14781%40racer.site In-Reply-To: ``` Hi, On Fri, 27 Jul 2007, Seth Falcon wrote: > The tempfile used by git rebase -i is called "todo". For those using > something like emacsclient, this is a bit too generic for the name of > a file in a pre-existing editor session. I realize this is a small > nit, but how about renaming the file to "git-rebase-todo"? Why? This is an implementation detail, and should not concern the user. However, I was thinking about adding a "git rebase --status" to show where you are, but that will have to be implemented by someone else, or it has to wait a little. Ciao, Dscho ``` ## Jeff King, 2007-07-28 08:45 Subject: Re: suggestion for git rebase -i Message-ID: <20070728084550.GA13993@sigill.intra.peff.net> URL: https://gitlist.dev/e/20070728084550.GA13993%40sigill.intra.peff.net In-Reply-To: ``` On Sat, Jul 28, 2007 at 09:43:15AM +0100, Johannes Schindelin wrote: > > a file in a pre-existing editor session. I realize this is a small > > nit, but how about renaming the file to "git-rebase-todo"? > > Why? This is an implementation detail, and should not concern the user. Because if you want to setup macros or syntax highlighting specific to editing git-rebase tempfiles, then you have to figure out somehow that you are editing one. Using a more informative name for the tempfile means that you can guess based on the filename. -Peff ``` ## Jeff King, 2007-07-28 08:58 Subject: Re: suggestion for git rebase -i Message-ID: <20070728085849.GB13993@sigill.intra.peff.net> URL: https://gitlist.dev/e/20070728085849.GB13993%40sigill.intra.peff.net In-Reply-To: <20070728084550.GA13993@sigill.intra.peff.net> ``` On Sat, Jul 28, 2007 at 04:45:51AM -0400, Jeff King wrote: > > > a file in a pre-existing editor session. I realize this is a small > > > nit, but how about renaming the file to "git-rebase-todo"? > > > > Why? This is an implementation detail, and should not concern the user. > > Because if you want to setup macros or syntax highlighting specific to > editing git-rebase tempfiles, then you have to figure out somehow that > you are editing one. Using a more informative name for the tempfile > means that you can guess based on the filename. Re-reading his original message, I actually don't think this is what he wanted it for. But I think it's reasonable to give the tempfile a meaningful name anyway for this reason. -Peff ``` ## Seth Falcon, 2007-07-28 15:52 Subject: Re: suggestion for git rebase -i Message-ID: URL: https://gitlist.dev/e/m2vec4seyf.fsf%40ziti.fhcrc.org In-Reply-To: ``` Johannes Schindelin writes: > Why? This is an implementation detail, and should not concern the > user. In the context of an existing editing session, the name of the tempfile is visible and relevant to the user (not just an implementation detail). For example, the tempfile name is useful when multi-tasking and one needs to get back to the buffer. As another reply pointed out, the name could also be used for editor mode customization. So. When editing a commit message, the buffer is named COMMIT_EDITMSG. When editing a rebase -i buffer it is named todo. Is this a real problem? Of course not. But I think a slightly more descriptive name makes sense -- especially if other commands gain interactive modes and a user might have a number of them going at once (in different repos, e.g.). + seth ``` ## Johannes Schindelin, 2007-07-28 20:09 Subject: Re: suggestion for git rebase -i Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.64.0707282109090.14781%40racer.site In-Reply-To: ``` Hi, On Sat, 28 Jul 2007, Seth Falcon wrote: > Johannes Schindelin writes: > > Why? This is an implementation detail, and should not concern the > > user. > > In the context of an existing editing session, the name of the tempfile > is visible and relevant to the user (not just an implementation detail). > For example, the tempfile name is useful when multi-tasking and one > needs to get back to the buffer. As another reply pointed out, the name > could also be used for editor mode customization. Okay, fair enough. But since you want that feature, you get to do the patch. Hint: it is just one line that has to be changed. Ciao, Dscho ``` ## Seth Falcon, 2007-07-28 23:11 Subject: Re: suggestion for git rebase -i Message-ID: URL: https://gitlist.dev/e/m2bqdwrun7.fsf%40ziti.fhcrc.org In-Reply-To: ``` Johannes Schindelin writes: > Okay, fair enough. But since you want that feature, you get to do the > patch. Hint: it is just one line that has to be changed. Heh, I actually wrote the patch first, but thought it was so trivial... :-) ok, I will send the patch and hopefully not screw up the formatting, etc. Feedback and hints welcome. + seth ```