From a770151359a9c573bfd967f79eab26670d499af0 Mon Sep 17 00:00:00 2001
From: alalazo <massimiliano.culpo@googlemail.com>
Date: Thu, 30 Jun 2016 16:49:24 +0200
Subject: [PATCH] module : minor improvement to output formatting

---
 bin/spack                     | 2 +-
 lib/spack/spack/cmd/module.py | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/bin/spack b/bin/spack
index e9307d1485..9b1276a866 100755
--- a/bin/spack
+++ b/bin/spack
@@ -77,7 +77,7 @@ import llnl.util.tty as tty
 from llnl.util.tty.color import *
 import spack
 from spack.error import SpackError
-import argparse
+from external import argparse
 
 # Command parsing
 parser = argparse.ArgumentParser(
diff --git a/lib/spack/spack/cmd/module.py b/lib/spack/spack/cmd/module.py
index 4f6b4764b6..4f0593e45e 100644
--- a/lib/spack/spack/cmd/module.py
+++ b/lib/spack/spack/cmd/module.py
@@ -95,9 +95,10 @@ def module_refresh(name, specs, args):
         tty.msg('No package matches your query')
         return
 
-    tty.msg('You are about to regenerate the {name} module files for the following specs:'.format(name=name))
+    tty.msg('You are about to regenerate the {name} module files for the following specs:\n'.format(name=name))
     for s in specs:
         print(s.format(color=True))
+    print('')
     ask_for_confirmation('Do you want to proceed ? ')
 
     cls = module_types[name]
@@ -112,10 +113,9 @@ def module_refresh(name, specs, args):
         message = 'Name clashes detected in module files:\n'
         for filename, writer_list in file2writer.items():
             if len(writer_list) > 1:
-                message += 'file : {0}\n'.format(filename)
+                message += '\nfile : {0}\n'.format(filename)
                 for x in writer_list:
                     message += 'spec : {0}\n'.format(x.spec.format(color=True))
-                message += '\n'
         tty.error(message)
         tty.error('Operation aborted')
         raise SystemExit(1)
-- 
GitLab