Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42548 )
Change subject: Add qc_blobs repository ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42548/3/.gitmodules File .gitmodules:
https://review.coreboot.org/c/coreboot/+/42548/3/.gitmodules@52 PS3, Line 52: update = none
Even with `update = none` it still gets cloned when one wants to update […]
I assume you mean `git submodule update --checkout`? If I understand it correctly, the `--checkout` is only there to explicitly tell Git to ignore any `update = none` and treat them all as if they said `update = checkout`. Do you have to pass --checkout? If you don't want it to check out the repositories that are marked `update = none`, just running `git submodule update` should be the right thing.
Maybe the problem is that we have too many repositories marked `update = none` here? Some of them should really have no reason to be skipped and were probably just copy&pasted without knowing what it means. We should make sure only the controversial repositories are marked `update = none` so people don't get into the habit of running `git submodule update` with `--checkout`.