JSX を vimproc#system で実行した場合に結果が返ってこない about vimproc.vim

[Shougo]

おそらく、何らかの原因でエラーメッセージが出力されない状態になっているのだと思います。
Windowsだとパイプ接続になるため、そこが原因となることもありえます。
同じコマンドをvimshellから実行するとどうなるでしょうか。

[Shougo]

Ubuntu Linux環境で試して見ました。動作することを確認。
やはりWindows限定の不具合でしょう。となると、vimproc側で対処するのは難しいです。
おそらく、jsxが接続先をパイプかどうか判別しています。
vimprocを用いるとパイプ接続になるため、エラーメッセージが出ないのでしょう。

[osyo-manga]

ご確認ありがとうございます。
vimshell で試してみたところ正しい値が出力されました。

[Shougo]

vimshellで動いた、というのはWindows環境ですか？
うーん、Windowsだとvimshellもパイプ接続になりほとんど動作が変わらないはずなんですが……

[osyo-manga]

はい、そうです。
上記テストは全て Windows 環境になります。

[Shougo]

となると、vimproc側でも対処のしようがあるという事ですかね？
とはいえ、私はWindows環境にJSXを用意していないので、動作検証がかなり大変そうです。
osyo_mangaさんの方でもvimprocのソースコードを調べてもらえるとかなり助かります。

[osyo-manga]

他の人の環境でも再現しないようであれば、わたしの方で原因を調べるのが早そうですね。
手が空いた時にでももうちょっと詳しく調べてみたいと思います。

[Shougo]

私も調べてみる予定ですが、おそらく時間がかかります。
osyo-mangaさんの方でも引き続き調査をお願いいたします。

[Shougo]

私の環境でJSXが入りませんでした。どうやら、インストールされるパッケージがJSXの要求するバージョンと合わない。
gemみたいに複数のバージョンをインストールできるわけではないし……。
インストーラに苦しめられるとは思いませんでしたね。
というわけで、しばらくこっちでは試すことができません。

[osyo-manga]

ふむ、なるほど。
了解しました。

[mattn]

これは jsx が process.exit(0) してるからです。exit(0) すると console.log() がバッファリングした内容を貯めたまま終了するので、コンソールの様にパイプでない出力先に対しては有効ですが、パイプに対しては途中で切れてしまう結果となります。(windowsの場合)
ここでjsxは

• process.exit(0)している
• console.log()を使っている

という動作になっているのですが、console.log() を process.stdout.write に、process.exit(0) を return 0 に変えると、正しく表示されます。

[gfx]

これってWIndows特有の問題ですか？
OSXでも古いnodejsはこのバグがあるようです。実行しているのはどのバージョンでしょう？
0.8.0でも再現しますか？

0.8.0で再現しないなら、特に対処はしません。0.8.0でも再現するなら何らかの対処をしたいと思います。

[mattn]

Sorry about I'm writing this comment in english so that github don't
allow japanese e-mails.
Please don't mind and keep to comment in japanese. btw, it's very hard
to write english text by my handy phone. :(

I reproduced this on version 0.8.0 .
And I could solved this issue with changing like above. Currentry, I
can't read node help file, so I'm not sure, process.stdout may have
functions or methods something like flush.

On 7/4/12, Fuji, Goro
[email protected]
wrote:

�$B$3$l$C$F�(BWIndows�$BFCM-$NLdBj$G$9$+!)�(B
OSX�$B$G$b8E$$�(Bnodejs�$B$O$3$N%P%0$,$"$k$h$&$G$9!#<B9T$7$F$$$k$N$O$I$N%P!<%8%g%s$G$7$g$&!)�(B
0.8.0�$B$G$b:F8=$7$^$9$+!)�(B

0.8.0�$B$G:F8=$7$J$$$J$i!"FC$KBP=h$O$7$^$;$s!#�(B0.8.0�$B$G$b:F8=$9$k$J$i2?$i$+$NBP=h$r$7$?$$$H;W$$$^$9!#�(B

---

Reply to this email directly or view it on GitHub:
#25 (comment)

• Yasuhiro Matsumoto

[gfx]

なるほど。これ、@mattn さんのやりかたでは解決できないんですよ。なぜなら今回問題となっているのは process.exit(1) をするエラーケースなので単純にexitをreturnにするとステータスコードをプロセスから返せなくなります。
また、console.log()は中身はprocess.stdout.write()なので、これの書き換えは意味がありません。
https://github.com/joyent/node/blob/master/lib/console.js#L24

そしてstdout.flush()に類するものはなく、stdout.end()/destroy()は例外を投げると。これは困りましたね。

[mattn]

https://groups.google.com/d/msg/nodejs/qXkr1C2c8vs/567P_mVZacsJ

drain にひっかけると良さそう。

[mattn]

これでうまいこと動いてそげです。

https://gist.github.com/3044319

#!node

function safe_quit(code) {
  if (!process.stdout.write(""))
    process.stdout.on('drain', function() {
      process.exit(code);
    })
  else
    process.exit(code);
}


var outputLength = 1000;
var output = '';
for (var i = 0; i < outputLength; i++) {
  output += i + ': ' + Math.random() + "\n";
}
console.log(output);
safe_quit(3);

[gfx]

ありがとうございます。修正したのでお試しいただけますか。

jsx/JSX@75147e4#L0R462

[mattn]

動作確認出来ました。
閉じても良いと思います。

[Shougo]

JSX本体側で対処してくれたのですね。ありがとうございます。