On Thu, Oct 01, 2015 at 10:20:02AM +0200, Jan Tluka wrote:
> Thu, Oct 01, 2015 at 09:35:42AM CEST, jprochaz@redhat.com wrote:
> >This may come in handy for user so he can immediately find correct log dir
> >
> >Closes issue #140
> >
> >Signed-off-by: Jiri Prochazka <jprochaz@redhat.com>
> >---
> > lnst-ctl | 1 +
> > 1 file changed, 1 insertion(+)
> >
> >diff --git a/lnst-ctl b/lnst-ctl
> >index 947f3ad..d92eaed 100755
> >--- a/lnst-ctl
> >+++ b/lnst-ctl
> >@@ -333,6 +333,7 @@ def main():
> >     log_ctl.set_recipe("", clean=False)
> >
> >     res_serializer.print_summary()
> >+    logging.info("Saving logs to: %s" % log_ctl.log_folder)
> >
> >     if result_path:
> >         result_path = os.path.expanduser(result_path)
> >--
> >2.4.3
> >
> >_______________________________________________
> >LNST-developers mailing list
> >LNST-developers@lists.fedorahosted.org
> >https://lists.fedorahosted.org/mailman/listinfo/lnst-developers
>
> Either this should be printed right after creating LoggingCtl,
>     log_ctl = LoggingCtl(debug,
>                      log_dir=lnst_config.get_option('environment', 'log_dir'),
>                      log_subdir=date, colours=coloured_output)
>
> or it should say something like "Logs saved to:" at this place. Since
> the subject says print this at the end I'd change the wording.
>
> Otherwise ack.

Or you could just modify the LoggingCtl class to print out the log file
path autmatically when it's being created.

-Ondrej