[Bug 70990] [1.0] Logger parallel-installability

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=70990

--- Comment #27 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> ---
(In reply to comment #15)
> Comment on attachment 91976 [details] [review]
> log-store-xml: stop using TPL_TEST_LOG_DIR
> 
> Review of attachment 91976 [details] [review]:
> -----------------------------------------------------------------
> 
> tests/logger/Makefile.am doesn't adjust XDG_DATA_HOME. Should it?

It was already set.

> It would be great if the Logger's XDG_DATA_HOME could move into the tests
> themselves so they can safely be run without the wrapper (as a bonus, they
> could use separate log directories and become parallel-safe) but that's a
> job for the future.

I tried that actually: relying on the C tests to set all the magic env variable
itself. My goal was to run each test in its own /tmp/logger-tests-log-XXX/
directory and so be more self contained. But that didn't work because GLib
internally cache the value of the variable, so it can't be changed during the
lifetime of the process:
https://git.gnome.org/browse/glib/tree/glib/gutils.c#n1156

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.