Thread: [Mlterm-dev-ja] pty name

	荒木です:-)

        現在のところ、pty_list で得られる pty 名は、/dev/... という
        その pty が開いている device 名になっていますが、

        ESC ] 0 ; [window/icon title] BEL
        ESC ] 2 ; [window title] BEL

        で設定できる Window タイトル名も返すようにしたいなと思って
        います。

        pty_list = [device name]:[window title]:[is active or sleeping]

        これだと、mlterm-menu などで表示される名前が分かりやすいもの
        にできていいかなとか思うんですがどうでしょうか。

        # mlconfig,mlterm-menu とも実装を変更する必要がでてきますが...
        
	では
--
Araki Ken
ara...@us...

久保田です。

From: Araki Ken <ara...@us...>
Subject: [Mlterm-dev-ja] pty name
Date: Sat, 08 Mar 2003 17:13:52 +0900

>         現在のところ、pty_list で得られる pty 名は、/dev/... という
>         その pty が開いている device 名になっていますが、
> 
>         ESC ] 0 ; [window/icon title] BEL
>         ESC ] 2 ; [window title] BEL
> 
>         で設定できる Window タイトル名も返すようにしたいなと思って
>         います。
> 
>         pty_list = [device name]:[window title]:[is active or sleeping]
> 
>         これだと、mlterm-menu などで表示される名前が分かりやすいもの
>         にできていいかなとか思うんですがどうでしょうか。
> 
>         # mlconfig,mlterm-menu とも実装を変更する必要がでてきますが...

良いですね。この改良のためなら、mlconfig/mlterm-menu の実装の変更作業も
やりがいがあると思います。

ただし、このままではいくつか問題がありそうです。

・window title に「:」が含まれたらどうなる?
・window title は任意の文字列が含まれるので、
  [Mlterm-dev-ja] FYI: TERMINAL EMULATOR SECURITY ISSUES
  で述べられている問題が起こりうる。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] pty name
	From: Tomohiro KUBOTA <ku...@de...>
	Message-ID: <200...@tm...>
	Date: Sat, 08 Mar 2003 18:20:47 +0900 (JST)

> 良いですね。この改良のためなら、mlconfig/mlterm-menu の実装の変更作業も
> やりがいがあると思います。
>
> ただし、このままではいくつか問題がありそうです。
>
> ・window title に「:」が含まれたらどうなる?

        そうですねぇ。
        
        ほかにも、Window Title に、たとえば、current working directory を
        表示する場合なんかも結構あるみたいですが、そういったときに、pty の
        名前として、 current working directory が表示されるのもどうなのか
        なとか思ってます。
        
        # Window Title に表示される名前と pty の名前が一致するというのは、
        # pty を切り替える際には、分かりやすいので捨てがたいのですが...

        てわけで、まず、pty_name を、Window Title とは別に保持するかどうか。

        次に、その上で、':' は pty_name としては使えないという仕様にするか、
        もしくは、例えば /dev/... をハンドルとして、その pty に対応する
        pty_name を問いあわせる protocol を別途追加するなどにより、':' が含
        まれていても問題ないようにするか。
        # mlconfig や mlterm-menu を変更する手間を考えると後者のほうがいいかな。

        という 2 点について、どう思われます?

> ・window title は任意の文字列が含まれるので、
>   [Mlterm-dev-ja] FYI: TERMINAL EMULATOR SECURITY ISSUES
>   で述べられている問題が起こりうる。

        これに限った問題ではないので、この件が終わり次第、対応を考えます。

	では
--
Araki Ken
ara...@us...

久保田です。

From: Araki Ken <ara...@us...>
Subject: Re: [Mlterm-dev-ja] pty name
Date: Mon, 10 Mar 2003 00:39:40 +0900

>         ほかにも、Window Title に、たとえば、current working directory を
>         表示する場合なんかも結構あるみたいですが、そういったときに、pty の
>         名前として、 current working directory が表示されるのもどうなのか
>         なとか思ってます。

すみません、なぜこれが問題なのか、分からないです。それでいいと思います。

>         てわけで、まず、pty_name を、Window Title とは別に保持するかどうか。

これは、不必要な複雑さだと思います。


>         次に、その上で、':' は pty_name としては使えないという仕様にするか、
>         もしくは、例えば /dev/... をハンドルとして、その pty に対応する
>         pty_name を問いあわせる protocol を別途追加するなどにより、':' が含
>         まれていても問題ないようにするか。
>         # mlconfig や mlterm-menu を変更する手間を考えると後者のほうがいいかな。

前者は仕様として奇妙なので、後者のほうがいいと思います。

ただし、実装は、セキュリティ問題について考えてからのほうがいいと
思います。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] pty name
	From: Araki Ken <ara...@us...>
	Message-ID: <200...@pd...>
	Date: Sat, 15 Mar 2003 00:39:21 +0900

