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

Removing most usage of Boost.Config #1877

Merged
merged 9 commits into from Dec 17, 2015
Merged

Removing most usage of Boost.Config #1877

merged 9 commits into from Dec 17, 2015

Conversation

sithhell
Copy link
Member

This decouples the HPX compile time configuration from that of boost. This
removes the include order dependency between boost.config and hpx.

  • replaced all Boost.Config related macro definitions with the equivalent
    HPX macros
  • Boost config macros used for OS Thread API selection are untouched

This decouples the HPX compile time configuration from that of boost. This
removes the include order dependency between boost.config and hpx.

 - replaced all Boost.Config related macro definitions with the equivalent
   HPX macros
 - Boost config macros used for OS Thread API selection are untouched
Conflicts:
	hpx/util/detail/basic_function.hpp
	hpx/util/detail/function_template.hpp
	hpx/util/detail/unique_function_template.hpp
Conflicts:
	hpx/util/plugin/dll.hpp
	hpx/util/plugin/virtual_constructor.hpp
	hpx/util/thread_specific_ptr.hpp
	hpx/util/time_logger.hpp
	hpx/util/value_logger.hpp
	tests/performance/local/native_tls_overhead.cpp
@hkaiser
Copy link
Member

hkaiser commented Dec 12, 2015

@sithhell What's the status of this PR?

Conflicts:
	hpx/lcos/local/dataflow.hpp
	hpx/runtime/naming/name.hpp
	hpx/runtime/serialization/access.hpp
@sithhell
Copy link
Member Author

I just merged master onto that branch, should be good now.

@hkaiser
Copy link
Member

hkaiser commented Dec 17, 2015

This compiles fine on MSVC, LGTM.

sithhell added a commit that referenced this pull request Dec 17, 2015
@sithhell sithhell merged commit 5b4e47d into master Dec 17, 2015
@sithhell sithhell deleted the remove_boost_config branch December 17, 2015 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants