Commons VFS
  1. Commons VFS
  2. VFS-322

Tar files to contain files over 2GB in size

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0
    • Fix Version/s: 2.0
    • Labels:
      None

      Description

      It would be nice to be able to work with tar files that contain files over 2GB in size.

      1. PDI-3157.patch
        9 kB
        Curtis Boyden

        Activity

        Hide
        Curtis Boyden added a comment -

        I have attached a patch that modifies the TarInputStream to iterate in Integer.MAX_SIZE blocks over the void skip(int) method if the size of a file is greater then Integer.MAX_SIZE.
        There is also a unit test attached that generates a 3MB tar.gz that contains a 3GB .tar containing a 3GB .txt file and verifies that the contents are correct.

        I hope this is useful and I would like to see this feature make it into the official VFS code line. (Pelase feel free to change the impl if you can improve it)

        -Curtis

        Show
        Curtis Boyden added a comment - I have attached a patch that modifies the TarInputStream to iterate in Integer.MAX_SIZE blocks over the void skip(int) method if the size of a file is greater then Integer.MAX_SIZE. There is also a unit test attached that generates a 3MB tar.gz that contains a 3GB .tar containing a 3GB .txt file and verifies that the contents are correct. I hope this is useful and I would like to see this feature make it into the official VFS code line. (Pelase feel free to change the impl if you can improve it) -Curtis
        Hide
        Ralph Goers added a comment -

        I tried to apply the patch but the unit test uses Junit 4 and the build is complaining that annotations and generics aren't supported in Java 1.4.

        Show
        Ralph Goers added a comment - I tried to apply the patch but the unit test uses Junit 4 and the build is complaining that annotations and generics aren't supported in Java 1.4.
        Hide
        Ralph Goers added a comment -

        The patch was applied. Please verify and close.

        Show
        Ralph Goers added a comment - The patch was applied. Please verify and close.

          People

          • Assignee:
            Unassigned
            Reporter:
            Curtis Boyden
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development