high-yak-85899
05/03/2022, 11:58 PMexport
is much slower with a pex-based lockfile. Is this expected?high-yak-85899
05/03/2022, 11:59 PMhundreds-father-404
05/04/2022, 12:00 AMhigh-yak-85899
05/04/2022, 12:03 AMhigh-yak-85899
05/04/2022, 12:18 AMhigh-yak-85899
05/04/2022, 12:19 AMhigh-yak-85899
05/04/2022, 12:19 AMhappy-kitchen-89482
05/04/2022, 12:50 AMhappy-kitchen-89482
05/04/2022, 12:51 AMhappy-kitchen-89482
05/04/2022, 12:51 AMhigh-yak-85899
05/04/2022, 12:53 AMhigh-yak-85899
05/04/2022, 12:58 AMhappy-kitchen-89482
05/04/2022, 1:33 AMhappy-kitchen-89482
05/04/2022, 1:33 AMhappy-kitchen-89482
05/04/2022, 1:33 AMhigh-yak-85899
05/04/2022, 1:38 AMenough-analyst-54434
05/04/2022, 1:41 AMenough-analyst-54434
05/04/2022, 1:45 AMenough-analyst-54434
05/04/2022, 1:48 AMenough-analyst-54434
05/04/2022, 1:49 AMhappy-kitchen-89482
05/04/2022, 2:02 AMenough-analyst-54434
05/04/2022, 2:07 AMhigh-yak-85899
05/04/2022, 3:50 PM./pants export ::
with and without a VCS dependency in my requirements.txt
(and implicitly the lockfile), but maybe it still got included?enough-analyst-54434
05/04/2022, 3:52 PMexport
, as most things in Pants, deals in the transitive closure of dependencies.enough-analyst-54434
05/04/2022, 3:53 PMhigh-yak-85899
05/04/2022, 3:54 PMrequirements.txt
, it is also removed from the lockfile as expected.high-yak-85899
05/04/2022, 4:22 PMrequirements.txt
and the lockfile, it shows up in the exported venv. When I remove it and export again, it is not in the venv as expected.enough-analyst-54434
05/04/2022, 5:04 PMhigh-yak-85899
05/04/2022, 5:18 PMhundreds-father-404
05/04/2022, 5:22 PMsince it's constructing a proper venv instead of symlinking.Given that change, are you still surprised?
high-yak-85899
05/04/2022, 5:23 PMhigh-yak-85899
05/04/2022, 5:24 PMhigh-yak-85899
05/04/2022, 5:24 PMhundreds-father-404
05/04/2022, 5:25 PMhappy-kitchen-89482
05/04/2022, 8:19 PMhappy-kitchen-89482
05/04/2022, 8:20 PMwitty-crayon-22786
05/23/2022, 9:07 PMpip install
in the venv. but if that is going to come at this much of a performance cost, it doesn’t seem like it should be the default. “symlink to immutable venv” is pretty safe otherwise.witty-crayon-22786
05/23/2022, 9:20 PM