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

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Patrick Corless added a comment -

              Applied build patch.

              Show
              Patrick Corless added a comment - Applied build patch.
              Hide
              Permalink
              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
              Permalink
              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: