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

Crash on launch on the mac #89842

Closed
fire opened this issue Mar 24, 2024 · 1 comment · Fixed by #89855
Closed

Crash on launch on the mac #89842

fire opened this issue Mar 24, 2024 · 1 comment · Fixed by #89855

Comments

@fire
Copy link
Member

fire commented Mar 24, 2024

Tested versions

  • Reproducible in master 99ff024
  • Not in 4.3 dev4

System information

Mac Mini, Apple Silicon

Issue description

% git bisect bad                   
023dcd44c1e628bb654b5472418d6a346b510a71 is the first bad commit
commit 023dcd44c1e628bb654b5472418d6a346b510a71
Author: Rémi Verschelde <[email protected]>
Date:   Wed Mar 6 18:50:35 2024 +0100

    Refactor OS exit code to be `EXIT_SUCCESS` by default
    
    - `Main::setup` early exits (failure or `--help`/`--version`) now
      consistently return `EXIT_FAILURE` or `EXIT_SUCCESS` on all platforms,
      instead of 255 on some and a Godot Error code on others.
    - `Main::start` now returns the exit code, simplifying the handling of early
      failures.
    - `Main::iteration` needs to explicit set the exit code in OS if it errors
      out.
    - Web and iOS now properly return `OS::get_exit_code()` instead of 0.

 core/os/os.h                         |  3 +-
 main/main.cpp                        | 96 ++++++++++++++++--------------------
 main/main.h                          |  2 +-
 platform/ios/godot_ios.mm            | 11 +++--
 platform/linuxbsd/godot_linuxbsd.cpp | 13 ++---
 platform/macos/godot_main_macos.mm   | 19 ++++---
 platform/web/web_main.cpp            | 16 +++---
 platform/windows/godot_windows.cpp   |  8 +--
 8 files changed, 82 insertions(+), 86 deletions(-)

Godot Engine is crashing on launch on the mac. There is no crash log.

Steps to reproduce

Start godot and watch it crash.

I used scons scu_build=yes compiledb=yes

Minimal reproduction project (MRP)

N/A

@Alex2782
Copy link
Contributor

Alex2782 commented Mar 24, 2024

Main::start() -> return EXIT_SUCCESS; and not more return true;

#define EXIT_SUCCESS 0

Fix:

	if (ret == EXIT_SUCCESS) {
		os.run();
	} else {
		os.set_exit_code(EXIT_FAILURE);
	}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants