~lttng/lttng-ust/lttng-ust

« back to all changes in this revision

Viewing changes to src/lib/lttng-ust/lttng-ust-comm.c

  • Committer: Mathieu Desnoyers
  • Date: 2023-10-20 15:57:20 UTC
  • Revision ID: git-v1:8bc1125eb851b2c52d3263c2992e6806017e98e7
Revert "Add support for LTTNG_UST_HOME"

This reverts commit 90d125c709f566f3663bf84677f100134cc618e0.

After discussion with Jeremie, we want to introduce two (not one)
environment variables:

- LTTNG_UST_APP_PATH,
- LTTNG_UST_CTL_PATH.

to accomodate use-cases where a sessiond within a container is traced by
a sessiond in the parent container. In that situation, we want the
sessiond in the parent container to access the tracee through the
LTTNG_UST_CTL_PATH, without making the unix sockets for tracing control
visible to the child container.

Therefore, remove the LTTNG_UST_HOME environment variable before it is
added into an official release.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

Show diffs side-by-side

added added

removed removed

Lines of Context:
370
370
/*
371
371
 * Returns the HOME directory path. Caller MUST NOT free(3) the returned
372
372
 * pointer.
373
 
 * The following env are checked in order of priority:
374
 
 *  1 - LTTNG_UST_HOME
375
 
 *  2 - LTTNG_HOME
376
 
 *  3 - HOME
377
373
 */
378
374
static
379
375
const char *get_lttng_home_dir(void)
380
376
{
381
377
       const char *val;
382
378
 
383
 
       val = (const char *) lttng_ust_getenv("LTTNG_UST_HOME");
384
 
       if (val != NULL) {
385
 
               return val;
386
 
       }
387
 
 
388
379
       val = (const char *) lttng_ust_getenv("LTTNG_HOME");
389
380
       if (val != NULL) {
390
381
               return val;
391
382
       }
392
 
 
393
383
       return (const char *) lttng_ust_getenv("HOME");
394
384
}
395
385