Build: add JS lint target

https://fedorahosted.org/freeipa/ticket/6418

Reviewed-By: Martin Basti <mbasti@redhat.com>
Reviewed-By: Christian Heimes <cheimes@redhat.com>
This commit is contained in:
Petr Spacek 2016-11-08 09:24:51 +01:00 committed by Martin Babinsky
parent 14c1c8dfd0
commit f31a489d24
2 changed files with 40 additions and 1 deletions

View File

@ -49,7 +49,10 @@ rpms: dist-gzip rpmroot rpmdistdir
if WITH_PYLINT
PYLINT_TARGET = pylint
endif WITH_PYLINT
lint: $(PYLINT_TARGET)
if WITH_JSLINT
JSLINT_TARGET = jslint
endif WITH_JSLINT
lint: $(PYLINT_TARGET) $(JSLINT_TARGET)
.PHONY: pylint
pylint:
@ -66,3 +69,24 @@ pylint:
echo "Pylint is running, please wait ..."; \
PYTHONPATH=$(top_srcdir) $(PYLINT) \
--rcfile=$(top_srcdir)/pylintrc $${FILES}
.PHONY: jslint jslint-ui jslint-ui-test jslint-html
jslint: jslint-ui jslint-ui-test jslint-html
# create temporary symlinks to allow jslint to find libs/loader.js
jslint-ui:
cd $(top_srcdir)/install/ui; \
$(MKDIR_P) js; \
rm -f js/libs; \
$(LN_S) ../src/libs js/libs; \
jsl -nologo -nosummary -nofilelisting -conf jsl.conf; \
rm js/libs; \
rmdir js
jslint-ui-test:
cd $(top_srcdir)/install/ui/test; \
jsl -nologo -nosummary -nofilelisting -conf jsl.conf
jslint-html:
cd $(top_srcdir)/install/html; \
jsl -nologo -nosummary -nofilelisting -conf jsl.conf

View File

@ -427,6 +427,21 @@ fi
AC_SUBST([PYLINT])
AM_CONDITIONAL([WITH_PYLINT], [test "x${PYLINT}" != "xno"])
AC_ARG_WITH([jslint],
AS_HELP_STRING([--with-jslint=path to jsl],
[path to JavaScript lint]),
dnl --without-jslint will set JSLINT=no
[JSLINT=$with_jslint],
[AC_PATH_PROG([JSLINT], [jsl])]
)
if test "x${JSLINT}" == "x"; then
AC_MSG_ERROR([cannot find JS lint])
fi
AC_SUBST([JSLINT])
AM_CONDITIONAL([WITH_JSLINT], [test "x${JSLINT}" != "xno"])
# Flags
AC_SUBST(CFLAGS)