tech-userlevel archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/usr.bin/mail

In article <20130307104716.fxf5kembQ0XOuLi3FEqsN/NG@dietcurd.local>,
Steffen "Daode" Nurpmeso  <> wrote:
>"Christos Zoulas" <> wrote:
> |Module Name: src
> |Committed By:        christos
> |Date:                Wed Mar  6 20:49:20 UTC 2013
> |
> |Modified Files:
> |     src/usr.bin/mail: fio.c
> |
> |Log Message:
> |since we are calling realpath() earlier now, we need to expand the name
> |of the folder in case it contained variables like $HOME.
>$HOME would simply be
>  :set folder=

Yes, and how do you get:

set MBOX=$HOME/Mail
set record=$HOME/Mail/sent-items

>however; Mail(1) (and POSIX) yet don't document expansion, but
>only state
>  If this name begins with a `/', mail considers it to be an
>  absolute pathname; otherwise, the folder directory is found
>  relative to your home directory.

Yes, but this worked and broke after your changes.

>patch in PR bin/47577 (the clean one using mblen(3))?
>It's working fine for me on UTF-8, and should do well for all
>multibyte encodings that have a clear notion of sequence leaders
>and follow bytes ('don't know wether there are ambiguous ones);
>it is anyway better than the version in CVS in respect to that.

I really don't want to do a one-off patch for partial handling of
multi-byte characters. If we are going to support multi-byte characters
it should work everywhere.


Home | Main Index | Thread Index | Old Index