Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Pavel Tsekov
On Sun, 5 Feb 2006, Arpi wrote:

 accept-screen.TERM-format.patch
 Newer GNU Screen sets TERM to screen.xxx where xxx is the client's
 terminal name. Currently mc only accepts TERM=screen, not TERM=screen*

This patch seems pretty straight forward, but I'd like to discuss it
first...

From screen's info manual:

[...]
   When `screen' tries to figure out a terminal name for itself, it
first looks for an entry named `screen.TERM', where TERM is the
contents of your `$TERM' variable.  If no such entry exists, `screen'
tries `screen' (or `screen-w', if the terminal is wide (132 cols or
more)).  If even this entry cannot be found, `vt100' is used as a
substitute.

   The idea is that if you have a terminal which doesn't support an
important feature (e.g. delete char or clear to EOS) you can build a new
termcap/terminfo entry for `screen' (named `screen.DUMBTERM') in which
this capability has been disabled.  If this entry is installed on your
machines you are able to do a rlogin and still keep the correct
termcap/terminfo entry.  The terminal name is put in the `$TERM'
variable of all new windows.
[...]

So one may endup with TERM set to screen.linux - in this case I am not
really sure that

  define_sequences (xterm_key_defines);

should be executed. In fact after reading the info manual of screen I
am not really sure that we should do this for screen in general  but
only for screen inside a xterm emulator. Maybe it won't hurt but still..

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


Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Pavel Tsekov
 deb-support-without-dpkg.patch
 For systems withoput dpkg (almost any non-debian OS).
 It handles .deb files as regular .ar archives, as they are in reality.

This would be nice addition. Only if we could avoid the 'O' option of gnu
tar... Does anyone know of a way to avoid it ? In any case I think this
patch is worth adding.

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


Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Pavel Tsekov
On Sun, 5 Feb 2006, Arpi wrote:

 select_dirs.patch
 In select/unselect files dialog:
 let mask/ select directories only, and /mask select dirs+files

This doesn't seem very intuitive. Isn't there another way for achieving
the same thing. How do other file managers behave in this scenario ?
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel


Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Arpi
Hi,

 On Sun, 5 Feb 2006, Arpi wrote:
 
  select_dirs.patch
  In select/unselect files dialog:
  let mask/ select directories only, and /mask select dirs+files
 
 This doesn't seem very intuitive.

Why?
/mask   things in the current (/) directory 
mask/ or /mask/   only directories (path has '/' ending) here
maskfiles  (old behaviour)

 Isn't there another way for achieving
 the same thing. How do other file managers behave in this scenario ?

no idea, probably no such feature there

A'rpi

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


Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Pavel Tsekov
On Mon, 6 Feb 2006, Arpi wrote:

  On Sun, 5 Feb 2006, Arpi wrote:
 
   select_dirs.patch
   In select/unselect files dialog:
   let mask/ select directories only, and /mask select dirs+files
 
  This doesn't seem very intuitive.

 Why?
 /mask   things in the current (/) directory
 mask/ or /mask/   only directories (path has '/' ending) here
 maskfiles  (old behaviour)

To me it is not obvious. When I see `/' I think root directory. Anyway,
this is just me. What worries me most is the fact that this patch changes
the old behaviour. I'd like to hear what other people think of it.

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


[bug #14155] 4.6.1: mouse wheel strangeness

2006-02-06 Thread Pavel Tsekov

Follow-up Comment #17, bug #14155 (project mc):

Under rxvt and aterm we should use the ESC[?1000h sequence since they do not
know better. Both can be detected by searching the environment for COLORTERM.

___

Reply to this item at:

  http://savannah.gnu.org/bugs/?func=detailitemitem_id=14155

___
  Message sent via/by Savannah
  http://savannah.gnu.org/

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


[bug #14155] 4.6.1: mouse wheel strangeness

2006-02-06 Thread Pavel Tsekov

Follow-up Comment #18, bug #14155 (project mc):

Eterm understands ESC[?1002h but it behaves as if it was given ESC[?1000h so
I suggest that we use ESC[?1000h.

A patch is on the way.

___

Reply to this item at:

  http://savannah.gnu.org/bugs/?func=detailitemitem_id=14155

___
  Message sent via/by Savannah
  http://savannah.gnu.org/

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


[bug #14155] 4.6.1: mouse wheel strangeness

2006-02-06 Thread Pavel Tsekov

Follow-up Comment #19, bug #14155 (project mc):

See attached patch. Mouse should work fine with rxvt, aterm, Eterm, xterm,
gnome-terminal, konsole... At least that's what I've tested.

___

Additional Item Attachment:

File name: mc-xterm-mouse-tweak.patch Size:2 KB

http://savannah.gnu.org/bugs/download.php?item_id=14155item_file_id=3373

___

Reply to this item at:

  http://savannah.gnu.org/bugs/?func=detailitemitem_id=14155

___
  Message sent via/by Savannah
  http://savannah.gnu.org/

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


[bug #15576] Switching panels in long mode

2006-02-06 Thread Pavel Tsekov

Update of bug #15576 (project mc):

  Status:   Confirmed = Ready For Test 
 Assigned to:None = ptsekov

___

Follow-up Comment #2:

The attached patch fixes the problem.

When both panels are in long list mode and Tab Tab is pressed both panels
remain dirty. A key press triggers a repaint of the dirty panels. First the
currently active panel is drawn and nex the inactive panel thus after the
keypress you see the other panel contents.

___

Additional Item Attachment:

File name: mc-panel-on-focus-fix.patchSize:0 KB

http://savannah.gnu.org/bugs/download.php?item_id=15576item_file_id=3374

___

Reply to this item at:

  http://savannah.gnu.org/bugs/?func=detailitemitem_id=15576

___
  Message sent via/by Savannah
  http://savannah.gnu.org/

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


[bug #14155] 4.6.1: mouse wheel strangeness

2006-02-06 Thread John Pye

Follow-up Comment #20, bug #14155 (project mc):

Hi there,

BTW, are you aware that the 'F10' keyboard shortcut in gnome-terminal doesn't
work: F10 seems to be reserved for accessibility purposes for activating the
GUI menu bar in GNOME programs.

___

Reply to this item at:

  http://savannah.gnu.org/bugs/?func=detailitemitem_id=14155

___
  Message sent via/by Savannah
  http://savannah.gnu.org/

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


Re: AMC patches ported to mc-2006-02-03-13.tar.gz

2006-02-06 Thread Roland Illig

Pavel Tsekov wrote:

On Mon, 6 Feb 2006, Arpi wrote:



On Sun, 5 Feb 2006, Arpi wrote:



select_dirs.patch
   In select/unselect files dialog:
   let mask/ select directories only, and /mask select dirs+files


This doesn't seem very intuitive.


Why?
/mask   things in the current (/) directory
mask/ or /mask/   only directories (path has '/' ending) here
maskfiles  (old behaviour)



To me it is not obvious. When I see `/' I think root directory. Anyway,
this is just me. What worries me most is the fact that this patch changes
the old behaviour. I'd like to hear what other people think of it.


Selecting directories with a mask looks useful to me, but the leading 
slash indeed looks odd.


Concerning prior art: The shell expands */ to all directories in the 
current directory, and * to all files and directories. Here, the 
problem is that mc's behavior already differs from sh's. Otherwise I'd 
say we just imitate the shell.


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