makefiles/info-global.inc.mk: bug: honor DEFAULT_MODULES

[cladmi]

Contribution description

The parsing for info-boards-supported was completely ignoring default
modules.

Testing procedure

Review that both lines are done in Makefile.include.
Murdock should compile everything without error.

I will post a diff using dependencies resolution output when I have it.

When running the same procedure as in #12014
And comparing the difference of the aggregated info-boards-supported debug output I get:

diff -y --suppress-common-lines build/deps/ref/deps_info-boards-supported build/deps/2_default_modules/deps_info-boards-supported  | grep  -v USEMODULE | sort -u
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init

And we can see that USEMODULE now has values from the DEFAULT_MODULE:

diff -y --suppress-common-lines build/deps/ref/deps_info-boards-supported build/deps/2_default_modules/deps_info-boards-supported  | head -n 10
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = checksum periph_common periph_gpio periph_pm riot | USEMODULE = board checksum core core_msg cpu periph_common pe
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = checksum periph_common periph_gpio periph_pm riot | USEMODULE = board checksum core core_msg cpu periph_common pe
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = boards_common_arduino-atmega boards_common_atmega | USEMODULE = board boards_common_arduino-atmega boards_common_
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init

Issues/PRs references

This is based on

• makefiles/info-global.inc.mk: bug: fix DISABLE_MODULE_GLOBAL value makefiles/info-global.inc.mk: bug: fix DISABLE_MODULE_GLOBAL value #12010
• make: add targets to debug dependencies variables make: add targets to debug dependencies variables #12004 for debugging

[cladmi]

Rebased due to conflict in Makefile.include.

[jcarrano]

Looks good to me and restores the original, sensible behaviour of this variable.

[cladmi]

This one is implemented on top of #12010 and the debug targets from #12004

Should I squash to remove the test commits ?

I just noticed I did not provide the diff in the description.

When running the same procedure as in #12014
And comparing the difference of the aggregated info-boards-supported debug output I get:

diff -y --suppress-common-lines build/deps/ref/deps_info-boards-supported build/deps/2_default_modules/deps_info-boards-supported  | grep  -v USEMODULE | sort -u
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init

And we can see that USEMODULE now has values from the DEFAULT_MODULE:

diff -y --suppress-common-lines build/deps/ref/deps_info-boards-supported build/deps/2_default_modules/deps_info-boards-supported  | head -n 10
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = checksum periph_common periph_gpio periph_pm riot | USEMODULE = board checksum core core_msg cpu periph_common pe
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = checksum periph_common periph_gpio periph_pm riot | USEMODULE = board checksum core core_msg cpu periph_common pe
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init
DISABLE_MODULE =                                              | DISABLE_MODULE = auto_init
USEMODULE = boards_common_arduino-atmega boards_common_atmega | USEMODULE = board boards_common_arduino-atmega boards_common_
DEFAULT_MODULE =                                              | DEFAULT_MODULE = board cpu core core_msg sys auto_init

[cladmi]

Rebased and removed the debug target commit now that #12010 is in.

[cladmi]

Thank you for the review.