Uploaded image for project: 'Avro'
  1. Avro
  2. AVRO-193

"ant test-avroj" fails on Ubuntu 9.10

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.0
    • Component/s: build
    • Labels:
      None
    • Environment:

      Ubuntu 9.10

      Description

      "ant test-avroj" fails because mktemp's template in src/test/bin/test_avroj.sh does not have Xs. The accompanying patch fixes it.

      1. AVRO-193.patch
        0.5 kB
        Thiruvalluvan M. G.
      2. AVRO-193.patch
        2 kB
        Doug Cutting

        Activity

        Hide
        cutting Doug Cutting added a comment -

        I just committed this.

        Show
        cutting Doug Cutting added a comment - I just committed this.
        Hide
        philip Philip Zeyliger added a comment -

        +1 on Doug's patch.

        Show
        philip Philip Zeyliger added a comment - +1 on Doug's patch.
        Hide
        cutting Doug Cutting added a comment -

        Thanks for finding this, Thiru! Your patch works for me.

        However we've sometimes had trouble in the past when using /tmp. Might it be better to use build/test like the other tests?

        Also, it would be safest to pass paths down from ant.

        Plus, it will make things easier to debug if the test output directory remains. So we should remove it before the tests, to start with a clean slate, and leave it after, since it's in build/ and will be removed by 'ant clean'.

        And finally, it's redundant to both echo every line and echo informative messages.

        Here's a new version that addresses all of these.

        Show
        cutting Doug Cutting added a comment - Thanks for finding this, Thiru! Your patch works for me. However we've sometimes had trouble in the past when using /tmp. Might it be better to use build/test like the other tests? Also, it would be safest to pass paths down from ant. Plus, it will make things easier to debug if the test output directory remains. So we should remove it before the tests, to start with a clean slate, and leave it after, since it's in build/ and will be removed by 'ant clean'. And finally, it's redundant to both echo every line and echo informative messages. Here's a new version that addresses all of these.

          People

          • Assignee:
            thiru_mg Thiruvalluvan M. G.
            Reporter:
            thiru_mg Thiruvalluvan M. G.
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development