Re: [PATCH 02/10] syntax-check: Exclude 'scripts/*.py' from 'sc_prohibit_magic_number_exit'

26 Jan 2024


      On Thu, Jan 25, 2024 at 08:34:40 -0800, Andrea Bolognani wrote:
...
On Tue, Jan 16, 2024 at 05:12:36PM +0100, Peter Krempa wrote:
...
+++ b/build-aux/syntax-check.mk
@@ -1516,6 +1516,9 @@ exclude_file_name_regexp--sc_black = \
 exclude_file_name_regexp--sc_spacing-check = \
   ^scripts/rpcgen/tests/test_demo\.[ch]$$
+exclude_file_name_regexp--sc_prohibit_magic_number_exit = \
+  ^scripts/.*\.py$$
If you use sys.exit() instead of exit(), like the other Python
scripts are doing, you don't need this anymore.
Oh, I had to look up the difference and 'exit()' is allegedly meant for
interactive use, while scripts should use sys.exit(). TIL.

    