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.

Sign up for GitHub

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

Jump to bottom

We need to use (forward) slash '/' for paths in #include directives of C++, instead of backslash '\', even on Windows #260

Open
kmyk opened this issue May 12, 2020 · 4 comments
Open

We need to use (forward) slash '/' for paths in #include directives of C++, instead of backslash '\', even on Windows #260

kmyk opened this issue May 12, 2020 · 4 comments
Labels
difficulty:middle enhancement New feature or request priority:low

Comments

@kmyk
Copy link
Member

kmyk commented May 12, 2020

a very similar problem to #259
We should print nice error messages.
@kmyk kmyk added bug Something isn't working priority:high difficulty:middle labels May 12, 2020
@kmyk kmyk changed the title We need to use (forward) slash '/' for paths in #include directives of C++, instead of backslash '\' We need to use (forward) slash '/' for paths in #include directives of C++, instead of backslash '\', even on Windows May 12, 2020
@kmyk
Copy link
Member Author

kmyk commented May 19, 2020

Windows 環境がある協力者募集 (#259 (comment) と同様です)

以下を順番に実行すると deps.txt というファイルができるはずなので、その中身を教えてください。WSL ではなく cmd.exe や PowerShell 上で実行してください。

  1. 中身が空の hoge\fuga.h という名前のファイルを作る
  2. 中身が空の hoge\piyo.h という名前のファイルを作る
  3. 中身に #include "hoge\fuga.h" および #include "hoge/piyo.h" が書かれた test.cpp という名前のファイルを作る
  4. $ g++ -std=c++14 -MD -MF deps.txt -MM test.cpp を実行する

cc @null0124 @wakuwinmail

@null0124
Copy link

@kmyk
こちらの deps.txt の中身は
test.o: test.cpp hoge\fuga.h hoge/piyo.h
でした

@kmyk
Copy link
Member Author

kmyk commented May 20, 2020

@null0124 ありがとうございます。

これも #259 と同様に、g++ の出力をそのまま借りる形で解決できそうです。

kmyk added a commit that referenced this issue May 21, 2020
@kmyk
Add an error message for failures of analyzing dependencies
85a6f9a 
see #259 and #260
@kmyk kmyk mentioned this issue May 21, 2020
Merged
@kmyk
Copy link
Member Author

kmyk commented May 21, 2020

A hint message is added at #265

@kmyk kmyk added enhancement New feature or request priority:low and removed bug Something isn't working priority:high labels May 21, 2020
This was referenced Dec 10, 2022
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty:middle enhancement New feature or request priority:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kmyk @null0124