Shindig
  1. Shindig
  2. SHINDIG-869

Ignore SVN files in shindig-features-XXX.jar

        Details

        • Type: Bug Bug
        • Status: Resolved
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: 1.0, 1.1-BETA1
        • Fix Version/s: 1.0, 1.1-BETA1
        • Component/s: Javascript
        • Labels:
          None

          Description

          Actually, the shindig-features-XXX.jar contains IDE files, for instance .classpath. We should remove them.

          1. Text File
            SHINDIG-869.patch
            5 kB
            Vincent Siveton

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Vincent Siveton added a comment -

            This patch contains:

            • ordered Maven plugins
            • ignored files like define in the etc/svn-ignores

            Since I am there, I also configured yuicompressor-maven-plugin to not pollute the build with warn/info logs, ie

            • jswarn=false
            • statistics=false
            Show
            Vincent Siveton added a comment - This patch contains: ordered Maven plugins ignored files like define in the etc/svn-ignores Since I am there, I also configured yuicompressor-maven-plugin to not pollute the build with warn/info logs, ie jswarn=false statistics=false
            Hide
            Permalink
            Paul Lindner added a comment -

            looks good reduced features jar by a few kilobytes.

            Show
            Paul Lindner added a comment - looks good reduced features jar by a few kilobytes.
            Hide
            Permalink
            Vincent Siveton added a comment -

            Paul, could you merge this patch to the branch?

            Show
            Vincent Siveton added a comment - Paul, could you merge this patch to the branch?
            Hide
            Permalink
            Paul Lindner added a comment -

            This patch depends on SHINDIG-820, SHINDIG-821

            Should those get merged too?

            Show
            Paul Lindner added a comment - This patch depends on SHINDIG-820 , SHINDIG-821 Should those get merged too?
            Hide
            Permalink
            Vincent Siveton added a comment -

            fixed in r735073 and r750907

            Show
            Vincent Siveton added a comment - fixed in r735073 and r750907

              People

              • Assignee:
                Unassigned
                Reporter:
                Vincent Siveton
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development