Tools refine

[xiaoxiang781216]

No description provided.

[davids5]

@xiaoxiang781216 this breaks on make oldconfig - See image in comment

[davids5]

The defconfig change now is AOK.

version.sh script fails if not under git. That may be the NEW intent, it was not that way in the past.

[davids5]

Any Idea on how to add an test case in this repo, for a relative path custom board?

[xiaoxiang781216]

Any Idea on how to add an test case in this repo, for a relative path custom board?

We can let the relative path point to the normal location:
boards/arch/chip/board/configs/xxx
and enable the custom board related config

[patacongo]

Any Idea on how to add an test case in this repo, for a relative path custom board?

There is no real requirement that the test case be truly a custom board with external board support. I think we could create a sim configuration, perhaps called'custom', that could behave like a custom board in how it is configured.

I maybe overlooking somesomething, but I think it is just a matter of setting up all of the CONFIG_BOARD_CUSTOM* values and configuring the sim configuration as though it were a custom board.

[xiaoxiang781216]

Any Idea on how to add an test case in this repo, for a relative path custom board?

There is no real requirement that the test case be truly a custom board with external board support. I think we could create a sim configuration, perhaps called'custom', that could behave like a custom board in how it is configured.

I maybe overlooking somesomething, but I think it is just a matter of setting up all of the CONFIG_BOARD_CUSTOM* values and configuring the sim configuration as though it were a custom board.

Yes, we can put the board in the normal location, but define CONFIG_BOARD_CUSTOM* to trigger the custom board logic.

[xiaoxiang781216]

@davids5 do you have any more concern? If not, please help merge this PR, thanks.

[xiaoxiang781216]

@Ouss4 The new PR fix all issue you report, please take a look.

[Ouss4]

@xiaoxiang781216 LGTM. Thanks.

[davids5]

@xiaoxiang781216, @Ouss4 What do you think: rebased merge or squash?

[Ouss4]

I think this one should be rebased, the commits are fairly independent.

[davids5]

@Ouss4 - Agreed!