Skip to content
Snippets Groups Projects
Commit e1c33698 authored by Campbell Barton's avatar Campbell Barton
Browse files

GNUmakefile: prefer "check_" prefix instead of "test_"

- Reserve "test" for tests & testing frameworks.
- Add "check_mypy" to "make help" text.
- Output to the standard output instead of redirecting to log-files,
  leave redirecting output log-files to the user running the command.
parent 4c423ccb
No related branches found
Tags
No related merge requests found
...@@ -51,12 +51,6 @@ Testing Targets ...@@ -51,12 +51,6 @@ Testing Targets
* test: * test:
Run automated tests with ctest. Run automated tests with ctest.
* test_cmake:
Runs our own cmake file checker
which detects errors in the cmake file list definitions
* test_pep8:
Checks all python script are pep8
which are tagged to use the stricter formatting
Static Source Code Checking Static Source Code Checking
Not associated with building Blender. Not associated with building Blender.
...@@ -72,6 +66,10 @@ Static Source Code Checking ...@@ -72,6 +66,10 @@ Static Source Code Checking
using one of the accepted licenses in 'doc/license/SPDX-license-identifiers.txt' using one of the accepted licenses in 'doc/license/SPDX-license-identifiers.txt'
Append with 'SHOW_HEADERS=1' to show all unique headers Append with 'SHOW_HEADERS=1' to show all unique headers
which can be useful for spotting license irregularities. which can be useful for spotting license irregularities.
* check_cmake: Runs our own cmake file checker which detects errors in the cmake file list definitions.
* check_pep8: Checks all Python script are pep8 which are tagged to use the stricter formatting.
* check_mypy: Checks all Python scripts using mypy,
see: source/tools/check_source/check_mypy_config.py scripts which are included.
Spell Checkers Spell Checkers
This runs the spell checker from the developer tools repositor. This runs the spell checker from the developer tools repositor.
...@@ -399,16 +397,6 @@ package_archive: .FORCE ...@@ -399,16 +397,6 @@ package_archive: .FORCE
test: .FORCE test: .FORCE
@$(PYTHON) ./build_files/utils/make_test.py "$(BUILD_DIR)" @$(PYTHON) ./build_files/utils/make_test.py "$(BUILD_DIR)"
# run pep8 check check on scripts we distribute.
test_pep8: .FORCE
@$(PYTHON) tests/python/pep8.py > test_pep8.log 2>&1
@echo "written: test_pep8.log"
# run some checks on our CMAKE files.
test_cmake: .FORCE
@$(PYTHON) build_files/cmake/cmake_consistency_check.py > test_cmake_consistency.log 2>&1
@echo "written: test_cmake_consistency.log"
# ----------------------------------------------------------------------------- # -----------------------------------------------------------------------------
# Project Files # Project Files
...@@ -486,7 +474,6 @@ check_descriptions: .FORCE ...@@ -486,7 +474,6 @@ check_descriptions: .FORCE
@$(BLENDER_BIN) --background -noaudio --factory-startup --python \ @$(BLENDER_BIN) --background -noaudio --factory-startup --python \
"$(BLENDER_DIR)/source/tools/check_source/check_descriptions.py" "$(BLENDER_DIR)/source/tools/check_source/check_descriptions.py"
# run deprecation tests, see if we have anything to remove.
check_deprecated: .FORCE check_deprecated: .FORCE
@PYTHONIOENCODING=utf_8 $(PYTHON) \ @PYTHONIOENCODING=utf_8 $(PYTHON) \
source/tools/check_source/check_deprecated.py source/tools/check_source/check_deprecated.py
...@@ -496,6 +483,14 @@ check_licenses: .FORCE ...@@ -496,6 +483,14 @@ check_licenses: .FORCE
"$(BLENDER_DIR)/source/tools/check_source/check_licenses.py" \ "$(BLENDER_DIR)/source/tools/check_source/check_licenses.py" \
"--show-headers=$(SHOW_HEADERS)" "--show-headers=$(SHOW_HEADERS)"
check_pep8: .FORCE
@PYTHONIOENCODING=utf_8 $(PYTHON) \
tests/python/pep8.py
check_cmake: .FORCE
@PYTHONIOENCODING=utf_8 $(PYTHON) \
build_files/cmake/cmake_consistency_check.py
# ----------------------------------------------------------------------------- # -----------------------------------------------------------------------------
# Utilities # Utilities
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment