Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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