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

Allow to run non-Development Editor + few minor fixes in the same code #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sleeptightAnsiC
Copy link
Contributor

@sleeptightAnsiC sleeptightAnsiC commented Mar 15, 2024

Hi @adamrehn ,

I fixed the suport for launching non-Development Editor #56 There are also few other changes and fixes added by this PR, as they were either easy to fix, located in the same code or blocking me from fixing the mentioned issue. I tested it mostly on Linux but it should work™ on other platforms too since the underlying code was already there.

I also tried to deal with #61 and #62 but they don't seem worth fixing right now (see the comments under those issues). Though, some changes were made with those in mind.

Let me know what do you think. PR is open for your edits.

Cheers!

All changes:

  • API now allow to run non-Development Editor
  • You can do stuff like ue4 run DebugGame and ue4 editor DebugGame as both these commands now take [CONFIGURATION] as optional parameter
  • Fix: "ue4 run DebugGame" runs default Development executable instead of DebugGame #56
  • add validBuildTargets helper function that behaves similar as validBuildConfigurations
  • Fix: add validation for build target inside of getBuildScript as this was not handled anywhere unlike build configuration
  • Fix: validBuildTargets/Configurations now take into consideration if UE is installed via EpicLauncher or not (now the behavior lines up with the tables on this page)
  • allow to build Game target inside of getBuildScript as this was fairly easy to add was surprisingly not supported
  • added small FIXME note that we are missing support for 'Program' build target

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.

"ue4 run DebugGame" runs default Development executable instead of DebugGame
1 participant