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

Replace boost filesystem with std filesystem #649

Merged
merged 10 commits into from Oct 5, 2023

Conversation

WhiredPlanck
Copy link
Contributor

Pull request

Issue tracker

Fixes will automatically close the related issue

Fixes # N/A

Feature

Describe feature of pull request

Replace boost filesystem with std filesystem, since there is a filesystem library in the STL since C++17.

Unit test

  • Done

Manual test

  • Done

Code Review

  1. Unit and manual test pass
  2. GitHub Action CI pass
  3. At least one contributor reviews and votes
  4. Can be merged clean without conflicts
  5. PR will be merged by rebase upstream base

Additional Info

Copy link
Member

@lotem lotem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

帮忙研究一下 boost components 其中的 system,还有没有用。不确定是不是 filesystem 依赖于他才加的。

@WhiredPlanck WhiredPlanck force-pushed the filesystem branch 10 times, most recently from d4f6a55 to 8afc383 Compare May 14, 2023 09:06
@WhiredPlanck
Copy link
Contributor Author

帮忙研究一下 boost components 其中的 system,还有没有用。不确定是不是 filesystem 依赖于他才加的。

@lotem boost system 主要是提供 boost::system::error_code 等类,随着 boost filesystem 更换为 std filesystem,相关用例已替换为了 std::error_code,在 f40ee77 中已尝试移除,目前来看没有问题。

@WhiredPlanck
Copy link
Contributor Author

@lotem 这个 PR 目前(我的)第一阶段工作已完成,等待审阅。

@WhiredPlanck WhiredPlanck force-pushed the filesystem branch 5 times, most recently from 1b83688 to 6a50750 Compare October 3, 2023 08:01
use UCRT64 as msystem, which is recommended by msys2.
@WhiredPlanck WhiredPlanck force-pushed the filesystem branch 3 times, most recently from 9cec845 to 5b6b481 Compare October 4, 2023 05:28
@lotem lotem merged commit 8bf01e9 into rime:master Oct 5, 2023
8 checks passed
@WhiredPlanck WhiredPlanck deleted the filesystem branch October 29, 2023 09:56
graphemecluster pushed a commit to TypeDuck-HK/librime that referenced this pull request Nov 2, 2023
* chore: set default C++ standard as 17

* refactor: replace boost filesystem with std filesystem

* build: remove boost filesystem component

* feat: make boost dll use std filesystem

* build(Makefile): raise MACOSX_DEPLOYMENT_TARGET to 10.15

* ci(windows-build.yml): improve msys2 setup

use UCRT64 as msystem, which is recommended by msys2.

* ci(windows-build.yml): use pre-installed msys2

* ci(windows-build.yml): disable git line ending auto conversion on Windows ...

... before run actions/checkout to prevent potential problems from msys2.

Ref: https://github.com/marketplace/actions/setup-msys2#actionscheckout-and-line-endings

* build(src/CMakeLists.txt): explicitly link bcrypt to fix build error with mingw64

* ci(Dockerfile): temporarily apply unmerged patch to verify build
graphemecluster pushed a commit to TypeDuck-HK/librime that referenced this pull request Nov 8, 2023
* chore: set default C++ standard as 17

* refactor: replace boost filesystem with std filesystem

* build: remove boost filesystem component

* feat: make boost dll use std filesystem

* build(Makefile): raise MACOSX_DEPLOYMENT_TARGET to 10.15

* ci(windows-build.yml): improve msys2 setup

use UCRT64 as msystem, which is recommended by msys2.

* ci(windows-build.yml): use pre-installed msys2

* ci(windows-build.yml): disable git line ending auto conversion on Windows ...

... before run actions/checkout to prevent potential problems from msys2.

Ref: https://github.com/marketplace/actions/setup-msys2#actionscheckout-and-line-endings

* build(src/CMakeLists.txt): explicitly link bcrypt to fix build error with mingw64

* ci(Dockerfile): temporarily apply unmerged patch to verify build
graphemecluster pushed a commit to TypeDuck-HK/librime that referenced this pull request Nov 8, 2023
* chore: set default C++ standard as 17

* refactor: replace boost filesystem with std filesystem

* build: remove boost filesystem component

* feat: make boost dll use std filesystem

* build(Makefile): raise MACOSX_DEPLOYMENT_TARGET to 10.15

* ci(windows-build.yml): improve msys2 setup

use UCRT64 as msystem, which is recommended by msys2.

* ci(windows-build.yml): use pre-installed msys2

* ci(windows-build.yml): disable git line ending auto conversion on Windows ...

... before run actions/checkout to prevent potential problems from msys2.

Ref: https://github.com/marketplace/actions/setup-msys2#actionscheckout-and-line-endings

* build(src/CMakeLists.txt): explicitly link bcrypt to fix build error with mingw64

* ci(Dockerfile): temporarily apply unmerged patch to verify build
graphemecluster pushed a commit to TypeDuck-HK/librime that referenced this pull request Mar 18, 2024
* chore: set default C++ standard as 17

* refactor: replace boost filesystem with std filesystem

* build: remove boost filesystem component

* feat: make boost dll use std filesystem

* build(Makefile): raise MACOSX_DEPLOYMENT_TARGET to 10.15

* ci(windows-build.yml): improve msys2 setup

use UCRT64 as msystem, which is recommended by msys2.

* ci(windows-build.yml): use pre-installed msys2

* ci(windows-build.yml): disable git line ending auto conversion on Windows ...

... before run actions/checkout to prevent potential problems from msys2.

Ref: https://github.com/marketplace/actions/setup-msys2#actionscheckout-and-line-endings

* build(src/CMakeLists.txt): explicitly link bcrypt to fix build error with mingw64

* ci(Dockerfile): temporarily apply unmerged patch to verify build
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.

None yet

2 participants