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
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Assignee Brad Kroeger [ brad.kroeger ] Patrick Corless [ patrick.corless ]
              Patrick Corless made changes -
              Salesforce Case []
              Fix Version/s EE-2.0.0.Beta1 [ 10254 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              Applied build patch.

              Show
              Patrick Corless added a comment - Applied build patch.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              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"/>
              Patrick Corless made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              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.
              Patrick Corless made changes -
              Status Reopened [ 4 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: