Cargo
  1. Cargo
  2. CARGO-897

Add Eclipse IDE files to svn:ignore

    Details

    • Type: Wish Wish
    • Status: Closed Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0
    • Component/s: None
    • Labels:
      None
    • Complexity:
      Intermediate
    • Patch Submitted:
      Yes
    • Number of attachments :
      4

      Description

      For those of us using Eclipse, the attached patches would make contributing easier as it hides IDE-specific files from the patches.

      1. core.patch
        0.9 kB
        Benjamin Bentmann
      2. extensions.patch
        0.9 kB
        Benjamin Bentmann
      3. pom.patch
        0.2 kB
        Benjamin Bentmann
      4. resources.patch
        0.2 kB
        Benjamin Bentmann

        Activity

        Hide
        Anders Hammar added a comment -

        .classpath should be added to the ignore list as well.

        Show
        Anders Hammar added a comment - .classpath should be added to the ignore list as well.
        Hide
        Savas Ali Tokmen added a comment -

        For some reason the MacOS X patch tools says:

        patch: **** Only garbage was found in the patch input.
        

        Anyone else able of applying this patch?

        Show
        Savas Ali Tokmen added a comment - For some reason the MacOS X patch tools says: patch: **** Only garbage was found in the patch input. Anyone else able of applying this patch?
        Hide
        Anders Hammar added a comment -

        I'm trying to alter the svn:ignore property from inside Eclipse, but it doesn't work!

        Show
        Anders Hammar added a comment - I'm trying to alter the svn:ignore property from inside Eclipse, but it doesn't work!
        Hide
        Anders Hammar added a comment - - edited

        svn:ignore updated on trunk (r2697) with:
        *.iml
        *.ipr
        *.iws
        .classpath
        .project
        .settings
        target
        bin

        I aligned it on all directories, as there were differences.

        Show
        Anders Hammar added a comment - - edited svn:ignore updated on trunk (r2697) with: *.iml *.ipr *.iws .classpath .project .settings target bin I aligned it on all directories, as there were differences.
        Hide
        Benjamin Bentmann added a comment -

        Not sure it made sense to recursively set those on each and every sub directory but well.

        Show
        Benjamin Bentmann added a comment - Not sure it made sense to recursively set those on each and every sub directory but well.
        Hide
        Savas Ali Tokmen added a comment -

        The fix is incorrect as it adds many svn ignore entries to directories like src, main, java, etc.

        Please revert, and if the command line client does not let us easily set this I'll do with TortoiseSVN

        Show
        Savas Ali Tokmen added a comment - The fix is incorrect as it adds many svn ignore entries to directories like src, main, java, etc. Please revert, and if the command line client does not let us easily set this I'll do with TortoiseSVN
        Hide
        Anders Hammar added a comment -

        Sure, I'll look into it. Although I don't really see how these ignore entries would be a problem for the src subdirectories.
        The problem not doing changes like this recursively, IMHO, is that "correct" ignore entries are not applied to all the (Maven/eclipse) projects. That's how it was before, where there were inconsistencies in the entries.

        Show
        Anders Hammar added a comment - Sure, I'll look into it. Although I don't really see how these ignore entries would be a problem for the src subdirectories. The problem not doing changes like this recursively, IMHO, is that "correct" ignore entries are not applied to all the (Maven/eclipse) projects. That's how it was before, where there were inconsistencies in the entries.
        Hide
        Anders Hammar added a comment -

        Should be fixed in r2698.

        Show
        Anders Hammar added a comment - Should be fixed in r2698.
        Hide
        Savas Ali Tokmen added a comment -

        Thank you, I agree that these settings are pretty annoying to do

        Show
        Savas Ali Tokmen added a comment - Thank you, I agree that these settings are pretty annoying to do

          People

          • Assignee:
            Anders Hammar
            Reporter:
            Benjamin Bentmann
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: