rendergraph: Port perl script to Python and add pre-commit hook

[Holzhaus]

No description provided.

[acolombier]

Thanks for looking into this! The python version is definitely more beefy but also should support wider system setup, as well as provide logging a command usage.

[acolombier]

Very naive suggestion here, to help flattening the nested logic

Suggested change

	if in_shader_block == 2:
	if re.match(r"^\*\*", line):
	ok = True
	else:
	if not comment_added and not re.match(r"^#", line):
	yield "//// GENERATED - EDITS WILL BE OVERWRITTEN"
	comment_added = True
	if line:
	if buffered_blank_line:
	yield ""
	buffered_blank_line = False
	yield line
	else:
	buffered_blank_line = True
	elif in_shader_block == 1:
	if line.rstrip() == "Contents:":
	in_shader_block = 2
	else:
	if line.rstrip() == "Shader 1: GLSL 120 [Standard]":
	in_shader_block = 1
	if in_shader_block == 2:
	if re.match(r"^\*\*", line):
	ok = True
	else:
	if not comment_added and not re.match(r"^#", line):
	yield "//// GENERATED - EDITS WILL BE OVERWRITTEN"
	comment_added = True
	if line:
	if buffered_blank_line:
	yield ""
	buffered_blank_line = False
	yield line
	else:
	buffered_blank_line = True
	elif in_shader_block == 1 and line.rstrip() == "Contents:":
	in_shader_block = 2
	elif line.rstrip() == "Shader 1: GLSL 120 [Standard]":
	in_shader_block = 1

[acolombier]

Should we use $PATH as well?

Suggested change

	additional_paths=[
	"/usr/lib/qt6/bin",
	"/lib/qt6/bin",
	"/usr/local/lib/qt6/bin",
	],
	additional_paths=[
	"/usr/lib/qt6/bin",
	"/lib/qt6/bin",
	"/usr/local/lib/qt6/bin",
	] + [os.getenv("PATH", "").split(":")],

[Holzhaus]

We call shutil.which up to two times in the funtion, the first time without setting the path argument which implicitly defaults to $PATH. So yeah, that is already supported.

[acolombier]

Ah good to know. Thanks for the lesson!

[m0dB]

Thanks @Holzhaus !