Uploaded image for project: 'Terrier Core'
  1. Terrier Core
  2. TR-187

Maven support

          Details

          • Type: Improvement
          • Status: Resolved
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 3.5
          • Fix Version/s: 4.1
          • Component/s: build
          • Labels:
            None

            Description

            Hi,

            As a user of maven (for the dependency management system), I started to write the pom.xml that can be used for the moment to compile Terrier. It is not a finished one, since parts of the build process (e.g. tests, packaging, ...) are not ported, but by typing

            mvn install

            terrier (and its dependencies) are stored in the local maven repository, and I can use a simple
                
                <dependency>
                  <groupId>org.terrier</groupId>
                  <artifactId>terrier</artifactId>
                  <version>0.3.5</version>
                </dependency>

            in my projects.

            I also had to remove org/terrier/utility/SimpleJettyHTTPServer.java from the compilation (see the pom.xml) since there was no maven entry for jetty 5.1.15 (there is none below version 6). I sometimes selected versions a bit higher than those included with Terrier 3.5. For the moment, everything compiles fine and I will use this bug report if I make improvements to the maven build file.

            Benjamin
                

              Attachments

                Issue Links

                blocks

                Improvement - An improvement or enhancement to an existing feature or task. TR-352 Build .tar.gz and .zip distributions from Maven

                • Major - Major loss of function.
                • Resolved
                is blocked by

                Improvement - An improvement or enhancement to an existing feature or task. TR-190 Upgrade to Jetty 6

                • Minor - Minor loss of function, or other problem where easy workaround is present.
                • Resolved
                is related to

                Task - A task that needs to be done. TR-371 Finalize maven build routine and update bin scripts for the new classpath

                • Major - Major loss of function.
                • Resolved

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  craigm Craig Macdonald added a comment -

                  Thanks Benjamin. It seems very much that everyone is using Maven these days, but I dont have any experience with it. If you could continue to post updates, that would be appreciated!

                  Question, why is version
                  <version>0.3.5</version>
                  ?

                  Craig

                  Show
                  craigm Craig Macdonald added a comment - Thanks Benjamin. It seems very much that everyone is using Maven these days, but I dont have any experience with it. If you could continue to post updates, that would be appreciated! Question, why is version <version>0.3.5</version> ? Craig
                  Hide
                  Permalink
                  bpiwowar Benjamin Piwowarski added a comment - - edited

                  I will definitely do this.

                  <version>0.3.5</version>
                  is a mistake - it should be 3.5 - so used to make beta versions that the zero was typed before I thought of it.

                  By the way, I think it would be fairly easy to switch from ant to maven, but to complete the switch:
                  1) [optional, but better] Follow the maven layout
                  source files in src/main/java
                  antlr files in src/main/antlr
                  test files in src/test/java
                  2) optional, but much better since it is the only missing dependency, upgrade to jetty >= 6

                  benjamin

                  Show
                  bpiwowar Benjamin Piwowarski added a comment - - edited I will definitely do this. <version>0.3.5</version> is a mistake - it should be 3.5 - so used to make beta versions that the zero was typed before I thought of it. By the way, I think it would be fairly easy to switch from ant to maven, but to complete the switch: 1) [optional, but better] Follow the maven layout source files in src/main/java antlr files in src/main/antlr test files in src/test/java 2) optional, but much better since it is the only missing dependency, upgrade to jetty >= 6 benjamin
                  Hide
                  Permalink
                  bpiwowar Benjamin Piwowarski added a comment -

                  Updated pom.xml

                  Show
                  bpiwowar Benjamin Piwowarski added a comment - Updated pom.xml
                  Hide
                  Permalink
                  bpiwowar Benjamin Piwowarski added a comment -

                  Craig,

                  I just updated the pom.xml file with
                  1) antlr support [did not try to remove the java files]
                  2) test support [all the tests passed OK]

                  The pom.xml file is nearly complete (some developer information is missing, see the end of the file).

                  you can try it out yourself, by putting pom.xml into the base folder of terrier:
                  1) mvn compile : download the dependencies and compiles
                  2) mvn surefire:test : runs the tests
                  3) mvn install : install terrier in the local repository (after compiling & testing)

                  Show
                  bpiwowar Benjamin Piwowarski added a comment - Craig, I just updated the pom.xml file with 1) antlr support [did not try to remove the java files] 2) test support [all the tests passed OK] The pom.xml file is nearly complete (some developer information is missing, see the end of the file). you can try it out yourself, by putting pom.xml into the base folder of terrier: 1) mvn compile : download the dependencies and compiles 2) mvn surefire:test : runs the tests 3) mvn install : install terrier in the local repository (after compiling & testing)
                  Hide
                  Permalink
                  craigm Craig Macdonald added a comment -

                  Depends on Jetty upgrade

                  Show
                  craigm Craig Macdonald added a comment - Depends on Jetty upgrade
                  Hide
                  Permalink
                  craigm Craig Macdonald added a comment -

                  Tagged for 4.1

                  Show
                  craigm Craig Macdonald added a comment - Tagged for 4.1
                  Hide
                  Permalink
                  craigm Craig Macdonald added a comment - - edited

                  Fixed for Terrier 4.1. Thanks to all contributors, including Benjamin and Nicola

                  Show
                  craigm Craig Macdonald added a comment - - edited Fixed for Terrier 4.1. Thanks to all contributors, including Benjamin and Nicola

                    People

                    • Assignee:
                      richardm Richard McCreadie
                      Reporter:
                      bpiwowar Benjamin Piwowarski
                    • Watchers:
                      0 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: