Avro
  1. Avro
  2. AVRO-430

Remove subversion directories from Avro C tarball

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: c
    • Labels:
      None

      Description

      When a directory is included in an EXTRA_DIST list, all contents are recursively included. This cases a few directories to include '.svn' directories. This patch ensures there are no '.svn' directories in the distributed tarball for Avro C.

        Activity

        Matt Massie created issue -
        Hide
        Matt Massie added a comment -

        Simple patch

        --- lang/c/Makefile.am	(revision 915466)
        +++ lang/c/Makefile.am	(working copy)
        @@ -%ld,%ld +%ld,%ld @@
         EXTRA_DIST=version.sh m4
         SUBDIRS = docs jansson src tests examples
         
        +# Make sure we never package up '.svn' directories
        +dist-hook:
        +	find $(distdir) -name '.svn' | xargs rm -rf
        +
        
        Show
        Matt Massie added a comment - Simple patch --- lang/c/Makefile.am (revision 915466) +++ lang/c/Makefile.am (working copy) @@ -%ld,%ld +%ld,%ld @@ EXTRA_DIST=version.sh m4 SUBDIRS = docs jansson src tests examples +# Make sure we never package up '.svn' directories +dist-hook: + find $(distdir) -name '.svn' | xargs rm -rf +
        Hide
        Matt Massie added a comment -

        Committed to trunk

        Show
        Matt Massie added a comment - Committed to trunk
        Matt Massie made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Fix Version/s 1.3.0 [ 12314318 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4m 35s 1 Matt Massie 23/Feb/10 18:48
        Resolved Resolved Closed Closed
        6d 2h 59m 1 Doug Cutting 01/Mar/10 21:47

          People

          • Assignee:
            Matt Massie
            Reporter:
            Matt Massie
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development