More explicit env generation logging

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Latest
 
+- [Improvement] More explicit logging during environment generation
 - [Improvement] Configurable docker images (#122)
 - [Bugfix] Fix "android pullimage" command
 - [Improvement] Do not upgrade images as part of quickstart

tutor/android.py

@@ -23,7 +23,6 @@ def env(root):
     config["LMS_HOST_REVERSE"] = ".".join(config["LMS_HOST"].split(".")[::-1])
     tutor_env.render_target(root, config, "build")
     tutor_env.render_target(root, config, "android")
-    click.echo(fmt.info("Environment generated in {}".format(root)))
 
 @click.group(
     help="Build the application"

tutor/env.py

@@ -2,9 +2,11 @@ import codecs
 import os
 import shutil
 
+import click
 import jinja2
 
 from . import exceptions
+from . import fmt
 from . import utils
 
 
@@ -21,6 +23,7 @@ def render_target(root, config, target):
                 substituted = render_str(fi.read(), config)
             with open(dst, "w") as of:
                 of.write(substituted)
+    click.echo(fmt.info("Environment generated in {}".format(pathjoin(root, target))))
 
 def render_dict(config):
     """
@@ -67,6 +70,7 @@ def copy_target(root, target):
     for src, dst in walk_templates(root, target):
         if is_part_of_env(src):
             shutil.copy(src, dst)
+    click.echo(fmt.info("Environment generated in {}".format(pathjoin(root, target))))
 
 def is_part_of_env(path):
     return not os.path.basename(path).startswith(".")

tutor/images.py

@@ -23,7 +23,6 @@ argument_image = click.argument(
 @opts.root
 def env(root):
     tutor_env.copy_target(root, "build")
-    click.echo(fmt.info("Environment generated in {}".format(root)))
 
 @click.command(
     short_help="Download docker images",

tutor/k8s.py

@@ -39,7 +39,6 @@ def env(root):
     config = tutor_config.load(root)
     tutor_env.render_target(root, config, "apps")
     tutor_env.render_target(root, config, "k8s")
-    click.echo(fmt.info("Environment generated in {}".format(root)))
 
 @click.command(help="Run all configured Open edX services")
 @opts.root

tutor/local.py

@@ -51,7 +51,6 @@ def env(root):
     config = tutor_config.load(root)
     tutor_env.render_target(root, config, "apps")
     tutor_env.render_target(root, config, "local")
-    click.echo(fmt.info("Environment generated in {}".format(root)))
 
 @click.command(
     help="Update docker images",