On Wed, Jan 11, 2012 at 1:59 PM, jaalto <jari.aa...@cante.net> wrote: > On 2012-01-11 03:31, Sitaram Chamarty wrote: > | On Wed, Jan 11, 2012 at 12:12 AM, Jari Aalto <jari.aa...@cante.net> wrote: > | > retitle 653968 [PATCH] gitolite: gl-setup doesn't warn it opens another > editor (Emacs M-x shell) > | > forwarded 653968 Sitaram Chamarty <sitar...@gmail.com> > | > thanks > | > > | > Sitaram, could you apply this patch to fix Debian bug: > | > > | > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=653968 > | > | Sorry but this does not make any sense to me. > > Doesn't make sense how? What part of it? > > The patch adds additional check for two environment variables before > the incident. Very simple test to check if Emacs is in use and take > actions according to it.
Doesn't make sense to do such extremely specific platform checks inside code. You have the "-q" option -- which, once again, I apologise for not having documented but I now have (and I appreciate you causing it to come to my notice that it was not documented). If you're in an environment where you can't popup an editor, use the "-q" option. Simple. Please do not email my on this again; I am not going to budge on this. -- Sitaram -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org