ICEfaces
  1. ICEfaces
  2. ICE-7957

Make our Maven build dependencies consistent

          Details

          • Type: Task Task
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.0.1, EE-3.0.0.GA
          • Fix Version/s: 3.1.0.RC1, 3.1
          • Component/s: Release
          • Labels:
            None
          • Environment:
            Build
          • Affects:
            Sample App./Tutorial

            Description

            I noticed that in some of our Maven poms, that we still use the dependency for Mojarra where the API and impl jars are separate. We should review our poms so that they consistently use the single library dependency.


                            <dependency>
                              <groupId>org.glassfish</groupId>
                              <artifactId>javax.faces</artifactId>
                              <version>${jsf.version}</version>
                              <scope>compile</scope>
                            </dependency>


            It should also be moved to the highest consistent parent pom to avoid duplication in the children.

              Issue Links

              blocks

              Improvement - An improvement or enhancement to an existing feature or task. ICE-8230 Improve Maven builds

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Deryk Sinotte created issue -
                Deryk Sinotte made changes -
                Field Original Value New Value
                Salesforce Case []
                Fix Version/s 3.1 [ 10312 ]
                Assignee Deryk Sinotte [ deryk.sinotte ]
                Deryk Sinotte made changes -
                Link This issue blocks ICE-8230 [ ICE-8230 ]
                Ken Fyten made changes -
                Salesforce Case []
                Fix Version/s 3.1.0.RC1 [ 10337 ]
                Hide
                Permalink
                Deryk Sinotte added a comment -

                I've pulled up all the dependencies that are common to the sample and provided a property for setting the value. Also, where applicable, I've updated the artifact and repository information to reflect the "proper" one as documented by the owners.

                Show
                Deryk Sinotte added a comment - I've pulled up all the dependencies that are common to the sample and provided a property for setting the value. Also, where applicable, I've updated the artifact and repository information to reflect the "proper" one as documented by the owners.
                Deryk Sinotte made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Affects [Sample App./Tutorial]
                Resolution Fixed [ 1 ]
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

                  • Assignee:
                    Deryk Sinotte
                    Reporter:
                    Deryk Sinotte
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: