Skip to content

Conversation

@bradsaracik
Copy link

@bradsaracik bradsaracik commented Sep 30, 2025

This is adding to #386 from older work that previously enabled me to build a 4.27.2 image on ubuntu 24 (ue4-docker build 4.27.2 -basetag=ubuntu24.04 --cuda=12.8.1 ...)

I'm not entirely sure the global pip commit won't break earlier ubuntu. we can add gates like the libasound commit

confirmed this HEAD with minimal build with 5.4 via
ue4-docker build --ue-version custom:5.4.4 -repo https://github.com/EpicGames/UnrealEngine.git -username $UE4DOCKER_USERNAME -password $UE4DOCKER_PASSWORD -branch 5.4.4-release --target minimal -basetag ubuntu24.04 --cuda=12.8.1 --exclude templates

@bradsaracik bradsaracik marked this pull request as ready for review October 2, 2025 23:59
@bradsaracik
Copy link
Author

bradsaracik commented Oct 3, 2025

it appears #370 created an issue for full builds?
ue4BuildArgs is no longer set in build minimal, but still added in build full. do you want local fix to rename the hanging reference minimalArgs added to this PR or on its own?

https://github.com/adamrehn/ue4-docker/pull/370/files#diff-fb6fdc2d3cafb59e282a6ebae5c62c05285e88305399a9d2bdd5f596de187cdbL479

https://github.com/adamrehn/ue4-docker/pull/370/files#diff-fb6fdc2d3cafb59e282a6ebae5c62c05285e88305399a9d2bdd5f596de187cdbL526

@bradsaracik
Copy link
Author

bradsaracik commented Oct 3, 2025

similarly to prev comment, removal of linker-fixup.py in #370 but left hanging in linux export_installed.py
https://github.com/adamrehn/ue4-docker/blob/master/src/ue4docker/exports/export_installed.py#L78

deleted file in pr

@bradsaracik
Copy link
Author

similarly to prev comment, removal of linker-fixup.py in #370 but left hanging in linux export_installed.py https://github.com/adamrehn/ue4-docker/blob/master/src/ue4docker/exports/export_installed.py#L78

deleted file in pr

resolved ddbb360

@russkel
Copy link
Contributor

russkel commented Nov 13, 2025

For your ref: #378

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