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

Replace boost::filesystem with std::filesystem (C++17) #120

Closed
vasdommes opened this issue Aug 22, 2023 · 0 comments
Closed

Replace boost::filesystem with std::filesystem (C++17) #120

vasdommes opened this issue Aug 22, 2023 · 0 comments
Assignees
@vasdommes
Labels
cosmetics
Milestone
2.6.0

Comments

@vasdommes
Copy link
Collaborator

We're using C++ 17 (see also #113), and it's better to use standard library instead of boost whenever possible.
@vasdommes vasdommes self-assigned this Aug 22, 2023
vasdommes added a commit to vasdommes/sdpb that referenced this issue Aug 23, 2023
@vasdommes
Fix davidsd#120 Replace boost::filesystem with std::filesystem (C++17)
3544227 
namespace fs = std::filesystem;

remove boost_filesystem dependency from waf
@vasdommes vasdommes mentioned this issue Aug 24, 2023
Merged
vasdommes added a commit that referenced this issue Aug 24, 2023
@vasdommes
Merge pull request #121 from vasdommes/std_filesystem
825fb2f 
Fix #120 Replace boost::filesystem with std::filesystem (C++17)
@vasdommes vasdommes added this to the 2.6.0 milestone Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vasdommes vasdommes

Labels
cosmetics
Projects
None yet
Milestone
2.6.0
Development

No branches or pull requests
1 participant
@vasdommes