Bug 1535242 - Provide better message when choosing a directory to clone. r=glandium

- Do not use `ERROR` when user enters possibly parent-directory when
   choosing the directory to clone mozilla-unified
 - Show the path of pre-existing mozilla-unified path on error

Differential Revision: https://phabricator.services.mozilla.com/D23456

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Tooru Fujisawa 2019-03-15 08:50:28 +00:00
Родитель 01a0514d6f
Коммит 96a6c6f371
1 изменённых файлов: 4 добавлений и 4 удалений

Просмотреть файл

@ -151,9 +151,9 @@ CLONE_VCS_PROMPT = '''
Destination directory for {} clone (leave empty to not clone): '''.lstrip()
CLONE_VCS_NOT_EMPTY = '''
ERROR! Destination directory '{}' is not empty.
Destination directory '{}' is not empty.
Would you like to clone to '{}'?
Would you like to clone to '{}' instead?
1. Yes
2. No, let me enter another path
@ -162,7 +162,7 @@ Would you like to clone to '{}'?
Please enter your reply: '''.lstrip()
CLONE_VCS_NOT_EMPTY_FALLBACK_FAILED = '''
ERROR! Destination directory '{}' is not empty.
ERROR! Destination directory '{}' is not empty and '{}' exists.
'''
CLONE_VCS_NOT_DIR = '''
@ -325,7 +325,7 @@ class Bootstrapper(object):
newdest = os.path.join(dest, repo_name)
if os.path.exists(newdest):
print(CLONE_VCS_NOT_EMPTY_FALLBACK_FAILED.format(dest))
print(CLONE_VCS_NOT_EMPTY_FALLBACK_FAILED.format(dest, newdest))
continue
choice = self.instance.prompt_int(prompt=CLONE_VCS_NOT_EMPTY.format(dest,