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

[christos%astron.com@localhost (Christos Zoulas)]

In article <20130307104716.fxf5kembQ0XOuLi3FEqsN/NG@dietcurd.local>,
Steffen "Daode" Nurpmeso  <sdaoden%gmail.com@localhost> wrote:
>"Christos Zoulas" <christos%netbsd.org@localhost> 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.

christos