Skip to content

cmake: normalize .config on reconfigure - #19693

Open
raiden00pl wants to merge 1 commit into
apache:masterfrom
raiden00pl:cmake-config-normalise
Open

cmake: normalize .config on reconfigure#19693
raiden00pl wants to merge 1 commit into
apache:masterfrom
raiden00pl:cmake-config-normalise

Conversation

@raiden00pl

Copy link
Copy Markdown
Member

Summary

olddefconfig only ran when .config was generated from the defconfig, so changes applied to an existing .config (e.g. with kconfig-tweak) never got their dependent defaults. Run olddefconfig on reconfigure as well, as the Make flow does on every build.

Impact

align cmake with make

Testing

update cmake generated config with kconfig-tweak

jerpelea
jerpelea previously approved these changes Aug 5, 2026
@raiden00pl
raiden00pl force-pushed the cmake-config-normalise branch from 019f54c to 09a1e4b Compare August 5, 2026 08:36
@github-actions github-actions Bot added Area: Build system Size: XS The size of the change in this PR is very small labels Aug 5, 2026
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

xiaoxiang781216
xiaoxiang781216 previously approved these changes Aug 5, 2026
@raiden00pl
raiden00pl force-pushed the cmake-config-normalise branch from 09a1e4b to 219b1f0 Compare August 6, 2026 06:45
@github-actions github-actions Bot added Size: S The size of the change in this PR is small and removed Size: XS The size of the change in this PR is very small labels Aug 6, 2026
@raiden00pl
raiden00pl force-pushed the cmake-config-normalise branch from 219b1f0 to c74a5f0 Compare August 6, 2026 11:54
olddefconfig only ran when .config was generated from the defconfig,
so changes applied to an existing .config (e.g. with kconfig-tweak)
never got their dependent defaults. Run olddefconfig on reconfigure as
well, as the Make flow does on every build.

Signed-off-by: raiden00pl <raiden00@railab.me>
Assisted-by: Claude Code
@raiden00pl
raiden00pl force-pushed the cmake-config-normalise branch from c74a5f0 to 9d19e26 Compare August 6, 2026 16:09
@xiaoxiang781216

Copy link
Copy Markdown
Contributor

@michael-membrowse could you look at why MemBrowse report failure?

@Mrogovp

Mrogovp commented Aug 7, 2026

Copy link
Copy Markdown

@xiaoxiang781216 thank you for pointing it out. works now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Build system Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants