Enhancement: Add time reporting for make.py

[pushfoo]

What to add?

Add basic time elapsed output for each commands in ./make.py

Why?

1. Make it easier for new contributors to report performance
2. Remove cross-platform ambiguities

Current Behavior

build succeeded, 5 warnings.

The HTML pages are in build/html.
>> Command finished: sphinx-build --jobs auto -b html -d build/doctrees doc build/html 


Build finished. The HTML pages are in /home/name/Projects/arcade/build/html.

Potential Desired Behavior

build succeeded, 5 warnings.

The HTML pages are in build/html.
>> Command finished (9m 56s): sphinx-build --jobs auto -b html -d build/doctrees doc build/html 


Build finished. The HTML pages are in /home/name/Projects/arcade/build/html.

[alanJoshiGeorge]

Hi! I'm new to open source and would love to work on this issue as my first contribution. Could I take it on?

[einarf]

Hi! I'm new to open source and would love to work on this issue as my first contribution. Could I take it on?

Definitely go ahead.

[pushfoo]

@alanJoshiGeorge 👍 This seems the best place to do it:

arcade/make.py

Lines 134 to 157 in 4d8c653

	def run(args: str | list[str], cd: PathLike | None = None) -> None:
	"""
	Try to run `args` with subprocess, switching into & out of `cd` if provided.
	Switching back out should occur regardless of any exceptions, unless the
	interpreter crashes.
	Args:
	args: the command to run.
	cd: a directory to switch into beforehand, if any.
	"""
	cmd = " ".join(args)
	print(">> Running command:", cmd)
	if cd is not None:
	with cd_context(_resolve(cd, strict=True)):
	result = subprocess.run(args)
	else:
	result = subprocess.run(args)
	print(">> Command finished:", cmd, "\n")
	# TODO: Should we exit here? Or continue to let other commands run also?
	if result.returncode != 0:
	exit(result.returncode)

[csd4ni3l]

He made this PR: #2724

[einarf]

merged