Skip to content

Commit

Permalink
Run roundtrip on make check
Browse files Browse the repository at this point in the history
Adding check-no-roundtrip is done by @masatake.
NOTE: We cannot find the way to ru roundtrip on msys2.
See msys2/MSYS2-packages#84 .
  • Loading branch information
b4n authored and masatake committed Jan 5, 2018
1 parent f4e04d5 commit 4b27a5d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
9 changes: 8 additions & 1 deletion makefiles/testing.mak
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
# -*- makefile -*-
.PHONY: check units fuzz noise tmain tinst clean-units clean-tmain clean-gcov run-gcov codecheck cppcheck dicts cspell

check: tmain units
chekc-no-roundtrip: tmain units
check: tmain units roundtrip

clean-local: clean-units clean-tmain

Expand Down Expand Up @@ -159,10 +160,16 @@ tinst:
#
# Test readtags
#
if USE_READCMD
roundtrip: $(READ_TEST)
$(V_RUN) \
builddir=$$(pwd); \
$(SHELL) $(srcdir)/misc/roundtrip $(READ_TEST) $${builddir}/Units
else
# apparently Automake doesn't like adding an extra dependency in a conditional,
# so add it unconditionally and simply have it not do anything if it's disabled.
roundtrip:
endif

#
# Checking code in ctags own rules
Expand Down
4 changes: 2 additions & 2 deletions win32/appveyor.bat
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ c:\cygwin64\bin\file readtags.exe
.\ctags --version || exit 1

:: Run tests on msys2
bash -lc "make check APPVEYOR=1"
bash -lc "make check-no-roundtrip APPVEYOR=1"

@echo off
goto :eof
Expand Down Expand Up @@ -171,7 +171,7 @@ if "%normalbuild%-%ARCH%"=="yes-x64" (
@echo Tests for msys2 x64 are skipped.
exit 0
)
bash -lc "make check APPVEYOR=1"
bash -lc "make check-no-roundtrip APPVEYOR=1"

@echo off
goto :eof
Expand Down

0 comments on commit 4b27a5d

Please sign in to comment.