Skip to content

Feature/build fix#71

Merged
binji merged 7 commits into
binji:mainfrom
SelvinPL:feature/build_fix
May 8, 2026
Merged

Feature/build fix#71
binji merged 7 commits into
binji:mainfrom
SelvinPL:feature/build_fix

Conversation

@SelvinPL

@SelvinPL SelvinPL commented May 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@SelvinPL

SelvinPL commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Fixes #72

build-rgbds-live fails because my bad commit last time

@SelvinPL SelvinPL force-pushed the feature/build_fix branch 2 times, most recently from 00bf48e to 214bf92 Compare May 7, 2026 21:13
@SelvinPL

SelvinPL commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

explenation of force pushes:
first: obviously i did from wrong branch
second: i've copied from new rgbds-live build and frogot to remove "

- main
pull_request:

env:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copy from rgbds-live instead installing with

    mkdir $HOME/emsdk
    git clone --depth 1 https://github.com/emscripten-core/emsdk.git $HOME/emsdk
    $HOME/emsdk/emsdk update-tags
    $HOME/emsdk/emsdk install tot
    $HOME/emsdk/emsdk activate tot

we can use mymindstorm/setup-emsdk@v14 ... it will setup the path ans stuff

Comment thread .github/workflows/build.yml Outdated

- name: cmake
run: cmake -S . -B out -G Ninja -DCMAKE_TOOLCHAIN_FILE=$HOME/emsdk/upstream/emscripten/cmake/Modules/Platform/Emscripten.cmake -DCMAKE_BUILD_TYPE=Release -DWASM=true
run: emcmake cmake -S . -B out -G Ninja -DCMAKE_BUILD_TYPE=Release -DWASM=true

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using emcmake will add toolchain

Comment thread .github/workflows/build.yml Outdated
run: |
source $HOME/emsdk/emsdk_env.sh
cmake --build out
run: emmake cmake --build out

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with mymindstorm/setup-emsdk@v14 there is no need for "source ..."

Comment thread .github/workflows/build.yml Outdated

- name: update path
run: echo "PATH=$PATH:$HOME/emsdk" >> $GITHUB_ENV
build-rgbds-live:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it just a copy of wasm build but with rgbds-live specific flags

Comment thread scripts/build_tests.py
Run('nmake', cwd=WLA_DX_BUILD_DIR)
else:
Run('cmake', WLA_DX_DIR, cwd=WLA_DX_BUILD_DIR)
Run('cmake', '-DCMAKE_POLICY_VERSION_MINIMUM=3.5', '-DCMAKE_C_FLAGS=-Wno-strict-prototypes', WLA_DX_DIR, cwd=WLA_DX_BUILD_DIR)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WLA_DX is using cmake minimum veriosn 2.8 or somethig ... also without no-strict-prototypes build fails

@SelvinPL SelvinPL mentioned this pull request May 8, 2026
@SelvinPL SelvinPL force-pushed the feature/build_fix branch from c00106c to 599e83e Compare May 8, 2026 15:48
@SelvinPL

SelvinPL commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

last force push is after rebasing to new main commits

@SelvinPL

SelvinPL commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

and now ...it passes all builds finally

@binji binji left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks for doing this!

@binji binji merged commit b5f718e into binji:main May 8, 2026
6 checks passed
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