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

Fixes for legacy Windows versions #203

Merged
merged 11 commits into from
Mar 30, 2020

Conversation

Mixaill
Copy link
Contributor

@Mixaill Mixaill commented Mar 28, 2020

  • cmake: fix CMAKE_SYSTEM_VERSION comparison
  • cmake: add support for 5.2 version (2003 / 2003 R2 / XP 64-bit)
  • cmake: disable crashpad build with XP toolchain
  • cmake: link with pathcch.lib only for Windows 8.0+

  • src/path/windows: include <pathcch.h> only for Windows 8.0+
  • src/transports/winhttp: use WINHTTP_ACCESS_TYPE_AUTOMATIC_PROXY only for Windows 8.1+
  • src/transports/winhttp: fallback to WINHTTP_ACCESS_TYPE_DEFAULT_PROXY instead of WINHTTP_ACCESS_TYPE_NO_PROXY
  • src/sync: implement InitOnceExecuteOnce for Windows XP
  • src/unwinder/dbghelp: do not useCaptureStackBackTrace on Windows XP

  • readme: add info about CMAKE_SYSTEM_VERSION into readme

CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
src/sentry_sync.h Outdated Show resolved Hide resolved
src/unwinder/sentry_unwinder_dbghelp.c Outdated Show resolved Hide resolved
@Mixaill Mixaill force-pushed the win-system-version-2 branch 2 times, most recently from ed918a2 to db443d1 Compare March 28, 2020 23:06
@Mixaill
Copy link
Contributor Author

Mixaill commented Mar 29, 2020

@Swatinem now it pass all the unit tests on Windows XP SP3

@Swatinem Swatinem merged commit d9375d3 into getsentry:master Mar 30, 2020
@Mixaill Mixaill deleted the win-system-version-2 branch June 11, 2020 10:24
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.

2 participants