Maven Eclipse Plugin
  1. Maven Eclipse Plugin
  2. MECLIPSE-104

Add the ability to specify source inclusions/exclusions

    Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.7
    • Component/s: None
    • Labels:
      None
    • Testcase included:
      yes
    • Number of attachments :
      4

      Description

      When source files contain scm information (*/.svn/* or */CVS/*), which is pretty much a given, there's currently no way to specify that those directories be excluded except through the GUI. This isn't so much of a problem except that the next time a change is needed and this plugin is ran, it will overwrite this exclusion and will force me to exclude it, by hand, again.

      The above case is the driving reason why I decided to get involved and help out. So, I have written everything (I think) that is needed for this enhancement including, adding to the javadoc, creating a new test that verifies this enhancement, and fully testing this with my own projects (many of them @ Struts). If there is anything else I need to do as far as site documentation, please tell me where it is and I'll add it.

      This is my first patch to Maven. If this sucks, please don't ignore it, just say 'it sucks, no thanks" and I'll go about working on something else.

      Thanks so much for your attention.


      James Mitchell

        Issue Links

          Activity

          Hide
          Barrie Treloar added a comment -

          Have you tried 2.6-SNAPSHOT, or the proposed release 2.6?

          MECLIPSE-443 has added includes ="***.java" to the classpath entry - which will do a similar thing, i.e constrain the source folder to only use .java files.

          Show
          Barrie Treloar added a comment - Have you tried 2.6-SNAPSHOT, or the proposed release 2.6? MECLIPSE-443 has added includes ="***.java" to the classpath entry - which will do a similar thing, i.e constrain the source folder to only use .java files.
          Hide
          Richard van der Hoff added a comment -

          Yes - just before patching the proposed 2.6 to fix the problem and uploading the patch here.

          As I mentioned before, I need to exclude some .java files from my build; including only "*/.java" is therefore insufficient to resolve the issue.

          Show
          Richard van der Hoff added a comment - Yes - just before patching the proposed 2.6 to fix the problem and uploading the patch here. As I mentioned before, I need to exclude some .java files from my build; including only "* / .java" is therefore insufficient to resolve the issue.
          Hide
          Barrie Treloar added a comment -

          Committed to trunk for 2.6.1

          The configuration for EclipsePlugin now accepts
          <sourceInclusions>
          <sourceInclusion>XXX</sourceInclusion>
          <sourceInclusions>
          (same for sourceExclusions)

          to modifiy the classpath entries for source directories.

          */.java is always added if it is a java (or pde) project
          */.aj is always added if it is an ajdt project.

          Show
          Barrie Treloar added a comment - Committed to trunk for 2.6.1 The configuration for EclipsePlugin now accepts <sourceInclusions> <sourceInclusion>XXX</sourceInclusion> <sourceInclusions> (same for sourceExclusions) to modifiy the classpath entries for source directories. * / .java is always added if it is a java (or pde) project * / .aj is always added if it is an ajdt project.
          Hide
          Barrie Treloar added a comment -

          Still need to add documentation before I close this.

          Show
          Barrie Treloar added a comment - Still need to add documentation before I close this.
          Hide
          Barrie Treloar added a comment -

          Site documentation added

          Show
          Barrie Treloar added a comment - Site documentation added

            People

            • Assignee:
              Barrie Treloar
              Reporter:
              James Mitchell
            • Votes:
              3 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: