Skip to content

Conversation

@acassis
Copy link
Contributor

@acassis acassis commented Dec 22, 2023

Summary

Update defconfig after running refresh.sh

Impact

Only fvp-armv8r

Testing

N/A

@xiaoxiang781216
Copy link
Contributor

it's strange that ci doesn't catch this problem.

Copy link
Contributor

@jerpelea jerpelea left a comment

Choose a reason for hiding this comment

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

can you please add a commit message

# CONFIG_NSH_ARGCAT is not set
# CONFIG_NSH_CMDOPT_HEXDUMP is not set
CONFIG_ARCH="arm"
CONFIG_ARCH_BOARD="stm32f4discovery"
Copy link
Member

Choose a reason for hiding this comment

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

this doesn't look good :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

wow! Good finding!

I just run ./tools/refresh.sh and it created these modifications!

@acassis
Copy link
Contributor Author

acassis commented Dec 22, 2023

it's strange that ci doesn't catch this problem.

Could you please run:

./tools/refresh.sh --silent --defaults all

And see if same issue file will be updated?

@acassis
Copy link
Contributor Author

acassis commented Dec 22, 2023

I will close this PR, but first, please guys, run the same build script to confirm it is failing. I don't remember if I ran make distclean before running it, probably it was configured to stm32f4discovery.

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.

4 participants