[Merge] lp:~xnox/upstart/user-log-dir into lp:upstart
James Hunt
james.hunt at canonical.com
Mon Jan 14 13:53:28 UTC 2013
Review: Needs Fixing
* init/main.c:
- handle_logdir(): Is there a reason we are not honouring the environment variable? If not, why don't we put
the 'dir = getenv (LOGDIR_ENV)' immediately before the 'if (user_mode)' since we can then say in the
user_mode block: 'log_dir = dir ? dir : get_user_log_dir ()' and thus honour the env var for user mode too.
* init/xdg.c:
- get_user_log_dir():
- Spacing around '='.
- 'path' is leaked.
- We should be logging to $HOME/.cache/upstart/logs/ as per the spec: this allows us to create a
$HOME/.cache/upstart/sessions/ also in the spec.
* init/man/init.5: This needs updating for '--user' and location of
conf files and logfiles when running in this mode. Specifically the following sections need updating:
- 'User Jobs'
- 'Process environment' -> console -> log
- 'FILES': should reflect conf directories listed in the 'User
Session Mode' and specify location of User Session log files.
* init/man/init.8: Description of '--logdir' and '--confdir' also need updating.
--
https://code.launchpad.net/~xnox/upstart/user-log-dir/+merge/143091
Your team Upstart Reviewers is subscribed to branch lp:upstart.
More information about the upstart-devel
mailing list