witty-crayon-22786
08/24/2021, 6:35 PMdefault='default'
) in 2.7.x? are we concerned that those won’t be resolvable on some platforms?witty-crayon-22786
08/24/2021, 6:36 PMhundreds-father-404
08/24/2021, 6:54 PM./pants tool-lock
semantics to let you only generate for certain lockfiles
3. Further improve our staleness check error message to e.g. explain <default>
4. Determine if we are okay with lockfile generation changing between 2.6 and 2.7: is that a breaking change?
5. Determine if it's safe to set default="<default>"
without a deprecation?
I think that's achievable this week if Chris and I sprint on 1-3, and we decide 4-5 are fine! It means not working on multiple lockfiles for now, thowitty-crayon-22786
08/24/2021, 7:06 PMDetermine if we are okay with lockfile generation changing between 2.6 and 2.7: is that a breaking change?there are no lockfiles for tools in 2.6… to be clear, i’m only talking about tools right now
hundreds-father-404
08/24/2021, 7:29 PM[python-setup].requirement_constraints
🤢 I don't know how to go from that to default lockfiles
Key note w/ default lockfile: Chris's invalidation mechanism will warn/error if the default doesn't work with your settings, like you've changed --version
or --interpreter-constraints
hundreds-father-404
08/24/2021, 7:30 PMI don't know how to go from that to default lockfilesI think Benjy has said that because 2.6 was so broken, it's okay to not deprecate the broken functionality. I think the path forward is fix tool PEXes to never use the constraints file. Only way to lock is
[tool].lockfile
. And we default to <default>
, which will be breaking but we make an exception and document it well because things were so broken before. cc @happy-kitchen-89482, thoughts?happy-kitchen-89482
08/24/2021, 8:07 PMhundreds-father-404
08/24/2021, 8:24 PMwitty-crayon-22786
08/24/2021, 8:32 PMI think the path forward is fix tool PEXes to never use the constraints file. Only way to lock isyea, definitely.. And we default to[tool].lockfile
, which will be breaking but we make an exception and document it well because things were so broken before. cc @happy-kitchen-89482, thoughts?<default>
witty-crayon-22786
08/24/2021, 8:33 PMhundreds-father-404
08/24/2021, 8:36 PMwitty-crayon-22786
08/24/2021, 8:38 PMhundreds-father-404
08/24/2021, 10:14 PM[python-repos]
, which will impact generating tool lockfiles for some orgs. I think we can not let perfect be the enemy of good, though. Some orgs will have to manualy provide their lockfile or disable the feature
Let's do the right thing for everyone else by defaulthundreds-father-404
08/24/2021, 11:47 PM./pants lock
(user lockfiles) to be ./pants user-lock
and still feature gate it
./pants tool-lock
becomes ./pants lock
and is registered by pants.backend.python
. (subject to bikeshedding lock
vs resolve
for goal name)witty-crayon-22786
08/24/2021, 11:48 PMhundreds-father-404
08/24/2021, 11:48 PMwitty-crayon-22786
08/24/2021, 11:48 PMwitty-crayon-22786
08/24/2021, 11:49 PM