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

[project] Fix compilation issues in Linux env: filename case sensitivity in Linux. #13

Merged
merged 1 commit into from
May 22, 2023

Conversation

CofinCup
Copy link
Contributor

目前版本的代码由于include路径大小写不一致,在linux文件系统下是无法编译的;
因此我对代码中相关位置进行了修改,分别是以下几处,尽量符合原文中的大小写规范:include HAL/HAL.h->hal/hal.hinclude Startup.*->StartUp.*resources文件夹整体重命名为Resources

@SmallPond
Copy link
Owner

整个PR 没有问题。 commit msg 改成以下描述是否更好?因为 boost linux support 有些歧义,可能会被理解为 X-Knob 在 Linux 机器上又了新的功能,而不是理解为编译问题。

“[project] 修复 Linux 环境下编译问题:Linux 下文件名大小写敏感” 对应英文供参考 “[project] Fix compilation issues in Linux env: filename case sensitivity in Linux.”

@CofinCup CofinCup changed the title boost Linux support [project] Fix compilation issues in Linux env: filename case sensitivity in Linux. May 22, 2023
@CofinCup
Copy link
Contributor Author

似乎不能很优雅的改变commit msg,因此还是force push了

@SmallPond
Copy link
Owner

是的,这种需要 force push

@SmallPond SmallPond merged commit 6f8c02f into SmallPond:master May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants