Fix the ci break #7842

xiaoxiang781216 · 2022-12-10T03:14:50Z

Summary

risc-v/esp32c3: Move .gitignore from common/src to common
risc-v/esp32c3: Add ROMFS files into .gitignore #7839
ci: Exclude litex/arty_a7 from tools/ci/testlist/risc-v.dat
tools/ci: Update RISC-V toolchain in Dockerfile to latest version #7832
tools/cibuild.sh: Move the installation directory from prebuilt to tools
tools/cibuild.sh: Sync the installation between macOS and Ubuntu
workflow: Move ccache setting from build.yml to cibuild.sh
ci: Fix save and restore in GitHub cache action for MacOS build #7834

Impact

Skip litex/arty_a7 board
Fix ccache error

Testing

Pass CI

Signed-off-by: Xiang Xiao <[email protected]>

to workaround the new toolchain generate the wrong path for --print-libgcc-file-name and --print-file-name=libm.a: https://static.dev.sifive.com/dev-tools/freedom-tools/v2020.12/riscv64-unknown-elf-toolchain-10.2.0-2020.12.8-x86_64-linux-ubuntu14.tar.gz please reference here for more info: apache#7835 Signed-off-by: Xiang Xiao <[email protected]>

pkarashchenko · 2022-12-11T07:43:45Z

@xiaoxiang781216 should we ignore macOS build failures?

xiaoxiang781216 · 2022-12-11T08:17:53Z

@xiaoxiang781216 should we ignore macOS build failures?

I don't know why macOS build stuck, let' restart the build to try again.

xiaoxiang781216 · 2022-12-11T09:17:54Z

@pkarashchenko I found the root cause why macOS stuck, it should work now.

gustavonihei · 2022-12-11T09:25:42Z

Sorry for all the trouble my latest PRs may have caused.

~~But changing the prebuilt dir will bring back the #7636 issue.~~

~~The prebuilts should be isolated in their own out of tree dir, so that the GitHub action packs everything at the of the workflow.~~

I've just realized that the tools dir is not the one from the nuttx tree. Sorry for the mistake!

gustavonihei · 2022-12-11T10:38:59Z

nuttx/.github/workflows/build.yml

Line 201 in 61563d6

path: ./sources/prebuilt

We need to indicate to the GitHub Cache action the new path to be cached, which is now ./sources/tools.

sync with tools/ci/docker/linux/Dockerfile Signed-off-by: Xiang Xiao <[email protected]>

Signed-off-by: Xiang Xiao <[email protected]>

xiaoxiang781216 · 2022-12-11T10:52:32Z

nuttx/.github/workflows/build.yml

Line 201 in 61563d6

path: ./sources/prebuilt

We need to indicate to the GitHub Cache action the new path to be cached, which is now ./sources/tools.

@gustavonihei done.

pkarashchenko · 2022-12-11T11:29:53Z

AVR build fails

xiaoxiang781216 · 2022-12-11T11:49:28Z

avr doesn't change, I can't understand why avr fail.

pkarashchenko · 2022-12-11T12:00:38Z

I have Macbook and will try some steps later today

to avoid "avr-objcopy: Bad CPU type in executable" on macOS Signed-off-by: Xiang Xiao <[email protected]>

xiaoxiang781216 · 2022-12-11T12:27:31Z

@pkarashchenko I remove hex generation from build, let's see what happen with the new change.

pkarashchenko · 2022-12-11T14:12:21Z

@xiaoxiang781216 please fill an issue for hex generation for AVR, so it is not forgotten

pkarashchenko · 2022-12-11T14:15:17Z

@xiaoxiang781216 somehow I can't merge this PR, because when I press merge button, I'm getting some OAuth error

xiaoxiang781216 · 2022-12-11T14:41:43Z

It's strange, I can merge it.

pkarashchenko · 2022-12-11T23:42:15Z

@xiaoxiang781216 I just checked with another PR. This seems to be a mobile app issue since I can merge from a laptop, but not from the GitHub Android app.

xiaoxiang781216 · 2022-12-12T09:36:19Z

@xiaoxiang781216 please fill an issue for hex generation for AVR, so it is not forgotten

Here: #7851

xiaoxiang781216 requested a review from gustavonihei December 10, 2022 03:14

xiaoxiang781216 force-pushed the riscv branch 2 times, most recently from cd9756e to 52c2b70 Compare December 10, 2022 12:01

xiaoxiang781216 changed the title ~~ci: Exclude litex/arty_a7 from tools/ci/testlist/risc-v.dat~~ Fix the ci break Dec 10, 2022

xiaoxiang781216 added 2 commits December 10, 2022 22:42

risc-v/esp32c3: Move .gitignore from common/src to common

e43f4b2

Signed-off-by: Xiang Xiao <[email protected]>

xiaoxiang781216 force-pushed the riscv branch 2 times, most recently from 2d09c37 to c093383 Compare December 10, 2022 16:27

xiaoxiang781216 requested a review from pkarashchenko December 10, 2022 16:27

xiaoxiang781216 mentioned this pull request Dec 10, 2022

Improvements in TCP connections allocation. #7525

Merged

pkarashchenko approved these changes Dec 10, 2022

View reviewed changes

xiaoxiang781216 force-pushed the riscv branch from c093383 to 4e1b442 Compare December 11, 2022 08:15

xiaoxiang781216 force-pushed the riscv branch from 4e1b442 to d821589 Compare December 11, 2022 09:16

xiaoxiang781216 added 3 commits December 11, 2022 18:45

tools/cibuild.sh: Move the installation directory from prebuilt to tools

18912a3

sync with tools/ci/docker/linux/Dockerfile Signed-off-by: Xiang Xiao <[email protected]>

workflow: Move ccache setting from build.yml to cibuild.sh

1f447e1

Signed-off-by: Xiang Xiao <[email protected]>

tools/cibuild.sh: Sync the installation between macOS and Ubuntu

fa21698

Signed-off-by: Xiang Xiao <[email protected]>

xiaoxiang781216 force-pushed the riscv branch from d821589 to fa21698 Compare December 11, 2022 10:52

arduino-mega2560/nsh: Remove CONFIG_INTELHEX_BINARY

35ff676

to avoid "avr-objcopy: Bad CPU type in executable" on macOS Signed-off-by: Xiang Xiao <[email protected]>

pkarashchenko approved these changes Dec 11, 2022

View reviewed changes

xiaoxiang781216 merged commit 4a84555 into apache:master Dec 11, 2022

xiaoxiang781216 deleted the riscv branch December 11, 2022 14:41

xiaoxiang781216 mentioned this pull request Dec 12, 2022

Fix "avr-objcopy: Bad CPU type in executable" on macOS #7851

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix the ci break #7842

Fix the ci break #7842

xiaoxiang781216 commented Dec 10, 2022 •

edited

Loading

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

gustavonihei commented Dec 11, 2022 •

edited

Loading

gustavonihei commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022 •

edited

Loading

xiaoxiang781216 commented Dec 12, 2022

Fix the ci break #7842

Fix the ci break #7842

Conversation

xiaoxiang781216 commented Dec 10, 2022 • edited Loading

Summary

Impact

Testing

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

gustavonihei commented Dec 11, 2022 • edited Loading

gustavonihei commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022

xiaoxiang781216 commented Dec 11, 2022

pkarashchenko commented Dec 11, 2022 • edited Loading

xiaoxiang781216 commented Dec 12, 2022

xiaoxiang781216 commented Dec 10, 2022 •

edited

Loading

gustavonihei commented Dec 11, 2022 •

edited

Loading

pkarashchenko commented Dec 11, 2022 •

edited

Loading