Skip to content

Update multi gcc #689

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

Merged
merged 4 commits into from
Jul 29, 2025
Merged

Update multi gcc #689

merged 4 commits into from
Jul 29, 2025

Conversation

kilograham
Copy link
Contributor

No description provided.

@kilograham kilograham merged commit 81eac95 into develop Jul 29, 2025
2 of 4 checks passed
@kilograham kilograham deleted the update-multi-gcc branch July 29, 2025 03:15
shell: bash
run: cd ${{github.workspace}}/pico-examples; mkdir -p build; rm -rf build/*; cd build; PICO_SDK_PATH=../../pico-sdk cmake ../ -DCMAKE_BUILD_TYPE=Debug -DPICO_TOOLCHAIN_PATH=/opt/arm/arm-gnu-toolchain-11.3.rel1-x86_64-arm-none-eabi -DPICO_BOARD=pico_w; make --output-sync=target --no-builtin-rules --no-builtin-variables -j$(nproc)

- name: GCC 11.3.1 Release
if: always()
if: ${{ !cancelled() }}
shell: bash
run: cd ${{github.workspace}}/pico-examples; mkdir -p build; rm -rf build/*; cd build; PICO_SDK_PATH=../../pico-sdk cmake ../ -DCMAKE_BUILD_TYPE=Release -DPICO_TOOLCHAIN_PATH=/opt/arm/arm-gnu-toolchain-11.3.rel1-x86_64-arm-none-eabi -DPICO_BOARD=pico_w; make --output-sync=target --no-builtin-rules --no-builtin-variables -j$(nproc)

- name: GCC 12.2.1 Debug
Copy link
Contributor

Choose a reason for hiding this comment

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

For consistency, it might be nice if all these older compiler-builds were also labelled with "... Pico W", just like GCC 13 and above are? e.g.

    - name: GCC 12.2.1 Debug Pico W

(because at first I thought "Why are only the new compilers building for Pico W?", before I spotted that the old compilers are also doing -DPICO_BOARD=pico_w 😉 )

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