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

Windows builds with mingw-gcc on Linux get the .exe extension twice (godot.windows.editor.x86_64.exe.exe) #98967

Closed
akien-mga opened this issue Nov 8, 2024 · 3 comments · Fixed by #98968

Comments

@akien-mga
Copy link
Member

akien-mga commented Nov 8, 2024

Tested versions

  • Reproducible in 4.4-dev (36e6207)
  • Not reproducible in 4.3-dev3 or earlier

System information

Fedora Linux 41 (KDE Plasma) on Wayland - X11 display driver, Multi-window, 1 monitor - Vulkan (Forward+) - dedicated AMD Radeon RX 7600M XT (RADV NAVI33) - AMD Ryzen 7 7840HS w/ Radeon 780M Graphics (16 threads)

Issue description

Building for Windows from Linux with mingw-gcc seems to have regressed as it now doubles the .exe extension on the produced binaries.

scons p=windows

gives:

$ ls bin -l
total 123360
-rwxr-xr-x. 1 akien akien    188928 Nov  8 16:26 godot.windows.editor.x86_64.console.exe.exe
-rwxr-xr-x. 1 akien akien 126126080 Nov  8 16:26 godot.windows.editor.x86_64.exe.exe

Steps to reproduce

On Linux with mingw-gcc:

scons p=windows

Minimal reproduction project (MRP)

n/a

@akien-mga
Copy link
Member Author

Regression from #98105, CC @Repiteo @bruvzg.

But more than a bug in this PR, I think the PR fixed a bug and made an old hack redundant:

https://github.com/godotengine/godot/blame/master/platform/windows/detect.py#L722-L723

We can likely just remove those two lines.

akien-mga added a commit to akien-mga/godot that referenced this issue Nov 8, 2024
This old hack is no longer needed and now wrong after godotengine#98105.

Fixes godotengine#98967.
akien-mga added a commit to akien-mga/godot that referenced this issue Nov 8, 2024
This old hack is no longer needed and now wrong after godotengine#98105.

Fixes godotengine#98967.
@hpvb hpvb closed this as completed in 6d32654 Nov 8, 2024
@bruvzg
Copy link
Member

bruvzg commented Nov 8, 2024

I can confirm the same on macOS as well (have not noticed it when testing #98968, since I was not running executables built on Mac).

@bruvzg
Copy link
Member

bruvzg commented Nov 8, 2024

I can confirm the same on macOS

#98968 do fix it.

ChrisBase pushed a commit to ChrisBase/godot that referenced this issue Nov 15, 2024
vgezer pushed a commit to vgezer/godot that referenced this issue Nov 29, 2024
jss2a98aj pushed a commit to jss2a98aj/blazium that referenced this issue Dec 7, 2024
jss2a98aj pushed a commit to jss2a98aj/blazium that referenced this issue Dec 7, 2024
jss2a98aj pushed a commit to jss2a98aj/blazium that referenced this issue Dec 7, 2024
jss2a98aj pushed a commit to jss2a98aj/blazium that referenced this issue Dec 7, 2024
Ughuuu pushed a commit to Ughuuu/godot that referenced this issue Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Immediate Blocker
Development

Successfully merging a pull request may close this issue.

2 participants