Jetspeed 2
  1. Jetspeed 2
  2. JS2-911

Be able to build the installer in the trunk with Maven-2

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.0
    • Component/s: Installer
    • Labels:
      None

      Description

      poms are missing from subprojects jetspeed-ant-tasks and jetspeed-installer

      1. apache-jetspeed-trunk.patch
        104 kB
        Jeroen Hoffman
      2. JS2-911-B.patch
        26 kB
        Jeroen Hoffman

        Issue Links

          Activity

          Hide
          Jeroen Hoffman added a comment -

          Patch includes pom for jetspeed-ant-tasks, pom (plus associated files) for jetspeed-installer and changes in root pom.

          Enables building of an installer for demo and minimal Jetspeed running on Tomcat 5.5.27.

          Show
          Jeroen Hoffman added a comment - Patch includes pom for jetspeed-ant-tasks, pom (plus associated files) for jetspeed-installer and changes in root pom. Enables building of an installer for demo and minimal Jetspeed running on Tomcat 5.5.27.
          Hide
          Ate Douma added a comment -

          Hi Jeroen,

          I just committed your patch, with a few adjustments (some parts of the patch didn't seem to have come through correctly).
          I already build and tested a demo installer and that looks great!
          Thanks for a great contribution.

          There are a few issues we need to address but those are minor, e.g. db-psml installation from command-line, using the ant build after installation doesn't seem to work yet.
          In addition, we would also like to see support for using Tomcat 6 (hint ).

          NB: To be able to actually build an installer does require to checkout and build pluto, jetspeed and portals applications all locally first.

          Show
          Ate Douma added a comment - Hi Jeroen, I just committed your patch, with a few adjustments (some parts of the patch didn't seem to have come through correctly). I already build and tested a demo installer and that looks great! Thanks for a great contribution. There are a few issues we need to address but those are minor, e.g. db-psml installation from command-line, using the ant build after installation doesn't seem to work yet. In addition, we would also like to see support for using Tomcat 6 (hint ). NB: To be able to actually build an installer does require to checkout and build pluto, jetspeed and portals applications all locally first.
          Hide
          Jeroen Hoffman added a comment -

          Ate,
          Your comment about db-psml installation from command-line made me have a look at it again, and I have now a second patch that features:

          • addition of a spring-filter-key.properties file to set up the proper page manager so the pages imported into the database are actually used
          • more consistent target naming in database/build.xml
          • placing files used by database/build.xml to database/resources
          • no more copying of page manager files from and to assembly/alternate (unused)
          • no more unpacking files to database/import/db-ojb (unused)
          • removal of etc/database/import directory (unused)
          • removal of derby driver from database/lib, should use the one in tomcat/shared/lib (minimal installer is now below 40 Mb again)

          Jeroen

          Show
          Jeroen Hoffman added a comment - Ate, Your comment about db-psml installation from command-line made me have a look at it again, and I have now a second patch that features: addition of a spring-filter-key.properties file to set up the proper page manager so the pages imported into the database are actually used more consistent target naming in database/build.xml placing files used by database/build.xml to database/resources no more copying of page manager files from and to assembly/alternate (unused) no more unpacking files to database/import/db-ojb (unused) removal of etc/database/import directory (unused) removal of derby driver from database/lib, should use the one in tomcat/shared/lib (minimal installer is now below 40 Mb again) Jeroen
          Hide
          Jeroen Hoffman added a comment -

          In patch B there were some problemen found:

          • the added file 'resources/spring-filter-key.properties' wasn't in it
          • the removal of packaging derby driver should not be in it.
          • no more explicit filtering of datasource.xml but instead system properties should be used.

          Since I've been working locally on using JS2-912 (using Tomcat 6 instead of 5.5), I'll provide a new patch in that issue that also fixes these items.

          Show
          Jeroen Hoffman added a comment - In patch B there were some problemen found: the added file 'resources/spring-filter-key.properties' wasn't in it the removal of packaging derby driver should not be in it. no more explicit filtering of datasource.xml but instead system properties should be used. Since I've been working locally on using JS2-912 (using Tomcat 6 instead of 5.5), I'll provide a new patch in that issue that also fixes these items.
          Hide
          Jeroen Hoffman added a comment -

          Patch for JS2-912 will include improved patch for this issue

          Show
          Jeroen Hoffman added a comment - Patch for JS2-912 will include improved patch for this issue
          Hide
          Ate Douma added a comment -

          Feature completed and superseded by JS2-912 (also provided by Jeroen Hoffman)

          Show
          Ate Douma added a comment - Feature completed and superseded by JS2-912 (also provided by Jeroen Hoffman)

            People

            • Assignee:
              Ate Douma
              Reporter:
              Jeroen Hoffman
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development