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

LUAオプション付きビルド時のmingwでのパス指定について #1105

Closed
bamchoh opened this Issue Oct 8, 2017 · 6 comments

Comments

Projects
None yet
2 participants
@bamchoh

bamchoh commented Oct 8, 2017

質問・報告の内容

mingw32を用いて、以下の通りにビルドしてみましたところ、lua.hがないと言われました。
https://github.com/vim/vim/blob/master/src/INSTALLpc.txt#L580

LUAオプションの値を /c/projects/lua53 に変えたところ、ビルドが出来ました。これは私の環境のせいなのでしょうか?それとも、記載間違いなのでしょうか?

LUA=c:\\projects\\lua53 のように \ をエスケープするような書き方をしてもビルドは成功するようです。vimというよりmingwの動作仕様のようにも思えるのですが、そのあたりの知識が乏しく、申し訳ないですが、ご教授いただけないでしょうか。

Vimのバージョン

8.0.1176

OSの種類/ディストリ/バージョン

  • Windows 10 Pro 64bit (10.0.15063)

使用している or 関係していそうなプラグイン

  • mingw32-make (4.2.1)
  • lua (5.3)

その他

  • 特になし
@bamchoh

This comment has been minimized.

Show comment
Hide comment
@bamchoh

bamchoh Oct 8, 2017

質問内容にも記載している通り、\ の替わりに / を使用することで解決しておりますので、クローズします。すいませんでした。

bamchoh commented Oct 8, 2017

質問内容にも記載している通り、\ の替わりに / を使用することで解決しておりますので、クローズします。すいませんでした。

@bamchoh bamchoh closed this Oct 8, 2017

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Oct 9, 2017

Member

bash上から実行するか、cmd.exe上から実行するかで、異なってくると思います。(INSTALLpc.txtにはそのことは書いていないのですが…。)
bashでは \ はエスケープとして働きますので、2重にするか、' でくくるか、/ に置き換えるかのいずれかが必要です。
両方で使えるようにするには LUA=c:/projects/lua53 のように / を使うように書き直すのが良いかもしれません。

Member

k-takata commented Oct 9, 2017

bash上から実行するか、cmd.exe上から実行するかで、異なってくると思います。(INSTALLpc.txtにはそのことは書いていないのですが…。)
bashでは \ はエスケープとして働きますので、2重にするか、' でくくるか、/ に置き換えるかのいずれかが必要です。
両方で使えるようにするには LUA=c:/projects/lua53 のように / を使うように書き直すのが良いかもしれません。

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Oct 9, 2017

Member

改善の余地ありということで、reopenしておきます。

Member

k-takata commented Oct 9, 2017

改善の余地ありということで、reopenしておきます。

@k-takata k-takata reopened this Oct 9, 2017

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Oct 9, 2017

Member

最新版の INSTALLpc.txt にいろいろと間違いが見つかったので、まとめて投げておきました。
https://groups.google.com/d/msg/vim_dev/2x1bJqOYKEY/KrHdHmnJAgAJ

Member

k-takata commented Oct 9, 2017

最新版の INSTALLpc.txt にいろいろと間違いが見つかったので、まとめて投げておきました。
https://groups.google.com/d/msg/vim_dev/2x1bJqOYKEY/KrHdHmnJAgAJ

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Oct 21, 2017

Member

vim-jp.org に記載しているビルド手順のページも更新。
http://vim-jp.org/docs/build_windows_mingw.html

Member

k-takata commented Oct 21, 2017

vim-jp.org に記載しているビルド手順のページも更新。
http://vim-jp.org/docs/build_windows_mingw.html

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Nov 6, 2017

Member

Fixed by 8.0.1268: vim/vim@3bf8c3c

Member

k-takata commented Nov 6, 2017

Fixed by 8.0.1268: vim/vim@3bf8c3c

@k-takata k-takata closed this Nov 6, 2017

@k-takata k-takata added the xlose/fixed label Nov 6, 2017

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