Replies: 3 comments 1 reply
-
I definitely agree with making this an optional thing instead of a mandatory one. |
Beta Was this translation helpful? Give feedback.
-
It's used in CI:
Lines 18 to 19 in 9e9c2e2 No gripe dropping it from the default Think this can go, too (last time we chatted there was no difference anymore): Lines 77 to 84 in 9e9c2e2 |
Beta Was this translation helpful? Give feedback.
-
Completed with: #3510 |
Beta Was this translation helpful? Give feedback.
-
Currently when running the main Zeek BTest suite, its configuration includes:
ZEEK_PROFILER_FILE=%(testbase)s/.tmp/script-coverage/XXXXXX
. This generates 100s of MB of data in.tmp/script-coverage
every run. When doing significant development this quickly turns into GBs.Chatting with @timwoj we're not sure that this data is actually ever used in practice. In addition, seems for sure it should be an optional testing feature (which could be turned on using a BTest
-a
alternative) rather than the default.Beta Was this translation helpful? Give feedback.
All reactions