On Fri, Jan 30, 2026 at 03:20:22PM -0800, Song Liu wrote: > On Fri, Jan 30, 2026 at 2:54 PM Josh Poimboeuf <[email protected]> wrote: > > Something like this? > > > > if (( SHORT_CIRCUIT <= 2 )); then > > status "Validating patch(es)" > > validate_patches > > fix_patches # including fixing fuzz??? > > fi > > I was thinking to change the above as > if (( SHORT_CIRCUIT <= 0 )) > > Then we can save the fixed version of all the patches. > > But I think "SHORT_CIRCUIT <= 2" is cleaner, so this version is better.
Right. Just to clarify, the point of doing --short-circuit=2 (from my experience) is you want to use a new .patch file which is different from the one used by the previous klp-build incantation, but you don't want to wait for the original build again. Putting that behind "SHORT_CIRCUIT <= 0" would prevent the use of the new .patch file. -- Josh
