Re: [lttng-dev] Wrong "ar" used when cross-building lttng

2019-11-18 Thread Rolf Eike Beer
Am Freitag, 15. November 2019, 16:34:33 CET schrieb Simon Marchi: > On 2019-11-15 3:18 a.m., Rolf Eike Beer wrote: > > Simon Marchi wrote: > >> On 2019-11-14 11:24 a.m., Rolf Eike Beer wrote: > >>> I have not checked latest git, but maybe you may want to fix this > >>> warning, > >>> too? > >>> >

[lttng-dev] [PATCH 1/2] doc: pass AR when building examples

2019-11-18 Thread Simon Marchi
As reported here [1], when cross-compiling lttng-ust, the "hello-static-lib" example uses the ar tool made for the --build machine instead of the prefixed one, for the --host machine. The Makefiles in the subdirectories of doc/examples are written by hand, so that they can be easily copied and mod

[lttng-dev] [PATCH 2/2] doc: reformat long lines in doc/examples/Makefile.am

2019-11-18 Thread Simon Marchi
Format the long lines in the all-local target a bit like the "cmake" target is formatted already. I think it helps readability to have one argument per line instead of very long lines. At the same time, I removed the "cd .." at the end of parentheses. The parentheses start a new subshell, so it'

Re: [lttng-dev] [PATCH 1/2] doc: pass AR when building examples

2019-11-18 Thread Mathieu Desnoyers
Both patches merged into lttng-ust master branch (only). If you need those backported, please submit separate patches for stable branches, as those do not apply cleanly because the following commit is in master but not in stable branches: commit dc5af9e3311ab28f2728f540f06e61add9d7b5eb Author: Mi

Re: [lttng-dev] [PATCH 1/2] doc: pass AR when building examples

2019-11-18 Thread Simon Marchi
On 2019-11-18 12:15 p.m., Mathieu Desnoyers wrote: > Both patches merged into lttng-ust master branch (only). > > If you need those backported, please submit separate patches for stable > branches, as those do not apply cleanly because the following commit is > in master but not in stable branches

[lttng-dev] [PATCH lttng-ust] doc: fix build failure due to wrong whitespace character

2019-11-18 Thread Simon Marchi
The previous, commit: 54435b75df4c ("doc: reformat long lines in doc/examples/Makefile.am") introduced the following build failure, when the support for JUL is enabled: make[1]: Entering directory '/home/smarchi/build/lttng-ust/doc/examples/java-jul' javac -classpath "../../../libl

Re: [lttng-dev] [PATCH lttng-ust] doc: fix build failure due to wrong whitespace character

2019-11-18 Thread Mathieu Desnoyers
Merged into lttng-ust master, thanks! Mathieu - On Nov 18, 2019, at 1:44 PM, Simon Marchi simon.mar...@efficios.com wrote: > The previous, commit: > >54435b75df4c ("doc: reformat long lines in doc/examples/Makefile.am") > > introduced the following build failure, when the support for J

[lttng-dev] [PATCH lttng-tools] Fix: update apps on untrack only when session is active

2019-11-18 Thread Jonathan Rajotte
This mimics what is done on the track side. Fixes #1209 Signed-off-by: Jonathan Rajotte --- src/bin/lttng-sessiond/trace-ust.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/bin/lttng-sessiond/trace-ust.c b/src/bin/lttng-sessiond/trace-ust.c index 486b53d30..a6

[lttng-dev] [PATCH lttng-tools v2] Fix: update apps on untrack only when session is active

2019-11-18 Thread Jonathan Rajotte
This mimics what is done on the track side. Fixes #1210 Signed-off-by: Jonathan Rajotte --- Used wrong issue number. --- src/bin/lttng-sessiond/trace-ust.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/bin/lttng-sessiond/trace-ust.c b/src/bin/lttng-sessiond/