Re: Help fixes Misc.

2002-09-05 Thread Pavel Roskin
Hi, David! In the c) piece there was a fix for a similar weakness when looking for the closing bracket. Here you have the patch for this and the diff in the help file I used to make mc crash / test the patch (check with the previous one, anyway). Sorry, I didn't realize that. Applied now.

Re: tilde-bug in vfs.c

2002-09-05 Thread Pavel Roskin
Hello, Oskar! There is a bug which makes mc display regular file named '~root' (or '~someaccount') as a directory. I know about this problem. I even tried to fix it, but could not find a good solution. Basically, the title expansion should be completely done on the highest level, when the

hardcoded /tmp instead of TMPDIR

2002-09-05 Thread Oskar Liljeblad
According to the bug report at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=142179repeatmerged=yes MC does not use $TMP. (The reporter must have meant $TMPDIR.) I did a grep (with MC of course :) for files with /tmp and found one place in subshell.c. This patch ought to fix it. I tested it

Re: hardcoded /tmp instead of TMPDIR

2002-09-05 Thread Nerijus Baliunas
On Thu, 5 Sep 2002 22:16:55 +0200 Oskar Liljeblad [EMAIL PROTECTED] wrote: MC does not use $TMP. (The reporter must have meant $TMPDIR.) BTW, it would be nice if mc created temporary files in $TMPDIR/mc-username instead of $HOME/.mc/tmp (the ones with directory names inside as I understand).

Re: tilde-bug in vfs.c

2002-09-05 Thread Oskar Liljeblad
On Thursday, September 05, 2002 at 16:16, Pavel Roskin wrote: [..] What does your patch implement? How does it work? Please always comment non-obvious patches. In this case we really should make a decision before we go ahead and patch the code. Oh, I thought it was so simple. :) Anyway,

-P fix

2002-09-05 Thread Oskar Liljeblad
Here's my take at fixing this in the TODO file: reimplement -P in a safer way (temp file instead of stdout) - possibly security issues. My solution is to add another option -p that will allow you to specify a file that the last working directory is written to. A new function