ICEfaces-EE
  1. ICEfaces-EE
  2. IPCK-150

composite components 2 build script update to standard ant paths.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-2.0.0.Beta1
    • Fix Version/s: EE-2.0.0.Beta1
    • Component/s: Facelet Components
    • Labels:
      None
    • Environment:
      jsf 2.0/icefaces 2.0

      Description

      The ant script for composite components 2 needs a slight tweak to follow Ant best practices for specifying the icefaces build lib directory.

      In the build.properties there is property "icefaces.dir=icefaces"

      and uses it in the build.xml as follows

      <property name="icefaces.lib.dir" value="../${icefaces.dir}/lib"/>
      <property name="icefaces.compat.lib.dir" value="../${icefaces.dir}/compat/lib"/>


      The problem with this approach is that icefaces.dir is basically hard coded and very difficult to change. I would like the see the setup changed to .

      In the build.properties there is property "icefaces.dir=../icefaces"

      and uses it in the build.xml as follows

      <property name="icefaces.lib.dir" value="${icefaces.dir}/lib"/>
      <property name="icefaces.compat.lib.dir" value="${icefaces.dir}/compat/lib"/>

        Activity

        Hide
        Patrick Corless added a comment -

        Applied build patch.

        Show
        Patrick Corless added a comment - Applied build patch.
        Hide
        Patrick Corless added a comment -

        Reopening as the RC1 of ICEfaces changed the library paths around a bit. So compat lib is no longer valid.

        <property name="icefaces.compat.lib.dir" value="$

        {icefaces.dir}

        /compat/lib"/>

        Should be replaced with

        <property name="icefaces.compat.lib.dir" value="$

        {icefaces.lib.dir}

        /compat"/>

        Show
        Patrick Corless added a comment - Reopening as the RC1 of ICEfaces changed the library paths around a bit. So compat lib is no longer valid. <property name="icefaces.compat.lib.dir" value="$ {icefaces.dir} /compat/lib"/> Should be replaced with <property name="icefaces.compat.lib.dir" value="$ {icefaces.lib.dir} /compat"/>
        Hide
        Patrick Corless added a comment -

        Applied fix to svn/repo/icepack2/ . Closing.

        Show
        Patrick Corless added a comment - Applied fix to svn/repo/icepack2/ . Closing.

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Patrick Corless
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: