Skip to content

Conversation

@Alexey31rus
Copy link

Because on the latest build tools, the compilation of the engine and/or project may fail due to errors. Therefore, we use the recommended version of the build tools for UE 5.4 and 5.5

@Alexey31rus Alexey31rus force-pushed the suggested-msvc-version branch from 106b923 to 1189ccb Compare April 10, 2025 08:56
@slonopotamus
Copy link
Collaborator

slonopotamus commented Apr 30, 2025

We can use this as a quick hack. For a more long-term solution, I'd like to use Windows_SDK.json from Unreal.

See https://discord.com/channels/844118704941170688/844121719475535913/1310942558502912001 and previous messages in that thread.

I currently have a semi-finished branch that implements this approach, but not ready for PR yet.

$windows_os_version = "Windows10SDK"

# NOTE: See suggested components https://github.com/EpicGames/UnrealEngine/blob/release/Engine/Config/Windows/Windows_SDK.json
if ($unreal_engine_version.StartsWith("5.4"))
Copy link
Collaborator

Choose a reason for hiding this comment

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

$windows_os_version is not set in this branch

Copy link
Author

Choose a reason for hiding this comment

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

For this branch will be used value from line 68. UE 5.4 still uses Win10SDK. Or are you writing that it is better to explicitly specify value of variable in this branch?

$vs_args = $vs_args,
"--add", "Microsoft.VisualStudio.Component.VC.$vs_tools_version.x86.x64",
"--remove", "Microsoft.VisualStudio.Component.VC.Tools.x86.x64",
"--remove", "Microsoft.VisualStudio.Component.Windows10SDK.20348"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why we need --remove? There is no VS installation yet when our script is executed -> there is nothing to remove.

Copy link
Author

Choose a reason for hiding this comment

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

Hmm, yes, indeed this component does not appear by default anymore

@Alexey31rus Alexey31rus force-pushed the suggested-msvc-version branch 2 times, most recently from f47e3b7 to dfd76ec Compare May 12, 2025 13:18
@slonopotamus
Copy link
Collaborator

Fails on CI because UE version is undefined when we're building ue4-docker build --target=prerequisites. I suggest putting this PR on hold while we're implementing Windows_SDK.json-based approach.

@Alexey31rus Alexey31rus force-pushed the suggested-msvc-version branch from dfd76ec to d2d08e0 Compare June 4, 2025 07:42
@Alexey31rus Alexey31rus force-pushed the suggested-msvc-version branch from a934bc4 to fb01e83 Compare June 4, 2025 12:52
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