Add Maven, CI #14
Merged
Add Maven, CI #14
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…l cmake installation.
…duce cmake min version to 2.7 for now (b/c of travis-cli).
…g option for clarity.
- Adds -DNATIVE_LIB_DIR option - Adds boilerplate mingw profiles - Fixes building with MSVC
Completely removes cmake-maven plugin in favor of calling cmake directly, simplifying build system.
* Change maven id and project name to just "jssc" at "io.github.java-native" namespace * Adds two releasable jars (platform-specific jar and uber jar). TODO: add non-shaded fat jar #21 * Allow cmake to parse version info from pom.xml * Adds ability to skip native compilation steps including cmake and tests * Leverages maven-assembly-plugin for better classifier to the default artifact * Alphabetize, organize and remove hard-coding of dependencies/variables * Adds Travis-CI for many platforms. Note, some binaries are not yet usable, sorry! * Adds Ubuntu 14.04 cross-compiled binaries to the source tree. TODO: Move to git-lfs * Add LICENSE to pom.xml (#22)
|
This PR will be merged back to master very soon (days, not weeks). Items left are some decisions on cross-compiling (see #26 as an example) as well as where to permanently stash these binary files Once these remaining items are ironed out, we can merge back to master. Note, if there's any major stalling on #26, I'll merge with it still open. |
* Updated default artifacts. Fixes #31. - Default artifact is now the "global" jar containing all precompiled libraries. - The system-classifier jar is now an optional jar which is not generated by the package profile. - The system-classifier jar contains everything from the "global" jar, but with the system-classifier updated library. - Updated cmake output folder to `${cmake.generated.directory}`. - Removed MAVEN_OUTPUT_DIRECTORY from CMakeLists.txt.
@bmhm and I have agreed to keep Merging. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds:
How to build:
This is a work-in-progress. I'm placing the PR here in the open so that onlookers are aware of its existence and progress.
PR's against
mavenbranch are always welcome. Contributors can commit directly. If you want to be a contributor, just ask.Don't know maven? No problem! Compilation is painless, check out the wiki: https://github.com/java-native/jssc/wiki/Compiling
Also addresses:
scream3r#103
scream3r#59