Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: +finish_rebase () { + if test -f $state_dir/autostash + then + stash_sha1=$(cat $state_dir/autostash) + if git stash apply $stash_sha1 21 /dev/null + then + echo Applied

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Ramkumar Ramachandra
Junio C Hamano wrote: Writing it like this: [...] with a blank line before the next echo, it would be more readable. This feels funny. Why not [...] without an extra command substitution with an echo? I'll re-roll if there are more comments. Otherwise, can you fix these up locally?

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Matthieu Moy
Ramkumar Ramachandra artag...@gmail.com writes: +finish_rebase () { + if test -f $state_dir/autostash + then + stash_sha1=$(cat $state_dir/autostash) + if git stash apply $stash_sha1 21 /dev/null + then + echo Applied

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Ramkumar Ramachandra
Matthieu Moy wrote: Any reason why this is not using gettext and the other messages do. You can run git stash apply or git stash drop at any time. Fixed. Thanks. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Matthieu Moy
Ramkumar Ramachandra artag...@gmail.com writes: Matthieu Moy wrote: Any reason why this is not using gettext and the other messages do. You can run git stash apply or git stash drop at any time. Fixed. Thanks. After thinking a bit, I have another nit about the message: it's not clear

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: Junio C Hamano wrote: Writing it like this: [...] with a blank line before the next echo, it would be more readable. This feels funny. Why not [...] without an extra command substitution with an echo? I'll re-roll if there are more

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-05-13 Thread Ramkumar Ramachandra
Junio C Hamano wrote: No, thanks. I won't be even taking the patch right now, so you have plenty of time ;-). There were some additional comments from Matthieu, so I will re-roll (just this part). I've even posted a stash series based on this one. Do you have any additional comments? -- To

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-24 Thread Ramkumar Ramachandra
Phil Hord wrote: Because I am in a git-rebase which has apparently failed, I would expect 'git rebase --abort' would save me here. But it does not and you have given me some unique instructions to try to recover. I suppose rebase--abort cannot be made to recover in this case because this is

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-24 Thread Matthieu Moy
Phil Hord phil.h...@gmail.com writes: Because I am in a git-rebase which has apparently failed, I would expect 'git rebase --abort' would save me here. More generally, if I git rebase --abort in the middle of a rebase (not necessarily at the end), I'd expect the stash to be restored. Right

[PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-23 Thread Ramkumar Ramachandra
This new feature allows a rebase to be executed on a dirty worktree. It works by creating a temporary stash and storing it in $state_dir/autostash before the operation, and applying it after a successful operation. It will be removed along with the $state_dir if the operation is aborted. The

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-23 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: +apply_autostash () { + if test -f $state_dir/autostash + then + stash_sha1=$(cat $state_dir/autostash) + git stash apply $stash_sha1 21 /dev/null || + die +$(eval_gettext 'Applying autostash

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-23 Thread Ramkumar Ramachandra
Junio C Hamano wrote: Perhaps finish_rebase or something? Sure. That makes sense. I was too busy struggling with shell code to notice these things ;) -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCH 7/7] rebase: implement --[no-]autostash and rebase.autostash

2013-04-23 Thread Phil Hord
On Tue, Apr 23, 2013 at 10:02 AM, Ramkumar Ramachandra artag...@gmail.com wrote: This new feature allows a rebase to be executed on a dirty worktree. It works by creating a temporary stash and storing it in $state_dir/autostash before the operation, and applying it after a successful