Avro
  1. Avro
  2. AVRO-68

Don't package up C object files when building Avro tarball

    Details

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

      Description

      This patch does a VPATH build of the C source so that all object files are kept out of the source directory and not included in the release tarball.

      1. AVRO-68.patch
        29 kB
        Matt Massie

        Activity

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

        This patch ensures that the intermediate C build directory is not packaged with the Avro release tarball

        Show
        Matt Massie added a comment - This patch ensures that the intermediate C build directory is not packaged with the Avro release tarball
        Matt Massie made changes -
        Field Original Value New Value
        Attachment AVRO-68.patch [ 12412444 ]
        Hide
        Matt Massie added a comment -

        This patch ensure that C object files stay out of the Avro release tarball

        Show
        Matt Massie added a comment - This patch ensure that C object files stay out of the Avro release tarball
        Matt Massie made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Doug Cutting added a comment -

        I just committed this. I changed the build.xml a bit so that .o files are built in build/. I also updated the git and svn ignore lists.

        Show
        Doug Cutting added a comment - I just committed this. I changed the build.xml a bit so that .o files are built in build/. I also updated the git and svn ignore lists.
        Doug Cutting made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Assignee Matt Massie [ massie ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Fix Version/s 1.0.0 [ 12313858 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development