]> git.ipfire.org Git - thirdparty/vim.git/commitdiff
runtime(syntax-tests): Re-introduce support for "phoney" languages
authorAliaksei Budavei <0x000c70@gmail.com>
Fri, 7 Mar 2025 18:12:45 +0000 (19:12 +0100)
committerChristian Brabandt <cb@256bit.org>
Fri, 7 Mar 2025 18:12:45 +0000 (19:12 +0100)
As of patch v9.1.1176~1, there are no longer makeshift Make
targets to accommodate language names that can match phoney
targets.  For example, "clean_" was previously generated for
Clean because otherwise it clashed with the "clean" target.

Additionally, enable test filtering for makeshift targets.

Reference:
https://wiki.clean.cs.ru.nl/Clean

closes: #16810

Signed-off-by: Aliaksei Budavei <0x000c70@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
runtime/syntax/Makefile
runtime/syntax/testdir/runtest.vim

index 6d8019cdeda3ca018beb75101b031947a4b07bbe..b081cde78aa165256008ced0c482d1aeeb768c1c 100644 (file)
@@ -26,6 +26,10 @@ RUN_VIMTEST = VIMRUNTIME=$(VIMRUNTIME) $(VALGRIND) $(ENVVARS) ../$(VIMPROG) -f $
 # Uncomment this line for debugging
 # DEBUGLOG = --log testlog
 
+# All initial phony targets; these names may clash with file types.
+phonies = clean test testclean
+.PHONY: $(phonies)
+
 # Run the tests that didn't run yet or failed previously.
 # If a test succeeds a testdir/done/{name} file will be written.
 # If a test fails a testdir/failed/{name}.dump file will be written.
@@ -46,34 +50,28 @@ test:
 
 # add dependence on syntax files
 testdir/testdeps.mk:
-       ./testdir/tools/maketestdeps > $@
+       ./testdir/tools/maketestdeps $(phonies) > $@
 
 -include testdir/testdeps.mk
 
 clean testclean:
        rm -f testdir/failed/* testdir/done/* testdir/vimcmd testdir/messages testdir/Xtestscript testdir/Xfilter testdir/testdeps.mk
 
-# All initial phony targets; these names may clash with file extensions.
-phonies = clean test testclean
-.PHONY: $(phonies)
-
 # Collect all input filenames and their file types.
 testnames != set +f; \
 awk 'BEGIN { \
-       for (i = 1; i < ARGC; i++) { \
-               name = ARGV[i]; \
-               sub(/.+\//, "", name); \
-               print name; \
-               if (name ~ /.+_.+/) { \
-                       sub(/_.+/, "", name); \
-               } else { \
-                       sub(/.+\./, "", name) \
-               } \
-               if (name == "vim9") name = "vim"; \
-               exts[name]; \
-       } \
-       for (ext in exts) \
-               print ext; \
+    for (i = 1; i < ARGC; i++) { \
+        split(ARGV[i], names, /\//); \
+        split(names[3], parts, /[._]/); \
+        types[parts[1]]; \
+        print names[3]; \
+    } \
+    delete types["vim9"]; \
+    split("$(phonies)", scratch); \
+    for (phony in scratch) \
+        phonies[scratch[phony]]; \
+    for (type in types) \
+        print type ((type in phonies) ? "_" : ""); \
 }' testdir/input/*.*
 
 .PHONY: self-testing $(testnames)
index 34b5caee9117f73540710cb4e5097869862023be..2dc78eca486459e6aa7d354d9a9f4143aefa6792 100644 (file)
@@ -234,7 +234,7 @@ func RunTest()
   " Turn a subset of filenames etc. requested for testing into a pattern.
   let filter = filereadable('../testdir/Xfilter')
     \ ? readfile('../testdir/Xfilter')
-       \ ->map({_, v -> (v =~ '\.' ? '^' : '\.') .. v .. '$'})
+       \ ->map({_, v -> '^' .. substitute(v, '_$', '', '')})
        \ ->join('\|')
     \ : ''