Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test100 is failed on Windows #497

Closed
koron opened this issue Dec 12, 2013 · 7 comments
Closed

test100 is failed on Windows #497

koron opened this issue Dec 12, 2013 · 7 comments

Comments

@koron
Copy link
Member

koron commented Dec 12, 2013

Windows はテストを基本的に ff=dos で実行する。
しかし test100 に含まれる redir & echo は
ff の値に関係なく LF (ff=unix相当) なファイルを吐くらしい。

:redir >>test.out | echo "global value shouldn't be changed and still be 5!" | echo 'ONE: expecting global undolevels: 5, local undolevels: -123456 (default)'|:setglobal undolevels? | echon ' global' | setlocal undolevels? | echon ' local' |echo "" |redir end

test100 ではこれ以外にも ff=dos として test.out に出力するので改行コードが混在してしまい、
いまのままテスト用のMakefileをいじるだけではどうにもできない。


いまのところ考えられる対応方法は

  • test100 の該当部分の出力方法を変える。
  • redir >> {ファイル} 時に ff を見て出力改行コードを決定するようにvimを修正する

ほかに良い手はあるだろうか?

@koron
Copy link
Member Author

koron commented Dec 12, 2013

2 でやるなら message.c の redir_write() で putc() しているところ(3166行目付近)、
そこで判断&変換を噛ませる必要がありそう。

@koron
Copy link
Member Author

koron commented Dec 12, 2013

適当なパッチ作ったら、こんどは test17 でコケたw

あ~そういうことか…

@koron
Copy link
Member Author

koron commented Dec 12, 2013

test100 の :w してるところの前全部に :set ff=unix 足せば良さそう。

@koron
Copy link
Member Author

koron commented Dec 12, 2013

@ghost ghost assigned koron Dec 16, 2013
@k-takata
Copy link
Member

k-takata commented Feb 5, 2014

ずいぶん前に todo.txt に入っていました。

Patch to make test 100 work on MS-Windows. (Taro Muraoka, 2013 Dec 12)

@koron
Copy link
Member Author

koron commented Mar 30, 2014

これ、まだマージされてへんのか… (´・ω・`)

@k-takata
Copy link
Member

k-takata commented Sep 1, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants