Skip to content

Commit

Permalink
Merge pull request #2924 from cgwalters/drop-syntax-check
Browse files Browse the repository at this point in the history
build: Drop `make syntax-check`
  • Loading branch information
dbnicholson authored Jul 11, 2023
2 parents 57fe33f + 6591210 commit c0c2c9b
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 1,439 deletions.
1 change: 0 additions & 1 deletion .cci.jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ stage("Build") {
try {
shwrap("""
make check
make syntax-check
""")
} finally {
shwrap("cat test-suite.log || true")
Expand Down
4 changes: 0 additions & 4 deletions GNUmakefile
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,6 @@ include Makefile
_build-aux ?= build-aux
_autoreconf ?= autoreconf -v

include $(srcdir)/maint.mk

# Ensure that $(VERSION) is up to date for dist-related targets, but not
# for others: rerunning autoreconf and recompiling everything isn't cheap.
_have-git-version-gen := \
Expand Down Expand Up @@ -96,8 +94,6 @@ srcdir = .
_build-aux ?= build-aux
_autoreconf ?= autoreconf -v

include ./maint.mk

ifeq ($(.DEFAULT_GOAL),abort-due-to-no-makefile)
$(MAKECMDGOALS): abort-due-to-no-makefile
endif
Expand Down
1 change: 0 additions & 1 deletion ci/build-check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ ${dn}/build.sh
topdir=$(git rev-parse --show-toplevel)
resultsdir=$(mktemp -d)
make check
make syntax-check # TODO: do syntax-check under check
# See comment below
for x in test-suite.log config.log; do
mv ${x} ${resultsdir}
Expand Down
Loading

0 comments on commit c0c2c9b

Please sign in to comment.