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

Boost: Deprecated Math Header #2707

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Sep 25, 2018

Deprecation warning in Boost 1.67+

include/boost/math/common_factor_rt.hpp:13:93:
  note: #pragma message: This header is deprecated.
  Use <boost/integer/common_factor_rt.hpp> instead.

BOOST_HEADER_DEPRECATED("<boost/integer/common_factor_rt.hpp>");

https://github.com/boostorg/math/blob/boost-1.67.0/include/boost/math/common_factor_rt.hpp

Deprecation warning in Boost 1.67+

```
include/boost/math/common_factor_rt.hpp:13:93:
  note: #pragma message: This header is deprecated.
  Use <boost/integer/common_factor_rt.hpp> instead.

BOOST_HEADER_DEPRECATED("<boost/integer/common_factor_rt.hpp>");
```

https://github.com/boostorg/math/blob/boost-1.67.0/include/boost/math/common_factor_rt.hpp
@ax3l ax3l added warning code produces/produced a warning component: third party third party libraries that are shipped and/or linked labels Sep 25, 2018
@ax3l ax3l requested a review from psychocoderHPC September 25, 2018 21:20
@ax3l ax3l mentioned this pull request Sep 25, 2018
8 tasks
@psychocoderHPC psychocoderHPC merged commit 3a2c2e5 into ComputationalRadiationPhysics:dev Sep 26, 2018
@ax3l ax3l deleted the fix-deprecatedBoostHeader branch October 2, 2018 15:02
psychocoderHPC added a commit to psychocoderHPC/picongpu that referenced this pull request Oct 4, 2018
Boost 1.68.0 is not allowed since ComputationalRadiationPhysics#2685 but after we will integrate the next alapak release we will end in compile issues.
The reason is ComputationalRadiationPhysics#2707. The definition of `boost::math::gcd` is not available if `boost/integer/common_factor_rt.hpp` is included.

This PR fixes the behaviour be using always `boost/integer/common_factor_rt.hpp` and use `boost::integer::gcd` within PMacc.
@psychocoderHPC psychocoderHPC mentioned this pull request Oct 4, 2018
2 tasks
ax3l pushed a commit to ax3l/picongpu that referenced this pull request Oct 5, 2018
Boost 1.68.0 is not allowed since ComputationalRadiationPhysics#2685 but after we will integrate the next alapak release we will end in compile issues.
The reason is ComputationalRadiationPhysics#2707. The definition of `boost::math::gcd` is not available if `boost/integer/common_factor_rt.hpp` is included.

This PR fixes the behaviour be using always `boost/integer/common_factor_rt.hpp` and use `boost::integer::gcd` within PMacc.
steindev pushed a commit to steindev/picongpu that referenced this pull request Oct 16, 2018
Boost 1.68.0 is not allowed since ComputationalRadiationPhysics#2685 but after we will integrate the next alapak release we will end in compile issues.
The reason is ComputationalRadiationPhysics#2707. The definition of `boost::math::gcd` is not available if `boost/integer/common_factor_rt.hpp` is included.

This PR fixes the behaviour be using always `boost/integer/common_factor_rt.hpp` and use `boost::integer::gcd` within PMacc.
steindev pushed a commit to steindev/picongpu that referenced this pull request Oct 16, 2018
Boost 1.68.0 is not allowed since ComputationalRadiationPhysics#2685 but after we will integrate the next alapak release we will end in compile issues.
The reason is ComputationalRadiationPhysics#2707. The definition of `boost::math::gcd` is not available if `boost/integer/common_factor_rt.hpp` is included.

This PR fixes the behaviour be using always `boost/integer/common_factor_rt.hpp` and use `boost::integer::gcd` within PMacc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: third party third party libraries that are shipped and/or linked warning code produces/produced a warning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants