tools: print both errors and informational messages to stderr.
This script is now also used for usage where we pipe directly the stdout output to a file, e.g. now to create the gimprc man page. We don't want to have INFO messages in there.
This commit is contained in:
parent
f3b14044e4
commit
d2f6867064
1 changed files with 11 additions and 11 deletions
|
|
@ -16,7 +16,7 @@ try:
|
|||
GIMP3_DIRECTORY = os.path.join(GIMP_GLOBAL_BUILD_ROOT, f".GIMP3-build-config-{suffix}")
|
||||
os.makedirs(GIMP3_DIRECTORY, mode=0o700, exist_ok=False)
|
||||
os.environ["GIMP3_DIRECTORY"] = GIMP3_DIRECTORY
|
||||
print(f"INFO: temporary GIMP configuration directory: {GIMP3_DIRECTORY}")
|
||||
sys.stderr.write(f"INFO: temporary GIMP configuration directory: {GIMP3_DIRECTORY}")
|
||||
|
||||
# Earlier code used to set DYLD_LIBRARY_PATH environment variable instead, but
|
||||
# it didn't work on contributor's builds because of System Integrity
|
||||
|
|
@ -61,10 +61,10 @@ try:
|
|||
os.environ["PATH"] = tmp_path + os.pathsep + os.environ.get("PATH", "")
|
||||
|
||||
if "GIMP_DEBUG_SELF_WRAPPER" in os.environ and shutil.which("gdb"):
|
||||
print(f"RUNNING: {os.environ['GIMP_DEBUG_SELF_WRAPPER']} --batch -x {os.environ['GIMP_GLOBAL_SOURCE_ROOT']}/tools/debug-in-build-gimp.py --args {os.environ['GIMP_SELF_IN_BUILD']} {' '.join(sys.argv[1:])}")
|
||||
sys.stderr.write(f"RUNNING: {os.environ['GIMP_DEBUG_SELF_WRAPPER']} --batch -x {os.environ['GIMP_GLOBAL_SOURCE_ROOT']}/tools/debug-in-build-gimp.py --args {os.environ['GIMP_SELF_IN_BUILD']} {' '.join(sys.argv[1:])}")
|
||||
subprocess.run([os.environ["GIMP_DEBUG_SELF_WRAPPER"],"--return-child-result","--batch","-x",f"{os.environ['GIMP_GLOBAL_SOURCE_ROOT']}/tools/debug-in-build-gimp.py","--args", os.environ["GIMP_SELF_IN_BUILD"]] + sys.argv[1:], stdin=sys.stdin, check=True)
|
||||
else:
|
||||
print(f"RUNNING: {os.environ['GIMP_SELF_IN_BUILD']} {' '.join(sys.argv[1:])}")
|
||||
sys.stderr.write(f"RUNNING: {os.environ['GIMP_SELF_IN_BUILD']} {' '.join(sys.argv[1:])}")
|
||||
subprocess.run([os.environ["GIMP_SELF_IN_BUILD"]] + sys.argv[1:],stdin=sys.stdin, check=True)
|
||||
|
||||
if sys.platform not in ['win32', 'cygwin'] and different_python:
|
||||
|
|
@ -86,26 +86,26 @@ try:
|
|||
# that it's inside inside the project build's root.
|
||||
if "GIMP3_DIRECTORY" in os.environ and os.path.isdir(GIMP3_DIRECTORY):
|
||||
if os.path.islink(GIMP3_DIRECTORY):
|
||||
print(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) should not be a symlink.")
|
||||
sys.stderr.write(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) should not be a symlink.")
|
||||
sys.exit(1)
|
||||
used_dir_prefix = str(Path(GIMP3_DIRECTORY).resolve().as_posix())[:-6]
|
||||
tmpl_dir_prefix = f"{Path(os.environ['GIMP_GLOBAL_BUILD_ROOT']).resolve().as_posix()}/.GIMP3-build-config-"
|
||||
if used_dir_prefix != tmpl_dir_prefix:
|
||||
print(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) should be under the build directory with a specific prefix.")
|
||||
print(f' "{used_dir_prefix}" != "{tmpl_dir_prefix}"')
|
||||
sys.stderr.write(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) should be under the build directory with a specific prefix.")
|
||||
sys.stderr.write(f' "{used_dir_prefix}" != "{tmpl_dir_prefix}"')
|
||||
sys.exit(1)
|
||||
print(f"INFO: Running: shutil.rmtree({GIMP3_DIRECTORY})")
|
||||
sys.stderr.write(f"INFO: Running: shutil.rmtree({GIMP3_DIRECTORY})")
|
||||
shutil.rmtree(GIMP3_DIRECTORY)
|
||||
elif not os.access(GIMP3_DIRECTORY, os.W_OK):
|
||||
print(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) does not belong to the user")
|
||||
sys.stderr.write(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) does not belong to the user")
|
||||
sys.exit(1)
|
||||
else:
|
||||
print(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) is not a directory")
|
||||
sys.stderr.write(f"ERROR: $GIMP3_DIRECTORY ({GIMP3_DIRECTORY}) is not a directory")
|
||||
sys.exit(1)
|
||||
|
||||
except subprocess.CalledProcessError as e:
|
||||
print(f"Command failed with exit code {e.returncode}: {e.cmd}")
|
||||
sys.stderr.write(f"Command failed with exit code {e.returncode}: {e.cmd}")
|
||||
sys.exit(e.returncode)
|
||||
except Exception as e:
|
||||
print(f"Error: {str(e)}")
|
||||
sys.stderr.write(f"Error: {str(e)}")
|
||||
sys.exit(1)
|
||||
|
|
|
|||
Loading…
Reference in a new issue