Re: [PATCH] Add a command line switch to disable X11 connection

2006-01-28 Thread Roland Illig

Pavel Tsekov wrote:

The patch is pretty simple. Comments, etc are much appreciated.


You forgot to change the documentation in doc/mc.1.in. Besides that, it 
looks fine.


Roland
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel


Re: [PATCH] Add a command line switch to disable X11 connection

2006-01-28 Thread Pavel Tsekov
On Sat, 28 Jan 2006, Roland Illig wrote:

 Pavel Tsekov wrote:
  The patch is pretty simple. Comments, etc are much appreciated.

 You forgot to change the documentation in doc/mc.1.in. Besides that, it
 looks fine.

Are you sure that I didn't change it ?
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel


Re: [PATCH] Add a command line switch to disable X11 connection

2006-01-28 Thread Roland Illig

Pavel Tsekov wrote:

On Sat, 28 Jan 2006, Roland Illig wrote:



Pavel Tsekov wrote:


The patch is pretty simple. Comments, etc are much appreciated.


You forgot to change the documentation in doc/mc.1.in. Besides that, it
looks fine.



Are you sure that I didn't change it ?


From what I can read in your patch, yes. If you changed it locally, 
please include it in the patch next time.


Roland
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel


Re: [PATCH] Add a command line switch to disable X11 connection

2006-01-28 Thread Leonard den Ottolander
Hi Pavel,

On Sat, 2006-01-28 at 12:06 +0200, Pavel Tsekov wrote:
 On Sat, 28 Jan 2006, Roland Illig wrote:
  You forgot to change the documentation in doc/mc.1.in. Besides that, it
  looks fine.
 
 Are you sure that I didn't change it ?

? There's nothing in the patch that I can see...

Leonard.

-- 
mount -t life -o ro /dev/dna /genetic/research


___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel


Re: [PATCH] Add a command line switch to disable X11 connection

2006-01-28 Thread Pavel Tsekov
On Sat, 28 Jan 2006, Leonard den Ottolander wrote:

 On Sat, 2006-01-28 at 12:06 +0200, Pavel Tsekov wrote:
  On Sat, 28 Jan 2006, Roland Illig wrote:
   You forgot to change the documentation in doc/mc.1.in. Besides that, it
   looks fine.
 
  Are you sure that I didn't change it ?

 ? There's nothing in the patch that I can see...

Well, that's obvious - because it is not in the patch. When I submit a
patch am I supposed to post diff containg all my changes ? I wanted to
hear opinions on the patch...

___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel