Skip to content

Commit

Permalink
Fix export builds (#486)
Browse files Browse the repository at this point in the history
* Fix linux debug builds

Amusingly enough godot doesn't actually build the C# project when
exporting the build - we have to do that ourselves.

This required making all the `.csproj` files have the same .net target
version and then just adding the `dotnet build` command. This will also
probably fix the other platforms, but I can only easily test linux.

This also explains why I didn't encounter the issue when running locally
- my command when running locally is `dotnet build && godot
  --rendering-driver opengl3 # run game`, so I'm always rebuilding.

* Add the build command to the other platforms
  • Loading branch information
TomWerner authored Jan 4, 2025
1 parent 18d9116 commit b3821b1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/export-c7.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ jobs:
run: |
mkdir -v -p build/${FOLDER_NAME}
cd ${EXPORT_NAME}
dotnet build
godot -v --headless --export${BUILD_SUFFIX} "Windows Desktop" ../build/${FOLDER_NAME}/${EXPORT_NAME}.exe
- name: Copy Static Files
run: |
Expand Down Expand Up @@ -66,6 +67,7 @@ jobs:
run: |
mkdir -v -p build/${FOLDER_NAME}
cd ${EXPORT_NAME}
dotnet build
godot -v --headless --export${BUILD_SUFFIX} "Linux/X11" ../build/${FOLDER_NAME}/${EXPORT_NAME}.x86_64
- name: Copy Static Files
run: |
Expand Down Expand Up @@ -98,6 +100,7 @@ jobs:
run: |
mkdir -v -p build/${FOLDER_NAME}
cd ${EXPORT_NAME}
dotnet build
godot -v --headless --export${BUILD_SUFFIX} "macOS" ../build/${FOLDER_NAME}.zip
- name: Copy Static Files
run: |
Expand Down
2 changes: 1 addition & 1 deletion _Console/BuildDevSave/BuildDevSave.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

<PropertyGroup>
<OutputType>Exe</OutputType>
<TargetFramework>net8.0</TargetFramework>
<TargetFramework>net7.0</TargetFramework>
</PropertyGroup>

</Project>

0 comments on commit b3821b1

Please sign in to comment.