>         doc/en/PROTOCOL の仕様を、以下のように拡張しました。
>
>   set              = "\x1b" "]" "5379" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"
>   get(pty)         = "\x1b" "]" "5380" ";" [ "/dev/..." ":" ] <key> "\x07"
>   get(GUI menu)    = "\x1b" "]" "5381" ";" [ "/dev/..." ":" ] <key> "\x07"
>   save             = "\x1b" "]" "5382" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"
>   set&save         = "\x1b" "]" "5383" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"
>
>         ESC ] 5380 ; /dev/ttyp2 : pty_name BEL
>
>         とすれば、/dev/ttyp2 に対応する pty の名前が得られます。

        この拡張に mlterm-menu を対応させるパッチです。
        
	では
--
Araki Ken
ara...@us...

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] pty name
	From: Tomohiro KUBOTA <ku...@de...>
	Message-ID: <200...@tm...>
	Date: Sat, 15 Mar 2003 20:39:38 +0900 (JST)

> 試してみました。こっちはうまく動くんですが、やっぱり mlconfig は
> うまく動きません。ESC ] 5381 ; <device> : pty_name BEL の <device>
> 部分を、pty_list で取得したそのままを使うようにしても、やっぱり
> だめでした。たとえば dev="/dev/pts/0" などの状況で、
>
> sprintf(query, "%s:pty_name", dev);
> name = mc_get_str_value(query);
>
> とやっても、やっぱり "error" が返ってきます。mlterm-menu と同じ
> エスケープシーケンスを送ってるはずなんですが...
>
> たぶん、ぼくがなにか重大な見落としをやってるのだと思うのですが。。。

        mc_io.c:get_value() の
        if( count < 2 + strlen( key) || strcmp( ret , "#error") == 0)
            ^^^^^^^^^^^^^^^^^^^^^^^^
            ↑ここでひっかかってるのではないですか?

> ところで、<device>:pty_name を問い合わせると、/dev/hogehoge という
> 形式ではなく、ウィンドウタイトルが返ることがあるということでしょうか。
>
> デバイス名とは別に、ウィンドウタイトル名を問い合わせ/設定する
> key を、新たに追加するものだと思っていたので...

        追加する必要はないじゃろと判断したので、そうしてます。
        pty_name の意味を拡張するだけで必要十分かなと。

	では
--
Araki Ken
ara...@us...

久保田です。

From: Araki Ken <ara...@us...>
Subject: Re: [Mlterm-dev-ja] pty name
Date: Sat, 15 Mar 2003 22:52:18 +0900

> > たぶん、ぼくがなにか重大な見落としをやってるのだと思うのですが。。。
> 
>         mc_io.c:get_value() の
>         if( count < 2 + strlen( key) || strcmp( ret , "#error") == 0)
>             ^^^^^^^^^^^^^^^^^^^^^^^^
>             ↑ここでひっかかってるのではないですか?

ありがとうございます。ばっちりそのとおりでした。


> 
> > ところで、<device>:pty_name を問い合わせると、/dev/hogehoge という
> > 形式ではなく、ウィンドウタイトルが返ることがあるということでしょうか。
> >
> > デバイス名とは別に、ウィンドウタイトル名を問い合わせ/設定する
> > key を、新たに追加するものだと思っていたので...
> 
>         追加する必要はないじゃろと判断したので、そうしてます。
>         pty_name の意味を拡張するだけで必要十分かなと。

どっちでもいいと思うのですが、pty_name で問い合わせると必ず
デバイス名なのに、<device>:pty_name で問い合わせるとウィンドウ
タイトルが返るというのは、故意でしょうか?

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

久保田です。

From: Tomohiro KUBOTA <ku...@de...>
Subject: Re: [Mlterm-dev-ja] pty name
Date: Sun, 16 Mar 2003 00:37:12 +0900 (JST)

> ありがとうございます。ばっちりそのとおりでした。

こんな感じです。いかがでしょうか。
cvs commit してしまって、よろしいでしょうか?

(というのは、mlterm-menu のパッチを cvs commit しないのは、
どうしてかな、と思って)。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] pty name
	From: Tomohiro KUBOTA <ku...@de...>
	Message-ID: <200...@tm...>
	Date: Sun, 16 Mar 2003 00:37:12 +0900 (JST)

> どっちでもいいと思うのですが、pty_name で問い合わせると必ず
> デバイス名なのに、<device>:pty_name で問い合わせるとウィンドウ
> タイトルが返るというのは、故意でしょうか?

        故意です。って、やっぱややこいかも ^^;

        あと、mlconfig については、わたしはあまりタッチしませんので、
        久保田さんの御判断で、適宜 commit していただいて結構です。

        mlterm-menu パッチを commit しないのは、わたしが作ったもんじゃな
        いからです。
        コードのスタイルとかポリシーとかあるでしょうし、その辺も含めて、
        kadu さんの merge 待ちです。

	では
--
Araki Ken
ara...@us...

和良です。

