Hi all, I’m trying to sort out some dependency man...
# generalf
future-oxygen-10553
02/14/2023, 7:24 PMHi all, I’m trying to sort out some dependency management between first party code. I have a sample repo here: https://github.com/bryanwweber/pants-multi-deps The short version is I have a module
ops_core.data_io.pathlibclient_code.clientpantsfuture-oxygen-10553
02/14/2023, 7:25 PM Copy code
❯ pants --no-pantsd dependencies --transitive src/client_code/client_code/client.py
14:21:31.74 [WARN] Pants cannot infer owners for the following imports in the target src/client_co
de/client_code/client.py:
* ops_core.data_io.pathlib.AnyPath (line: 1)
If you do not expect an import to be inferrable, add `# pants: no-infer-dep` to the import line. O
therwise, see <https://www.pantsbuild.org/v2.14/docs/troubleshooting#import-errors-and-missing-depe>
ndencies for common problems.future-oxygen-10553
02/14/2023, 7:26 PMThe
--no-pantsdfuture-oxygen-10553
02/14/2023, 7:26 PMThe output of
pants rootssrce
enough-analyst-54434
02/14/2023, 7:27 PMYeah
rootsenough-analyst-54434
02/14/2023, 7:27 PMsrc/ is not the root
enough-analyst-54434
02/14/2023, 7:27 PMthe two subroots are.
f
future-oxygen-10553
02/14/2023, 7:27 PMOK, thanks! I actually just discovered that myself
future-oxygen-10553
02/14/2023, 7:28 PMThe problem is that in the real repo, that doesn’t work 😄
e
enough-analyst-54434
02/14/2023, 7:28 PMsource roots == sys.path entries you need to make your code importable == PYTHONPATH for Python.
enough-analyst-54434
02/14/2023, 7:28 PMOk, well more work to do on the real problem repro / debug then.
f
future-oxygen-10553
02/14/2023, 7:29 PMYep 😞 Thanks for the quick response again!
e
enough-analyst-54434
02/14/2023, 7:29 PMThanks for trying to make a repro case. That is really helpful.
f
future-oxygen-10553
02/14/2023, 7:30 PMNo problem. I’ve been an open-source maintainer for long enough that I get the pain of “THIS IS A PROBLEM” 🦗 😄
e
enough-analyst-54434
02/14/2023, 7:30 PMI mean, you don't even need that. You just need a grandpa with a printer.
enough-analyst-54434
02/14/2023, 7:30 PMWho lives elsewhere.
f
future-oxygen-10553
02/14/2023, 7:32 PMOK, I can repro again if I’m using resolves
future-oxygen-10553
02/14/2023, 7:33 PMI just pushed a new commit
future-oxygen-10553
02/14/2023, 7:36 PMI guess, reading the new error message, the question is how can I add first-party code to the
resolvesfuture-oxygen-10553
02/14/2023, 7:40 PMOK, I can
parametrizeops_coreclient_codefuture-oxygen-10553
02/14/2023, 7:40 PMCan that be auto-parametrized for all my source roots?
e
enough-analyst-54434
02/14/2023, 7:44 PMDoes this work for you? https://www.pantsbuild.org/docs/targets#field-default-values
enough-analyst-54434
02/14/2023, 7:45 PMSet up 1 BUILD file per source root that sets defaults.
f
future-oxygen-10553
02/14/2023, 7:48 PMHmm, I think that could work
future-oxygen-10553
02/14/2023, 7:49 PMWe’d obviously have to keep it up-to-date as we add new resolves, but it should only happen in one or two places (I hope), so that’s not too bad
e
enough-analyst-54434
02/14/2023, 7:52 PMYeah, exactly. As simple as a comment in the
pants.toml3 Views