Skip to content

Commit

Permalink
Fix Windows CI build (#1846)
Browse files Browse the repository at this point in the history
* Use powershell for windows CI build

* Remove unnecessary warning
  • Loading branch information
will-v-pi authored Aug 19, 2024
1 parent cd71c89 commit 8886f27
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,7 @@ jobs:
run: choco install .github/workflows/choco_packages.config

- name: Build Project
# bash required otherwise this mysteriously (no error) fails at "Generating cyw43_bus_pio_spi.pio.h"
shell: bash
shell: pwsh
run: |
mkdir build
cd build
Expand Down
2 changes: 1 addition & 1 deletion src/cmake/on_device.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ function(pico_add_dis_output TARGET)
add_custom_command(TARGET ${TARGET} POST_BUILD
COMMAND ${CMAKE_OBJDUMP} -h $<TARGET_FILE:${TARGET}> > ${output_path}$<IF:$<BOOL:$<TARGET_PROPERTY:${TARGET},OUTPUT_NAME>>,$<TARGET_PROPERTY:${TARGET},OUTPUT_NAME>,$<TARGET_PROPERTY:${TARGET},NAME>>.dis
COMMAND ${CMAKE_OBJDUMP} -d $<TARGET_FILE:${TARGET}> >> ${output_path}$<IF:$<BOOL:$<TARGET_PROPERTY:${TARGET},OUTPUT_NAME>>,$<TARGET_PROPERTY:${TARGET},OUTPUT_NAME>,$<TARGET_PROPERTY:${TARGET},NAME>>.dis
${EXTRA_COMMAND} || ${CMAKE_COMMAND} -E echo "WARNING: Disassembly is not correct"
${EXTRA_COMMAND}
VERBATIM
)
endfunction()
Expand Down

0 comments on commit 8886f27

Please sign in to comment.