On Sun, 16 Mar 2003 11:33:52 +0900
Araki Ken <ara...@us...> wrote:
> > どっちでもいいと思うのですが、pty_name で問い合わせると必ず
> > デバイス名なのに、<device>:pty_name で問い合わせるとウィンドウ
> > タイトルが返るというのは、故意でしょうか?
> 
>         故意です。って、やっぱややこいかも ^^;
> 
>         あと、mlconfig については、わたしはあまりタッチしませんので、
>         久保田さんの御判断で、適宜 commit していただいて結構です。
> 
>         mlterm-menu パッチを commit しないのは、わたしが作ったもんじゃな
>         いからです。
>         コードのスタイルとかポリシーとかあるでしょうし、その辺も含めて、
>         kadu さんの merge 待ちです。
うう、ごめんなさい。patch を merge しました。

あと、mlterm-menu は contribute したものなので、だれがいじるのもまった
く問題ありません。
オープンソースで、CVS にも入れてあるのに、ひとりで開発じゃ寂しいですし :-)

# 私事ですけど、しばらく引越で忙しくなるので、なおさら共同開発推進だっ
# たり
--
KATO Kazuyoshi  +++[>+++++[>+++++<-]<-]>>.----------.>+++++[<+
++++>-]<.-----.++++.----------.++++.-----------.+.>++++++++++.

久保田です。

From: Araki Ken <ara...@us...>
Subject: Re: [Mlterm-dev-ja] pty name
Date: Sat, 15 Mar 2003 12:34:04 +0900


> >         ESC ] 5380 ; /dev/ttyp2 : pty_name BEL
> >
> >         とすれば、/dev/ttyp2 に対応する pty の名前が得られます。
> 
>         この拡張に mlterm-menu を対応させるパッチです。

試してみました。こっちはうまく動くんですが、やっぱり mlconfig は
うまく動きません。ESC ] 5381 ; <device> : pty_name BEL の <device>
部分を、pty_list で取得したそのままを使うようにしても、やっぱり
だめでした。たとえば dev="/dev/pts/0" などの状況で、

sprintf(query, "%s:pty_name", dev);
name = mc_get_str_value(query);

とやっても、やっぱり "error" が返ってきます。mlterm-menu と同じ
エスケープシーケンスを送ってるはずなんですが...

たぶん、ぼくがなにか重大な見落としをやってるのだと思うのですが。。。


ところで、<device>:pty_name を問い合わせると、/dev/hogehoge という
形式ではなく、ウィンドウタイトルが返ることがあるということでしょうか。

デバイス名とは別に、ウィンドウタイトル名を問い合わせ/設定する
key を、新たに追加するものだと思っていたので...

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/

	荒木です:-)

	Subject: Re: [Mlterm-dev-ja] pty name
	From: Tomohiro KUBOTA <ku...@de...>
	Message-ID: <200...@tm...>
	Date: Mon, 10 Mar 2003 07:56:42 +0900 (JST)

>>         次に、その上で、':' は pty_name としては使えないという仕様にするか、
>>         もしくは、例えば /dev/... をハンドルとして、その pty に対応する
>>         pty_name を問いあわせる protocol を別途追加するなどにより、':' が含
>>         まれていても問題ないようにするか。
>>         # mlconfig や mlterm-menu を変更する手間を考えると後者のほうがいいかな。
>
> 前者は仕様として奇妙なので、後者のほうがいいと思います。

        doc/en/PROTOCOL の仕様を、以下のように拡張しました。

  set              = "\x1b" "]" "5379" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"
  get(pty)         = "\x1b" "]" "5380" ";" [ "/dev/..." ":" ] <key> "\x07"
  get(GUI menu)    = "\x1b" "]" "5381" ";" [ "/dev/..." ":" ] <key> "\x07"
  save             = "\x1b" "]" "5382" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"
  set&save         = "\x1b" "]" "5383" ( ";" [ "/dev/..." ":" ] <key> [ "=" <value> ] )* "\x07"

        ESC ] 5380 ; /dev/ttyp2 : pty_name BEL

        とすれば、/dev/ttyp2 に対応する pty の名前が得られます。

        問題ないようでしたら、 commit します。

	では
--
Araki Ken
ara...@us...

久保田です。

From: Araki Ken <ara...@us...>
Subject: Re: [Mlterm-dev-ja] pty name
Date: Sat, 15 Mar 2003 00:39:21 +0900

>         ESC ] 5380 ; /dev/ttyp2 : pty_name BEL
> 
>         とすれば、/dev/ttyp2 に対応する pty の名前が得られます。
> 
>         問題ないようでしたら、 commit します。

ためしてみましたが、うまく動かないようです。

char *
get_pty_title(char *dev)
{
	int n;
	char query[256], *name;

	sscanf(dev, "%*[A-Za-z/]%d", &n);

	sprintf(query, "/dev/ttyp%d:pty_name", n);
	name = mc_get_str_value(query);
	return name;
}

というのを作ってみたのですが...

tool/mlconfig/mc_pty.c へのパッチを添付します。

---
久保田智広 Tomohiro KUBOTA <ku...@de...>
http://www.debian.or.jp/~kubota/