On Thu, Jan 20, 2022 at 12:49 PM Ivan Zhakov <i...@visualsvn.com> wrote: > > On Thu, 20 Jan 2022 at 00:43, William A Rowe Jr <wr...@rowe-clan.net> wrote: >> >> On Wed, Jan 19, 2022 at 11:31 AM Yann Ylavic <ylavic....@gmail.com> wrote: >> > >> > I'm a bit surprised that _WIN32_WCE used CreateThread() while more >> > recent ones use _beginthreadex(), shouldn't it be the opposite? >> > CreateThread() aligns more with the usual/modern naming convention on >> > Windows.. >> >> Ignore naming conventions. We are a library that sits on top of >> MSVCRT/LIBCMT, and >> so the stdc lib we use needs to know a thread has started. >> CreateThread doesn't let >> the clib particpate and this causes specific problems for .dll loaded >> modules like our lib. > > That's true. > > But Microsoft finally fixed this in Universal CRT ( Visual Studio 2015). In > UCRT beginthreadex() is almost a wrapper around CreateThread(). So maybe it > makes sense to require VS 2015 for APR trunk and just use CreateThread?
This is my docker build env... you won't find any argument from me; https://github.com/envoyproxy/envoy-build-tools/blob/main/build_container/build_container_windows.ps1
