[FIX] Fixed bug where nm would add '@' on functions calls on linux. Also fixed that you couldn't select --no-tests when not using fillit

Signed-off-by: Jules Lasne (seluj78) <jlasne@student.42.fr>

Author: Seluj78

42PyChecker.py

@@ -51,12 +51,6 @@ def check_args_rules(parser, args):
     # disabled, no matter the project.
     if args.project == "other" and args.no_tests:
         parser.error("`--no-tests' Can only be applied on projects, not when 'other' is selected.")
-    if args.no_tests:
-        args.no_libftest = True
-        args.no_maintest = True
-        args.no_moulitest = True
-        args.no_libft_unit_test = True
-        args.no_fillit_checker = True
 
     if args.no_author and args.project == "42commandements":
         parser.error("`--no-author' Can only be applied on project, but not on 42commandements.")
@@ -105,6 +99,12 @@ def check_args_rules(parser, args):
     if args.no_fillit_checker and args.project != "fillit":
         parser.error("`--no-fillit-checker' can only be applied if fillit is selected"
                      " with `--project'")
+    if args.no_tests:
+        args.no_libftest = True
+        args.no_maintest = True
+        args.no_moulitest = True
+        args.no_libft_unit_test = True
+        args.no_fillit_checker = True
 
 
 def print_header():

PyChecker/utils/forbidden_functions.py

@@ -48,7 +48,12 @@ def check(project_path: str, authorized_func, root_path: str):
         sys_calls = ['Error']
 
     sys_calls = [item for item in sys_calls if not item.startswith("ft_")]
+    if platform.system() == "Linux":
+        for i, calls in enumerate(sys_calls):
+            sys_calls[i] = calls.split('@', 1)[0]
     extra_function_call = [item for item in sys_calls if item not in authorized_func]
+    print(sys_calls)
+    print(authorized_func)
     with open(root_path + "/.myforbiddenfunctions", 'w+') as file:
         for item in extra_function_call:
             # This is to ignore functions like `__stack_chk_fail'