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

witty-crayon-22786

07/17/2019, 11:20 PM
2) make the local v2 runner run in something that is not a subdirectory of
.pants.d
, which would break these in the local case as well
...and then have a single blacklist for both local and remote
h

hundreds-father-404

07/17/2019, 11:22 PM
don’t want a single blacklist, as there are failures unrelated to this issue
w

witty-crayon-22786

07/17/2019, 11:26 PM
ok. i think that in the long run we want the local and remote behavior to be as close as possible to identical (although remote will always have better sandboxing)
👍 1
so we might have to do this anyway at some point
but fine with waiting.
h

hundreds-father-404

07/17/2019, 11:26 PM
i think that in the long run
and in the long long run (but hopefully not too long), no blacklist 🙂
w

witty-crayon-22786

07/17/2019, 11:26 PM
yep
h

hundreds-father-404

07/17/2019, 11:27 PM
I think https://github.com/pantsbuild/pants/pull/8066 might fix most the issues. Let me test. (Unless that PR is not acceptable)
w

witty-crayon-22786

07/17/2019, 11:28 PM
it sounded like you said that the
//:
part wasn't accurate
assuming that part gets figured out and we're able to apply relevant
files
targets to get things pulled in, then yea... i think that should be ok.
h

hundreds-father-404

07/17/2019, 11:29 PM
I couldn’t get the
//:
part working before. But now it seems like it is working! The PR would end up being what I have up (minus a typo)
w

witty-crayon-22786

07/17/2019, 11:30 PM
oh. yea, that should be great then.
note that the other alternative (and this is probably more portable?) would be to have the base test class write out stable copies of those files during its initialization
h

hundreds-father-404

07/17/2019, 11:32 PM
note that the other alternative (and this is probably more portable?) would be to have the base test class write out stable copies of those files during its initialization
Sure, but that would still require using
files
for those targets so that the base test class can access the original versions.
Oh, or do you mean
python_test_runner.py
?
w

witty-crayon-22786

07/17/2019, 11:32 PM
no, i mean
test_base
unit tests can create files in their buildroot
so you could think of an override near the end of in
def setUp
to call
self.create_file('BUILD.tools', '$somehardcodedcontent')