MultiInv icon indicating copy to clipboard operation
MultiInv copied to clipboard

Partial out-of-the-box build support

Open RoyCurtis opened this issue 8 years ago • 0 comments

As-is, MultiInv is impossible to build from a cloned repository. This is because of missing required libraries and missing POM file entries. This makes it difficult for people to contribute changes and bug fixes.

This PR fixes the POM file and includes jar files for almost all the required libraries. This allows other contributors to be able to clone the repository and build from Maven or IDE without issues.

Changes

  • Included required jar files as a local Maven repository (see http://randomizedsort.blogspot.co.uk/2011/10/configuring-maven-to-use-local-library.html )
  • Removed Spout repo and dependency (unused, broken)
  • Changed minimum code level to Java 1.7 (fixes IDE error with @Override in interfaces)
  • Fixed version variable in generated JAR manifest

Testing

  • Successfully built MultiInv.jar from IntelliJ 14 using mvn clean package
  • Built jar not yet tested in server environment

Considerations

  • This commit increases the download size of the repository by at least 2MB
  • TuxTwoLib has been left out until it is ready for distribution, as per Tux2's wishes. When ready:
    • "Install" the lib to the local repo using this command in MultiInv's directory: mvn install:install-file -Dfile=path/to/TuxTwoLib.jar -DgroupId=Tux2.TuxTwoLib -DartifactId=TuxTwoLib -Dversion=bBUILDNUMBERHERE -Dpackaging=jar -DlocalRepositoryPath=lib
    • Update the <version>b7</version> tag in pom.xml for TuxTwoLib

RoyCurtis avatar Apr 10 '16 21:04 RoyCurtis