https://pantsbuild.org/ logo
#development
Title
# development
f

fast-nail-55400

11/05/2021, 6:27 PM
@witty-crayon-22786 / @ancient-vegetable-10556: Any thoughts on the coursier bug described in https://github.com/pantsbuild/pants/issues/13496?
w

witty-crayon-22786

11/05/2021, 6:27 PM
none
does it look hard to fix?
f

fast-nail-55400

11/05/2021, 6:30 PM
should be easy to fix upstream
the harder part is deciding whether we need to deploy our own build of coursier until they do a release upstream
w

witty-crayon-22786

11/05/2021, 6:31 PM
that’s probably the path then… he’s been really helpful getting releases out
well, if we need to temporarily disable strict until then, that’s fine too. it’s not a blocker to have it on… just something i felt would be good to have in place before people are deploying at scale
f

fast-nail-55400

11/05/2021, 6:32 PM
okay, relaxing that requirement until we see a fix simplifies the issue then
w

witty-crayon-22786

11/05/2021, 6:32 PM
@wide-energy-11069: any idea why Twitter never hit this one? see https://github.com/pantsbuild/pants/issues/13496
afaik,
force=True
used this flag, right?
w

wide-energy-11069

11/05/2021, 6:35 PM
does it have anything do with leaving with the version constraints to be open? In twitter, all versions are fixed/pinned.
f

fast-nail-55400

11/05/2021, 6:36 PM
the issue is with the
--strict-include
flag parsing
w

witty-crayon-22786

11/05/2021, 6:37 PM
@fast-nail-55400: i only passed the
$org:$name
portion, because i thought that that was what the flag wanted
f

fast-nail-55400

11/05/2021, 6:38 PM
yeah the issue is that the parser doesn’t like underscores
w

witty-crayon-22786

11/05/2021, 6:38 PM
iirc it is what it wants, but i think that’s what @wide-energy-11069 is asking about
f

fast-nail-55400

11/05/2021, 6:38 PM
which of course blows up with
_2.13
is a suffix
w

witty-crayon-22786

11/05/2021, 6:38 PM
…can always look at the v1 code i guess
f

fast-nail-55400

11/05/2021, 6:39 PM
I’ll fix it upstream by allowing underscore.
w

wide-energy-11069

11/05/2021, 6:39 PM
(FWIW, twitter also never used this flag
--strict-include
)
w

witty-crayon-22786

11/05/2021, 6:39 PM
mm, yea: got it. looks like
force=True
used
-V
…?
f

fast-nail-55400

11/05/2021, 6:40 PM
seems not relevant. the issue is with the coursier strict resolver and the --strict-include flag which Pants v1 doesn’t use.
w

witty-crayon-22786

11/05/2021, 6:40 PM
Copy code
--force-version | -V  <organization:name:forcedVersion>
        Force module version
so what is the difference between forced and strict then… hm.
f

fast-nail-55400

11/05/2021, 6:41 PM
🤷
w

witty-crayon-22786

11/05/2021, 6:41 PM
…oh.
-V
just says: “definitely use this one, even if it’s not compatible.”
i.e., the example in the
strict
ticket passes, and you just get exactly that version.
which … would work, but semantically doesn’t do as much anything to help you out if you’ve pinned to something incompatible
@fast-nail-55400: so yea, i think that helping out to fix
strict
would make sense. because i think that it makes a good default
…in fact. wait a sec. can we just enable it globally instead, rather than poking only the deps we want to be strict?
f

fast-nail-55400

11/05/2021, 6:49 PM
It still gave me an error when I used
--strict
w

witty-crayon-22786

11/05/2021, 6:49 PM
oh… it parses them internally?
f

fast-nail-55400

11/05/2021, 6:49 PM
yup
w

witty-crayon-22786

11/05/2021, 6:49 PM
k.
f

fast-nail-55400

11/05/2021, 6:49 PM
it has a meta-language (because of course it does) for describing the constraints and it parses them …
w

witty-crayon-22786

11/05/2021, 6:50 PM
i think that the path forward is still to fix parsing for
strict
and disable it in the meantime.
in the future we may need to expose
-V
for “i know this isn’t compatible, but use it anyway”, but not right now.
f

fast-nail-55400

11/05/2021, 6:53 PM
i think that the path forward is still to fix parsing for 
strict
 and disable it in the meantime.
agreed
w

witty-crayon-22786

11/05/2021, 6:53 PM
@wide-energy-11069: thanks! hope you’re doing well.
w

wide-energy-11069

11/05/2021, 6:55 PM
np. thanks! you too!