ICEfaces
  1. ICEfaces
  2. ICE-3199

Using command-line switches with sample app. ant build scripts doesn't remove libraries when it should.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.8DR#2
    • Component/s: Release
    • Labels:
      None
    • Environment:
      All
    • Affects:
      Sample App./Tutorial

      Description

      When using the sample app ant build scripts, specifying both a target (e.g. "tomcat6.0') and a command-line switch (e.g. '-Dmyfaces=""') will result in all the libraries specified by both the target and the switch being added to the resulting .war file. The problem is that for certain switches, such as '-Dmyfaces=""', the default JSF runtime libs defined by the build targets should not be included and the MyFaces runtime libs should be used instead.

      Related to ICE-3197.

        Activity

        yip.ng created issue -
        yip.ng made changes -
        Field Original Value New Value
        Assignee Yip Ng [ yip.ng ]
        yip.ng made changes -
        Fix Version/s 1.7.2 [ 10130 ]
        Ken Fyten made changes -
        Summary Wrong jars included in war file by certain build targets in build-common. Using command-line switches with sample app. ant build scripts doesn't remove libraries when it should.
        Description Some build targets in build-common include wrong jars in the war file, e.g. the tomcat6.0 build target with the myfaces switch set includes both the myfaces and JSF1.2 jars in the war file. See ICE-3197.

        When using the sample app ant build scripts, specifying both a target (e.g. "tomcat6.0') and a command-line switch (e.g. '-Dmyfaces=""') will result in all the libraries specified by both the target and the switch being added to the resulting .war file. The problem is that for certain switches, such as '-Dmyfaces=""', the default JSF runtime libs defined by the build targets should not be included and the MyFaces runtime libs should be used instead.

        Related to ICE-3197.

        Priority Major [ 3 ] Minor [ 4 ]
        Ken Fyten made changes -
        Fix Version/s 1.8DR#2 [ 10142 ]
        Fix Version/s 1.7.2 [ 10130 ]
        Assignee Yip Ng [ yip.ng ] Jim Zhang [ jim.zhang ]
        Ken Fyten made changes -
        Fix Version/s 1.8DR#3 [ 10143 ]
        Fix Version/s 1.8DR#2 [ 10142 ]
        Ken Fyten made changes -
        Fix Version/s 1.8DR#2 [ 10142 ]
        Fix Version/s 1.8DR#3 [ 10143 ]
        Affects [Sample App./Tutorial]
        Jim Zhang made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Jim Zhang
            Reporter:
            yip.ng
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: