> [WARN] DEPRECATED: indirectly referring to a ...
# generalh
high-yak-85899
08/30/2023, 6:59 PM[WARN] DEPRECATED: indirectly referring to a target by using a corresponding file argument, when the target owning the file isn't applicable is scheduled to be removed in version 2.18.0.dev1What's the background on this one? I find it incredibly convenient to not have to remember what the target address is in the build file.
b
bitter-ability-32190
08/30/2023, 7:00 PMThis is specifically when running
package path/to/file.pybitter-ability-32190
08/30/2023, 7:00 PM(or it should be! 😅 )
h
high-yak-85899
08/30/2023, 7:01 PMThis came up for us running
pants run path/to/file.pyhigh-yak-85899
08/30/2023, 7:01 PMAnd I think it's because we have the
python_sourcespex_binaryhigh-yak-85899
08/30/2023, 7:01 PMSo maybe there was some ambiguity there
high-yak-85899
08/30/2023, 7:01 PMI removed the
pex_binaryb
bitter-ability-32190
08/30/2023, 7:01 PMWhat version is this?
h
high-yak-85899
08/30/2023, 7:03 PM2.17.0 😎
b
bitter-ability-32190
08/30/2023, 7:03 PM😬
bitter-ability-32190
08/30/2023, 7:06 PMThis should've been fixed in https://github.com/pantsbuild/pants/pull/19224
bitter-ability-32190
08/30/2023, 7:08 PMUmmmmmmmmmm this is likely a showstopper
bitter-ability-32190
08/30/2023, 7:09 PMCC @happy-kitchen-89482
h
high-yak-85899
08/30/2023, 7:10 PMThings worked fine with the warning fwiw
b
bitter-ability-32190
08/30/2023, 7:40 PMI'm not 100% sure how we'd fix this either.
runbitter-ability-32190
08/30/2023, 7:42 PMThat's not true, we could easily add a new parameter to
TargetRootsToFieldSetsRequestbitter-ability-32190
08/30/2023, 7:42 PM😮💨 I'll go ahead and fix this on 2.17.x. Then @happy-kitchen-89482 @witty-crayon-22786 we should probably discuss somewhere what to do next
👍 1
w
witty-crayon-22786
08/30/2023, 7:45 PMoy. that deprecation has been a pain in the butt. thanks Josh: it should be worth it!
🤮 1
b
bitter-ability-32190
08/30/2023, 7:51 PMbitter-ability-32190
08/30/2023, 8:23 PM