Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36343 )
Change subject: payloads/external/GRUB2: fix constantly rebuilding due to git checkout ......................................................................
Patch Set 2:
Patch Set 2: Code-Review+2
Is it possible that the old version maintained local changes and the new doesn't? If so, that could lead to a lot of frustration for users that are used to it.
hm, possible but frustrating, too, because one needs to handle merge conflicts while running `make`
Yeah, just saying we might break somebody's workflow. We could try to handle it more nicely. For instance, issue a warning if the index/tree aren't clean and skip the checkout in that case?
Done!