Can't compile in cygwin

[Milly]

7bbe961 において追加された Ignore tty 処理で使われている TIOCNOTTY が Cygwin には無いためコンパイルエラーとなります。

同じようなことで修正されてるソースがあったので以下の様にしてみました。
これで以下の環境で動作しています。

• Windows8.1 CYGWIN_NT-6.2 1.7.25(0.270/5/3) x86_64

--- a/autoload/proc.c
+++ b/autoload/proc.c
@@ -486,6 +486,9 @@ vp_pipe_open(char *args)
         }

         {
+#ifdef __CYGWIN__
+            setsid();
+#else
             /* Ignore tty. */
             char name[L_ctermid];
             if (ctermid(name)[0] != '\0') {
@@ -495,6 +498,7 @@ vp_pipe_open(char *args)
                     close(tfd);
                 }
             }
+#endif
         }

         argv = malloc(sizeof(char *) * (argc+1));

[ichizok]

#ifdef __CYGWIN__ より #ifndef TIOCNOTTY でガードした方がいいですね

[Milly]

tty 切り離すのに setsid() 使うのは Cygwin だけなんですかね？
#else~#endif の間で改めて #ifdef TIOCNOTTY したほうがいいのかな(切り離すのあきらめるとして)

[ichizok]

単純に TIOCNOTTY がなければ setsid() でいいと思います。
TIOCNOTTY があるなら、セッション作らずにttyを切り離せるということなので

[Milly]

なるほどです 👍

[Shougo]

修正しました。確認をおねがいします。

[Milly]

Cygwin でのコンパイル、動作に問題ありません。
ありがとうございました。

[Shougo]

了解です