ICEfaces
  1. ICEfaces
  2. ICE-6196

icefaces-ace.jar build only include used javascript files

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ACE, build

      Description

      Since MandatoryResourceComponent will be phased out as a way of determining all javascript dependencies, in favour of using the YUI 3 loader, we no longer will have a means of enumerating all javascript in use. So, we'll abandon the automatic pruning of icefaces-ace.jar's YUI files, and try an explicit strategy in the build.xml.

        Issue Links

          Activity

          Mark Collette created issue -
          Mark Collette made changes -
          Field Original Value New Value
          Link This issue depends on ICE-6183 [ ICE-6183 ]
          Mark Collette made changes -
          Link This issue depends on ICE-6194 [ ICE-6194 ]
          Mark Collette made changes -
          Link This issue depends on ICE-6195 [ ICE-6195 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 2.1 [ 10241 ]
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-6490 [ ICE-6490 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #24009 Thu Feb 24 19:52:24 MST 2011 mark.collette ICE-6196 : icefaces-ace.jar build only include used javascript files
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/build.properties
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/build.xml
          Hide
          Mark Collette added a comment -

          Since MandatoryResourceComponent will be phased out as a way of determining all javascript dependencies, in favour of using the YUI 3 loader, we no longer will have a means of enumerating all javascript in use. So, we'll abandon the automatic pruning of icefaces-ace.jar's YUI files, and try an explicit strategy in the build.xml.

          Committed is a change that involves explicitly stating which parts of YUI 2 and 3 to include in the JAR. That is, it's a white list, not a black list. For now I've left the pruning turned off by default, in case I've missed anything from the white list. After testing, someone can turn it on by default. So do so, uncomment the exclude.unused.yui property in the ace/components/build.properties file.

          Subversion 24009

          Show
          Mark Collette added a comment - Since MandatoryResourceComponent will be phased out as a way of determining all javascript dependencies, in favour of using the YUI 3 loader, we no longer will have a means of enumerating all javascript in use. So, we'll abandon the automatic pruning of icefaces-ace.jar's YUI files, and try an explicit strategy in the build.xml. Committed is a change that involves explicitly stating which parts of YUI 2 and 3 to include in the JAR. That is, it's a white list, not a black list. For now I've left the pruning turned off by default, in case I've missed anything from the white list. After testing, someone can turn it on by default. So do so, uncomment the exclude.unused.yui property in the ace/components/build.properties file. Subversion 24009
          Mark Collette made changes -
          Link This issue is duplicated by ICE-6249 [ ICE-6249 ]
          Hide
          Ken Fyten added a comment -

          Let's enable this by default now for 2.0.1.

          Show
          Ken Fyten added a comment - Let's enable this by default now for 2.0.1.
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 2.0.1 [ 10255 ]
          Fix Version/s 2.1 [ 10241 ]
          Assignee Priority P1
          Assignee Mark Collette [ mark.collette ]
          Ken Fyten made changes -
          Salesforce Case []
          Affects Version/s 2.0.0 [ 10230 ]
          Affects Version/s 2.0-Beta2 [ 10242 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #24075 Wed Mar 02 12:36:58 MST 2011 mark.collette ICE-6196 : icefaces-ace.jar build only include used javascript files
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/build.properties
          Hide
          Mark Collette added a comment -

          Modified the build.properties to enable this by default.

          Subversion 24075

          Show
          Mark Collette added a comment - Modified the build.properties to enable this by default. Subversion 24075
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P1
          Ken Fyten made changes -
          Link This issue depends on ICE-6183 [ ICE-6183 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-6194 [ ICE-6194 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-6195 [ ICE-6195 ]
          Ken Fyten made changes -
          Link This issue depends on ICE-6490 [ ICE-6490 ]
          Ken Fyten made changes -
          Link This issue is duplicated by ICE-6249 [ ICE-6249 ]
          Ken Fyten made changes -
          Comment [ Possibly not all our components will have @ManadatoryResourceComponent annotations, or that processing may be disabled, or others using the Advanced Component Environement for their own component might not use it. And since this is done at build time, possible it would make sense to scan the Base classes an the Renderer classes and the hand coded components to see their @ResourceDependency annotations, and build the list of all used javascript files from there. We'll have to see how the depended on jiras work out. ]
          Ken Fyten made changes -
          Salesforce Case []
          Description Once ICE-6183, ICE-6194, ICE-6195 are implemented, we'll have a list of all the javascript files that the ACE library needs, and so can make the JAR build only include those, and none of the YUI libraries which we're not using. We should consider what this will mean for developers who use the ACE platform to create their own components, and how their JARs and components will interact with our ACE one.
          Since MandatoryResourceComponent will be phased out as a way of determining all javascript dependencies, in favour of using the YUI 3 loader, we no longer will have a means of enumerating all javascript in use. So, we'll abandon the automatic pruning of icefaces-ace.jar's YUI files, and try an explicit strategy in the build.xml.
          Ken Fyten made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Security Private [ 10001 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Issue Type New Feature [ 2 ] Improvement [ 4 ]
          Salesforce Case []
          Mark Collette made changes -
          Link This issue is duplicated by ICE-6633 [ ICE-6633 ]
          Mark Collette made changes -
          Link This issue is duplicated by ICE-6249 [ ICE-6249 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Mark Collette
              Reporter:
              Mark Collette
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: