Skip to content

Add option to choose boost filesystem over std filesystem - #6005

Merged
larshg merged 2 commits into
PointCloudLibrary:masterfrom
mvieth:prefer_boost_filesystem
Apr 21, 2024
Merged

Add option to choose boost filesystem over std filesystem#6005
larshg merged 2 commits into
PointCloudLibrary:masterfrom
mvieth:prefer_boost_filesystem

Conversation

@mvieth

@mvieth mvieth commented Apr 14, 2024

Copy link
Copy Markdown
Member

Currently, std filesystem is chosen if it is available (if compiled as C++17 or higher). However, it might be good to have the option to choose boost filesystem over std filesystem, even if the latter is available, in case there is some problem with std filesystem.
With this commit, std filesystem is still used if available, unless the user has defined the preprocessor symbol PCL_PREFER_BOOST_FILESYSTEM

Currently, std filesystem is chosen if it is available (if compiled as C++17 or higher). However, it might be good to have the option to choose boost filesystem over std filesystem, even if the latter is available, in case there is some problem with std filesystem.
@mvieth mvieth added changelog: enhancement Meta-information for changelog generation module: common labels Apr 14, 2024
@mvieth mvieth added this to the pcl-1.14.1 milestone Apr 18, 2024
@larshg

larshg commented Apr 19, 2024

Copy link
Copy Markdown
Contributor

Could we make a nice cmake option to enable this?

@larshg
larshg merged commit 6e1973d into PointCloudLibrary:master Apr 21, 2024
@mvieth
mvieth deleted the prefer_boost_filesystem branch April 21, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: enhancement Meta-information for changelog generation module: common

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants