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

tools/config.hpp fixes #89

Closed

smr99 opened this issue Sep 26, 2017 · 1 comment

smr99 commented Sep 26, 2017 •

edited

Hi,

Two items.

Commit cc8e300 commented out the following define in tools/config.hpp:

#if (defined(__CYGWIN__) || defined(__FreeBSD__) || defined(__NetBSD__) \
   || (defined(__hppa) && !defined(__OpenBSD__)) || (defined(__NO_LONG_DOUBLE_MATH) && (DBL_MANT_DIG != LDBL_MANT_DIG))) \
   && !defined(BOOST_MATH_NO_LONG_DOUBLE_MATH_FUNCTIONS)
//#  define BOOST_MATH_NO_LONG_DOUBLE_MATH_FUNCTIONS
#endif

To me, this appears to be an error.

Debian linux for hppa has been shipping with a patch for this condition. Specifically to exclude linux in this situation:

#if (defined(__CYGWIN__) || defined(__FreeBSD__) || defined(__NetBSD__) \
  || (defined(__hppa) && !defined(__OpenBSD__) && !defined(__linux__)) || (defined(__NO_LONG_DOUBLE_MATH) && (DBL_MANT_DIG != LDBL_MANT_DIG))) \
  && !defined(BOOST_MATH_NO_LONG_DOUBLE_MATH_FUNCTIONS)

...

The text was updated successfully, but these errors were encountered:

Collaborator

jzmaddock commented Apr 22, 2018

It was a mistake for sure, fixed in develop.

Thanks for the report!

jzmaddock closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment