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

Update Windows Fleet for December 2023 Patch Tuesday #35640

Merged
merged 31 commits into from
Dec 18, 2023

Conversation

BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented Dec 13, 2023

Follow on from #35116 now that the compiler bug that blocked that appears fixed.

Full Windows rebuild: https://dev.azure.com/vcpkg/public/_build/results?buildId=97631
Full UWP rebuild: https://dev.azure.com/vcpkg/public/_build/results?buildId=97657

Included is a workaround for https://developercommunity.visualstudio.com/t/Missing-threadsh-in-MSVC-178/10514752 implemented as upstream's commit ebdf9d724cfd0a04f194ecfcb678b702ad062be2
…ber-2023

# Conflicts:
#	ports/baresip-libre/portfile.cmake
#	ports/baresip-libre/vcpkg.json
#	ports/tinyorm/portfile.cmake
#	scripts/azure-pipelines/azure-pipelines.yml
#	versions/b-/baresip-libre.json
#	versions/baseline.json
#	versions/j-/jkqtplotter.json
#	versions/t-/tinyorm.json
@BillyONeal BillyONeal changed the title Update Windows Fleet for December Patch Tuesday Update Windows Fleet for December 2023 Patch Tuesday Dec 13, 2023
@BillyONeal BillyONeal added info:internal This PR or Issue was filed by the vcpkg team. category:infrastructure Pertaining to the CI/Testing infrastrucutre labels Dec 13, 2023
@FrankXie05 FrankXie05 self-assigned this Dec 13, 2023
@BillyONeal
Copy link
Member Author

The qtwebengine failure is #35260 which should be fixed by #35639

@BillyONeal
Copy link
Member Author

The qtwebengine failure is #35260 which should be fixed by #35639

I merged these and submitted https://dev.azure.com/vcpkg/public/_build/results?buildId=97668&view=results only retesting x64-windows since the other triplets were clean without it

@Neumann-A
Copy link
Contributor

Neumann-A commented Dec 15, 2023

Strange since my logs show:

[14265/39164] CXX obj/third_party/perfetto/src/tracing/core/service/trace_buffer.obj
[14266/39164] CXX obj/third_party/webrtc/api/audio_options_api/audio_options.obj
[14267/39164] CXX obj/third_party/perfetto/src/tracing/core/core/shared_memory_abi.obj

with configure log:

-- The C compiler identification is MSVC 19.38.33133.0
-- The CXX compiler identification is MSVC 19.38.33133.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.38.33130/bin/Hostx64/x64/cl.exe - skipped

vs

-- The C compiler identification is MSVC 19.38.33133.0
-- The CXX compiler identification is MSVC 19.38.33133.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.38.33130/bin/Hostx64/x64/cl.exe - skipped

but I only tested with Release builds. One other difference I see is:
Geolocation ............................ no but that is probably unrelated.

Maybe I diffed the wrong trees. Let me check that didn't diff the wrong tree

@Neumann-A
Copy link
Contributor

Fun fact: It only fails for Debug builds o.O. I mean I can start comparing compiler flags/preprocessed sources but somehow I would say that indicates the compiler is broken? (considering that clang-cl just compiles the code fine) I mean libtorch is also broken by the release with an ICE.
Who failed to test the compiler release against vcpkg? I can only hope you have internally something like bugpoint which could simplify preprocessed sources to generate a reduced test case.

@Neumann-A
Copy link
Contributor

Difference in compiler flags according to the ninja file:
image

@Neumann-A
Copy link
Contributor

@BillyONeal: Try cleaning up the patch and removing std::remove_cvref_t

Comment on lines -77 to +76
$packagesRoot = Join-Path $WorkingRoot 'packages'
$packagesRoot = Join-Path $WorkingRoot 'p'
Copy link
Contributor

@Neumann-A Neumann-A Dec 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fun fact I already had a case where I had to do this ;)

@Neumann-A
Copy link
Contributor

@BillyONeal
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@BillyONeal
Copy link
Member Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@BillyONeal BillyONeal merged commit ab00a63 into microsoft:master Dec 18, 2023
15 checks passed
@BillyONeal BillyONeal deleted the patch-tuesday-december-2023 branch December 18, 2023 18:27
Osyotr pushed a commit to Osyotr/vcpkg that referenced this pull request Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:infrastructure Pertaining to the CI/Testing infrastrucutre info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants