Cleanups

* Move headers to include
* Auto detect c++17 support

Author: Shachar

Makefile.am

@@ -1,5 +1,7 @@
 SUBDIRS=lib
 
+include_HEADERS = include/practical-sa.h
+
 ut:
 	$(MAKE) -C lib ut
 

configure.ac

@@ -7,6 +7,8 @@ AC_CONFIG_HEADERS([config.h])
 AM_INIT_AUTOMAKE([foreign subdir-objects])
 LT_INIT
 
+m4_include([m4/ax_cxx_compile_stdcxx.m4])
+
 AC_LANG(C++)
 # Checks for programs.
 AC_PROG_CXX
@@ -23,9 +25,11 @@ PKG_PROG_PKG_CONFIG
 PKG_CHECK_MODULES([CPPUNIT], [cppunit], , [AC_MSG_FAILURE([CppUnit module not found])])
 
 # Checks for header files.
+AC_SUBST([AM_CPPFLAGS], [-I\$\(top_srcdir\)/include])
 
 # Checks for typedefs, structures, and compiler characteristics.
 AC_TYPE_SIZE_T
+AX_CXX_COMPILE_STDCXX(17, [noext], [mandatory])
 
 # Checks for library functions.
 

lib/practical-sa.h include/practical-sa.h

@@ -3,10 +3,11 @@ noinst_PROGRAMS = practical-sa-ut practiparse
 
 libpractical_sa_la_LDFLAGS = -version-info 0:0:0
 libpractical_sa_la_SOURCES = tokenizer.cpp parser.cpp scope_tracing.cpp semantic_analyzer.cpp
-include_HEADERS = practical-sa.h
 
 practical_sa_ut_SOURCES = ut_runner.cpp slice_ut.cpp tokenizer_ut.cpp tokenizer.cpp
-practical_sa_ut_CPPFLAGS = -I. # Otherwise we can't use the same CPP files for both library and UT
+# We need automake to compile cpp files for the UTs distinctly than for the library. We do this by adding a useless compile flag
+# that applies only to the UTs executable. Otherwise we can't use the same CPP files for both library and UT
+practical_sa_ut_CPPFLAGS = -I.
 practical_sa_ut_LDADD = @CPPUNIT_LIBS@
 practical_sa_ut_CFLAGS = @CPPUNIT_CFLAGS@ $(AM_CFLAGS)