hundreds-breakfast-4901009/22/2020, 7:15 PM
block where the presence/absence of the
is controllable via a flag?
hundreds-breakfast-4901009/22/2020, 9:48 PM
hundreds-breakfast-4901009/22/2020, 11:01 PM
option with pants, it prints every log line twice
hundreds-breakfast-4901009/22/2020, 11:01 PM
hundreds-breakfast-4901009/23/2020, 12:16 AM
hundreds-breakfast-4901009/23/2020, 12:17 AM
, then hit ctrl-c, some kind of python process will remain around and screwing up the terminal
wonderful-iron-5401909/23/2020, 5:05 PM
/`python_tests` get resolved as we move our tests to v2 seeing an interesting issue when bringing in our common test utils
hundreds-father-40409/23/2020, 9:28 PM
for things like Pytest? I think the
can be consolidated into the
. Iirc, it was to avoid having to recreate the
when you do something like bump a tool’s version. But that doesn’t seem like we save much; all the
ever does is set up an entry-point and
, which is trivial to do directly on the
in the first place.
wonderful-iron-5401909/24/2020, 6:23 PM
hundreds-breakfast-4901009/24/2020, 11:58 PM
as a default ignore pattern in the
might be undesirable
hundreds-breakfast-4901009/25/2020, 6:06 PM
witty-crayon-2278609/25/2020, 6:25 PM
should probably not be a `SessionValue`: https://github.com/pantsbuild/pants/issues/6845 and https://github.com/pantsbuild/pants/blob/master/src/rust/engine/graph/README.md#inputs-and-identity attempt to explain the connection between identity and cacheability, but they don’t do a great job of it
fast-nail-5540009/26/2020, 3:10 AM
Compiling process_execution v0.0.1 (/data/home/tdyas/pants/src/rust/engine/process_execution) Compiling engine v0.0.1 (/data/home/tdyas/pants/src/rust/engine) Finished release [optimized + debuginfo] target(s) in 9m 25s /data/home/tdyas/pants/src/python/pants/base/exception_sink.py:313: DeprecationWarning: PY_SSIZE_T_CLEAN will be required for '#' formats process_title=setproctitle.getproctitle(), /data/home/tdyas/pants/src/python/pants/engine/internals/native_engine.so: undefined symbol: PyModule_Create2TraceRefs NoneType: None
fast-nail-5540009/26/2020, 3:11 AM
hundreds-breakfast-4901009/26/2020, 8:02 AM
https://github.com/pantsbuild/pants/blob/master/src/rust/engine/src/externs/interface.rs#L1778` needs to take a
and not a
aloof-angle-9161609/27/2020, 2:45 PM
in the local store source above the
TODO: Use LMDB database statistics when lmdb-rs exposes them.
method: https://github.com/pantsbuild/pants/blob/237e6a6e8b1c41d58a38426d92dac96f878f2159/src/rust/engine/fs/store/src/local.rs#L121, and in looking over that i found this
object from the upstream lmdb-rs crate: https://github.com/danburkert/lmdb-rs/blob/12b4611006d5ba3bf80840faaa31a8b2e2bb6e88/src/environment.rs#L171. it's described as
. is that the info we would be looking for here?
Contains information about the size and layout of an LMDB environment.
aloof-angle-9161609/27/2020, 2:52 PM
anywhere. were we doing that manually in the parallel
task before? i'm looking to see whether we could support an optimization to avoid snapshotting process execution output files/directories that we already know about as per this comment on the mutable caches doc (which i didn't know was already implemented -- this is super awesome): https://docs.google.com/document/d/1n_MVVGjrkTKTPKHqRPlyfFzQyx2QioclMG_Q3DMUgYk/edit?disco=AAAAIva7gMw
aloof-angle-9161609/27/2020, 2:53 PM
aloof-angle-9161609/27/2020, 9:58 PM
today i think as per the linked comment
hundreds-father-40409/28/2020, 7:23 PM
Going from 120 pixels wide to 121 pixels wide in my terminal fixed it..?
▶ ./pants test src/python/pants/backend/python/lint/docformatter/rules_integration_test.py -- -k type_stubs ⠉ ⠉ ⠉ ⠉ ⠉ ⠉ 12:21:21.38 [WARN] Completed: Run tests - src/python/pants/backend/python/lint/docformatter/rules_in 𐄂 src/python/pants/backend/python/lint/docformatter/rules_integration_test.py:integration failed.
hundreds-breakfast-4901009/28/2020, 7:49 PM
in https://github.com/pantsbuild/pants/blob/master/src/rust/engine/process_execution/src/lib.rs#L491 ? @witty-crayon-22786 I think you might've touched this code?
witty-crayon-2278609/28/2020, 8:52 PM
messages for test processes
wooden-thailand-838609/29/2020, 8:14 PM
target when using
it can get files from a different folder by using
right? I mean, that’s basically what I’m doing… my BUILD file is inside
and I need to have files under
to be packaged together.
wonderful-iron-5401909/30/2020, 1:53 PM
in Pants 1.30.2rc0. Stack in thread
./pants --no-v1 lint
aloof-angle-9161609/30/2020, 3:02 PM
which doesn't register any options itself, but allows environment variables in the form
to be converted to just
environment variables when we go to run a process (where
is the value of a class-level string field
. making this a subsystem (or not, and using some other method) means we could hook it up to
and allow the user to see what variables are in their environment which would affect a pants execution, and we could also incorporate the tool's own help text for its environment variables. posting here because i made the comment specifically regarding using this for pex in the issue, and i want to see what response i get to that before trying to make a more general issue