Uploaded image for project: 'Pig'
  1. Pig
  2. PIG-68

Improving build.xml in many ways :)

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.1.0
    • Component/s: None
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      The build file can be improve in many ways:

      • add revision number to pig.jar name (like: pig-r1234.jar)
      • put pig.jar in the dist dir
      • "clean" target leave a "depend" folder undeleted
      • use a regexp to delete files in "org\apache\pig\impl\logicalLayer\parser" folder instead of listing all files one by one that you want to delete
      • put all artifacts (classes, jar, etc...) in the dist folder so that when doing clean you just need to specify dist
      • provide a description for targets (for "ant -projecthelp" command)
      • use spaces or tabs but not both (spaces are better for patch and diff in my opinion)

        Attachments

        1. out
          17 kB
          Olga Natkovich
        2. build.xml-PIG-68-v09-SG.patch
          20 kB
          Stefan Groschupf
        3. build.xml-PIG-68-v08-SG.patch
          20 kB
          Stefan Groschupf
        4. build.xml-PIG-68-v07-SG.patch
          20 kB
          Stefan Groschupf
        5. build.xml-PIG-68-v06-SG.patch
          20 kB
          Stefan Groschupf
        6. build.xml-PIG-68-v05.patch
          14 kB
          Benjamin Francisoud
        7. build.xml-PIG-68-v04.patch
          14 kB
          Benjamin Francisoud
        8. build.xml-PIG-68-v03.patch
          14 kB
          Benjamin Francisoud
        9. build.xml-PIG-68-v02.patch
          14 kB
          Benjamin Francisoud
        10. build.xml-PIG-68-v01.patch
          14 kB
          Benjamin Francisoud
        11. build.xml
          13 kB
          Stefan Groschupf

          Issue Links

            Activity

              People

              • Assignee:
                joa23 Stefan Groschupf
                Reporter:
                francisoud Benjamin Francisoud
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: