Add Maven, CI (#14)

- Lower required cmake version from 3.9 to 3.0
- Adds unit test (disabled for cross-compiles)
- Move version info into pom.xml; parse using cmake; #cmakedefine
- Output natives to target/cmake/natives/
- Handle MSVC build flags
- Add cross-compilation toolchains for arm, ppc, mingw
- Add Travis-CI build matrix
- Adds ability to build a one-off jar for patching/distribution

Author: tresf

.gitignore

@@ -1,4 +1,6 @@
 build/
+# the maven convention for built files
+target/
 
 # Prerequisites
 *.d
@@ -13,11 +15,6 @@ build/
 *.gch
 *.pch
 
-# Compiled Dynamic libraries
-*.so
-*.dylib
-*.dll
-
 # Fortran module files
 *.mod
 *.smod
@@ -32,3 +29,10 @@ build/
 *.exe
 *.out
 *.app
+
+# IDE files: IntellIJ
+.idea/
+*.iml
+
+# IDE files: eclipse
+.settings/

.travis.yml

@@ -0,0 +1,75 @@
+language: java
+
+cache:
+  directories:
+    - $HOME/.m2
+    - $HOME/.jabba/jdk
+
+before_cache:
+  - rm -rf $HOME/.m2/repository/com/github/java-native
+
+matrix:
+  include:
+    - os: linux
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++
+    - os: linux
+      env: PROFILE=m32
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-multilib
+    - os: linux
+      env: PROFILE=mingw32
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-mingw-w64-i686
+    - os: linux
+      env: PROFILE=mingw64
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-mingw-w64-x86-64
+    - os: linux
+      env: PROFILE=armsf,armtrusty
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-arm-linux-gnueabi
+    - os: linux
+      env: PROFILE=aarch64
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-aarch64-linux-gnu
+    - os: linux
+      env: PROFILE=ppc64
+      addons:
+        apt:
+          packages:
+            - cmake3
+            - g++-powerpc64le-linux-gnu
+    - os: osx
+      addons:
+        homebrew:
+          packages:
+            - cmake
+
+install:
+  - mvn dependency:resolve
+
+script: if [ -z "$PROFILE" ]; then mvn --batch-mode; else mvn -P "$PROFILE" --batch-mode; fi
+
+after_success:
+  - bash <(curl -s https://codecov.io/bash)
+
+

CMakeLists.txt

@@ -1,5 +1,8 @@
-cmake_minimum_required(VERSION 3.9)
-project(jssc VERSION 2.8.0 DESCRIPTION "Java Simple Serial Connector")
+cmake_minimum_required(VERSION 3.0)
+cmake_policy(SET CMP0048 NEW)
+cmake_policy(SET CMP0042 NEW)
+
+project(jssc VERSION 2.7.1 LANGUAGES CXX)
 
 find_package(Java)
 find_package(JNI)
@@ -56,15 +59,60 @@ else()
 	set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} -O2")
 endif()
 
-add_library(jssc SHARED
-    src/cpp/${JSSC_PLATFORM}/jssc.cpp
-)
+# Detect platform if -DNATIVE_LIB_DIR is not provided
+# TODO: Handle arm, hardfloat, etc
+if(NOT NATIVE_LIB_DIR)
+	# windows, linux, darwin, etc
+	string(TOLOWER "${CMAKE_SYSTEM_NAME}" OS_NAME)
+	if(OS_NAME MATCHES "darwin")
+		set(OS_NAME "osx")
+	endif()
+
+	# 32-bit or 64-bit
+	#FIXME: Might fail on cross-compile
+	if("${CMAKE_SIZEOF_VOID_P}" EQUAL "8")
+		set(OS_BITS 64)
+	else()
+		set(OS_BITS 32)
+	endif()
+	SET(NATIVE_LIB_DIR ${OS_NAME}_${OS_BITS})
+endif()
+set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/natives/${NATIVE_LIB_DIR})
+
+# version.h using #cmakedefine for version from pom.xml.
+set(JSSC_VERSION "0.0.0-UNKNOWN")
+file(STRINGS ${CMAKE_CURRENT_SOURCE_DIR}/pom.xml POM_FILE)
+foreach(POM_LINE ${POM_FILE})
+	# Assume first "<version>" is the project version
+	if(POM_LINE MATCHES "<version>")
+		string(REGEX REPLACE "^[ \t]+|<[^>]*>" "" DETECTED_VERSION "${POM_LINE}")
+		string(STRIP "${DETECTED_VERSION}" DETECTED_VERSION)
+		if(DETECTED_VERSION STREQUAL "")
+			MESSAGE(WARNING "Could not parse version from pom.xml, defaulting to \"${JSSC_VERSION}\"")
+		else()
+			SET(JSSC_VERSION "${DETECTED_VERSION}")
+			MESSAGE(STATUS "Found version \"${JSSC_VERSION}\" in pom.xml")
+		endif()
+		break()
+	endif()
+endforeach()
+configure_file(src/cpp/version.h.in ${CMAKE_CURRENT_BINARY_DIR}/version.h @ONLY)
+
+add_library(jssc SHARED src/cpp/${JSSC_PLATFORM}/jssc.cpp)
 
-target_include_directories(jssc PRIVATE ${JNI_INCLUDE_DIRS} ${JSSC_ADDITIONAL_INCLUDES})
+target_include_directories(jssc PRIVATE ${JNI_INCLUDE_DIRS} ${CMAKE_CURRENT_BINARY_DIR} ${JSSC_ADDITIONAL_INCLUDES})
 
-set_target_properties(jssc PROPERTIES VERSION ${PROJECT_VERSION})
-set_target_properties(jssc PROPERTIES PUBLIC_HEADER src/jssc_SerialNativeInterface.h)
+set_target_properties(jssc PROPERTIES PUBLIC_HEADER ${CMAKE_CURRENT_BINARY_DIR}/jssc_SerialNativeInterface.h)
 set_target_properties(jssc PROPERTIES POSITION_INDEPENDENT_CODE ON)
+if(WIN32)
+	# Fix paths for MSVC (Debug/Release) and MINGW
+	set_target_properties(jssc PROPERTIES RUNTIME_OUTPUT_DIRECTORY_DEBUG "${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
+	set_target_properties(jssc PROPERTIES RUNTIME_OUTPUT_DIRECTORY_RELEASE "${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
+endif()
+if(FORCE_M32)
+	# Build 32-bit binary on Linux
+	set_target_properties(jssc PROPERTIES COMPILE_FLAGS -m32 LINK_FLAGS -m32)
+endif()
 
 # Call strip on non-debug builds
 if(CMAKE_STRIP AND NOT CMAKE_BUILD_TYPE MATCHES "Deb")
@@ -73,3 +121,12 @@ if(CMAKE_STRIP AND NOT CMAKE_BUILD_TYPE MATCHES "Deb")
 	endif()
 	add_custom_command(TARGET jssc POST_BUILD COMMAND "${CMAKE_STRIP}" ${STRIP_ARGS} $<TARGET_FILE:jssc>)
 endif()
+
+# Handle compiler warnings
+if(MSVC)
+	#TODO Treat warnings as errors /WX
+	target_compile_options(jssc PRIVATE /W4)
+else()
+	#TODO Treat warnings as errors -Werror
+	target_compile_options(jssc PRIVATE -Wall -Wextra -pedantic)
+endif()