diff options
author | Alexandru Croitor <alexandru.croitor@qt.io> | 2021-10-22 18:32:18 +0200 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2021-10-27 16:36:49 +0000 |
commit | 25ab3ca6cda0fe5409dfc3845db691cac2268a17 (patch) | |
tree | 48eaec9c1908ee6a90ed071abd8a240c411002ec /build_scripts | |
parent | b02e70ca9aebced0a76cc36e9071cb73281afa63 (diff) |
setup.py: Make file paths in 'file copying' logs easier to read
Placing both source and destination paths on new lines makes it
easier to visually compare the difference between paths.
Change-Id: I0d44bb77f597e6144eae841a7d5625d5df2f4769
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
(cherry picked from commit 52617575feaa6f48ac542c14b628bdd7b7b8fc92)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'build_scripts')
-rw-r--r-- | build_scripts/utils.py | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/build_scripts/utils.py b/build_scripts/utils.py index 4f22f7d7b..45d7481cf 100644 --- a/build_scripts/utils.py +++ b/build_scripts/utils.py @@ -246,7 +246,7 @@ def copyfile(src, dst, force=True, vars=None, force_copy_symlink=False, dst = dst.format(**vars) if not os.path.exists(src) and not force: - log.info(f"**Skipping copy file {src} to {dst}. Source does not exists.") + log.info(f"**Skipping copy file\n {src} to\n {dst}\n Source does not exist") return if not os.path.islink(src) or force_copy_symlink: @@ -258,7 +258,7 @@ def copyfile(src, dst, force=True, vars=None, force_copy_symlink=False, log.info(f"{dst} is up to date.") return dst - log.info(f"Copying file {src} to {dst}.") + log.info(f"Copying file\n {src} to\n {dst}.") shutil.copy2(src, dst) if make_writable_by_owner: make_file_writable_by_owner(dst) @@ -276,13 +276,13 @@ def copyfile(src, dst, force=True, vars=None, force_copy_symlink=False, if os.path.exists(link_name): if (os.path.islink(link_name) and os.readlink(link_name) == link_target): - log.info(f"Symlink {link_name} -> {link_target} already exists.") + log.info(f"Symlink already exists\n {link_name} ->\n {link_target}") return dst os.remove(link_name) - log.info(f"Symlinking {link_name} -> {link_target} in {target_dir}.") + log.info(f"Symlinking\n {link_name} ->\n {link_target} in\n {target_dir}") os.symlink(link_target, link_name) except OSError: - log.error(f"{link_name} -> {link_target}: Error creating symlink") + log.error(f"Error creating symlink\n {link_name} ->\n {link_target}") finally: os.chdir(current_directory) else: @@ -323,11 +323,11 @@ def copydir(src, dst, filter=None, ignore=None, force=True, recursive=True, vars ignore[i] = ignore[i].format(**vars) if not os.path.exists(src) and not force: - log.info(f"**Skipping copy tree {src} to {dst}. Source does not exists. " + log.info(f"**Skipping copy tree\n {src} to\n {dst}\n Source does not exist. " f"filter={filter}. ignore={ignore}.") return [] - log.info(f"Copying tree {src} to {dst}. filter={filter}. ignore={ignore}.") + log.info(f"Copying tree\n {src} to\n {dst}. filter={filter}. ignore={ignore}.") names = os.listdir(src) |