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
Yes, `git submodule update --checkout`. It's also an alias in our gitconfig […]
Hmm... well, I think it's hard to make everybody happy if there are so many different cases to cover. I don't think removing the repo entirely from .gitmodules is fair either because that does create a lot of hassle for the people who do need to work with it. Instead we should try to build something where people aren't lead to manually run update --checkout unless they really know what that does and really want all those blobs.
Can we maybe just add more .gitconfig aliases to handle your use case instead? I'm fine with changing the existing one to exclude qc_blobs if you want (personally, I don't use it... if other people do and want qc_blobs to be included, we can split it into `git sup` and `git sup-dangerous` or something like that). `git submodule status` prints all submodules, so with some cut and grep it shouldn't be hard to build commands that sync whichever subset you prefer.