[lttng-dev] [PATCH 1/3] tests/lib/Makefile.am: Remove test_seek_empty_packet and test_seek_big_trace from SCRIPT_LIST
simon.marchi at polymtl.ca
Thu Apr 28 18:36:17 UTC 2016
On 2016-04-28 14:28, Michael Jeanson wrote:
> ----- On Apr 28, 2016, at 12:44 PM, Simon Marchi
> simon.marchi at polymtl.ca wrote:
>> Ok thanks.
>> Just an idea: I find it quite cumbersome to add a new entry in
>> configure.ac for each test script that we want autoconf to process
>> as in patch 3/3). I think it could be easier to have a single file
>> (e.g. vars.sh) and have it contain definitions of various directories:
>> This way, you could have only a single generated file (vars.sh.in ->
>> vars.sh), and each test script would just have to source that file to
>> use the values.
> As discussed on #lttng, here is the PR resulting from my updates to
> patches :
Thanks for following up.
> Concerning the include file, I'm not convinced it would be simpler, we
> end up with static scripts in the source dir including a generated file
> the builddir.
In the current state, even the static scripts are copied to the build
dir, so it
wouldn't be a problem. Anyway, if it's not a problem, I think it looks
good in your
More information about the lttng-dev