ancient-vegetable-1055608/12/2021, 5:27 PM
hundreds-father-40408/12/2021, 5:39 PM
I currently still calculate the invalidation digest with the requirements strings and the interpreter constraints.That should change. The digest is now only for the requirement strings, and then interpreter constraints are a distinct entry in the header
, then doing
(Completion of this task is blocked on merging #12448)Perhaps add the
method as a prework PR?
I was hoping to wait a little longer before merging #12448 because it's going to hurt performance for Pants devs and result in annoying warnings for you and me w/ our M1s that we override
, until #12542 lands that is
(Another option is to add
to the enum, which we may want to do anyways. Even if we end up deleting that before 2.7 stable goes out.)
ancient-vegetable-1055608/12/2021, 6:00 PM
witty-crayon-2278608/12/2021, 6:00 PM
seems reasonable. although it should obviously be discouraged, i don’t think that removing it before launch is even critical.