Hi Martin,
sounds like a really good idea!
On 21.01.24 05:16, Martin Roth via coreboot wrote:
Here's a spreadsheet with the submodule size data, along for recommendations for each. https://docs.google.com/spreadsheets/d/1DAnFFnoLxdLE15CsUTE_AuCZeKG-UdrBndiF... https://docs.google.com/spreadsheets/d/1DAnFFnoLxdLE15CsUTE_AuCZeKG-UdrBndiFxhPDEO8/edit?usp=sharing
Submodules recommended to use --depth=1 are: arm-trusted-firmware, intel-sec-tools, stm, chromeec, blobs, fsp, vboot
Not sure if we actually need to make a distinction. Most people probably won't look into the submodules anyway. Your selection seems fine to me, though.
All other submodules would fetch the current branch. My thought is that we set the default for each submodule to the recommended value in the sheet, then we can add a Make target or Kconfig option that pulls down the rest, for anyone who wants the full data.
Most people who edit submodules probably know Git well enough. So I would only invest into the Kconfig/Makefile logic if people request it.
Cheers, Nico