Skip to content

Fixing windows SFU platform scripts.#618

Merged
mcottontensor merged 1 commit intoEpicGamesExt:masterfrom
mcottontensor:win_sfu_script_fix
May 19, 2025
Merged

Fixing windows SFU platform scripts.#618
mcottontensor merged 1 commit intoEpicGamesExt:masterfrom
mcottontensor:win_sfu_script_fix

Conversation

@mcottontensor
Copy link
Copy Markdown
Collaborator

Relevant components:

  • Signalling server
  • Common library
  • Frontend library
  • Frontend UI library
  • Matchmaker
  • Platform scripts
  • SFU

Problem statement:

Running the SFU platform scripts on windows results in an error.

Solution

There is a environment variable used in the platform scripts that is never setup. This leads to the script referring to a location that doesn't exist. The variable is not actually needed so removing it fixes the issue.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 15, 2025

⚠️ No Changeset found

Latest commit: 2453a63

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mcottontensor mcottontensor added auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5 auto-backport-to-UE5.6 labels May 15, 2025
@mcottontensor mcottontensor merged commit fd90d53 into EpicGamesExt:master May 19, 2025
4 checks passed
@mcottontensor mcottontensor deleted the win_sfu_script_fix branch May 19, 2025 03:16
github-actions bot pushed a commit that referenced this pull request May 19, 2025
github-actions bot pushed a commit that referenced this pull request May 19, 2025
@github-actions
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
UE5.5
UE5.6

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

mcottontensor added a commit that referenced this pull request May 19, 2025
(cherry picked from commit fd90d53)

Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com>
mcottontensor added a commit that referenced this pull request May 19, 2025
(cherry picked from commit fd90d53)

Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Used to specify we want a PR to auto backport to a branch, must be paired with auto-backport-to-UEX. auto-backport-to-UE5.5 auto-backport-to-UE5.6

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant