1. Shindig
  2. SHINDIG-1691

Maven project enhancements to improve embedding and extending shindig-server


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.0-beta1
    • Fix Version/s: 2.5.0-beta1
    • Component/s: Build, Java
    • Labels:


      As proposed on dev@shindig mailing list before, this issue is about enhancements for the Maven build configuration and modules to improve embedding and extending shindig-server for downstream projects like Apache Rave and further.

      The full proposal thread can be found here:

      In short, my proposal is the following, and I'll post a patch for these changes up for review shortly:

      a) Move java/server/src/main/java/* (sample container code)
      to new jar module java/sample-container
      b) Move java/server/src/main/webapps/*
      to new (shallow) war module java/server-resources
      c) Move resources currently merged into server module from ./../content and
      ../../config also into new (shallow) war module java/server-resources
      d) Add new pom module java/server-dependencies which (only) defines
      the runtime dependencies needed for the shindig-server module
      Note: this should also add new dependency on shindig-sample-container
      e) The remainder of the shindig-server module then only needs to have
      dependencies on the shindig-server-resources (war overlay) and
      shindig-server-dependencies (pom), as well as some needed test only
      dependencies for the remaining "endtoend" unit-tests

      The end result of the above will again be a shindig-server war equal to the current shindig-server war but now downstream users can more easily embed and extend it, with proper maven (transitive) dependency resolution support.

      The above changes really are quite trivial and will have zero impact for development and current end usages.



          • Assignee:
            Ate Douma
          • Votes:
            0 Vote for this issue
            0 Start watching this issue


            • Created: