On Fri, 13 Nov 2015, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > On Fri, Nov 13, 2015 at 12:16:59PM +0000, Chris Wilson wrote: >> On Fri, Nov 13, 2015 at 02:05:39PM +0200, Ander Conselvan de Oliveira wrote: >> > Calling git --amend invokes the editor, which will not run if it relies >> > on the terminal for input. So don't do that from dim_apply. >> > >> > Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@xxxxxxxxx> >> >> Presumable the ammend is there for a good reason and removing it keeps >> the state dirty? So maybe --amend --no-edit? > > I don't think there's any reason other than Daniel's gvim workflow. Yes, that's it, there's no functional reason other than to fire up an editor to edit the commit message (add r-b, etc.) This always fails for me and I just do this in the cli separately. Basically three options, just remove it, add tty test suggested by Tvrtko, or add some DIM_POST_APPLY_ACTION configuration. I'm fine with any of them. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
