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

asyncrt_utils.h: include xlocale on macOS #1804

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

barracuda156
Copy link

Nothing seems to define HAVE_XLOCALE_H, at least on macOS, but including that header may be required for the build to work.

@barracuda156
Copy link
Author

Failure without the fix:

In file included from /opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/json.h:18,
                 from /opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/src/pch/stdafx.h:88:
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h: At global scope:
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:317:13: error: 'locale_t' does not name a type
  317 |     typedef locale_t xplat_locale;
      |             ^~~~~~~~
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:320:24: error: 'xplat_locale' does not name a type
  320 |     static _ASYNCRTIMP xplat_locale __cdecl c_locale();
      |                        ^~~~~~~~~~~~
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:327:5: error: 'locale_t' does not name a type
  327 |     locale_t m_prevLocale;
      |     ^~~~~~~~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant