incubator-rat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hyrum K. Wright" <hyrum_wri...@mail.utexas.edu>
Subject Re: [PATCH] Allow RAT to run on .tar.gz archives
Date Tue, 17 Nov 2009 16:44:16 GMT

On Nov 16, 2009, at 11:12 PM, Stefan Bodewig wrote:

> On 2009-11-14, Hyrum K. Wright <hyrum_wright@mail.utexas.edu> wrote:
> 
>> In using RAT to check potential Subversion release tarballs, I've
>> bumped into a couple of its, uh, limitations.  To that end, I've
>> hacked up the attached patch which fixes the immediate itch of not
>> wanting to have to unpack the tarball before running the tool.
> 
> The RAT Antlib already supports working on arbitrary resource
> collections, which means you wouldn't even have to download the tarball
> yourself if you wanted to.
> 
> <rat:report ...>
>  <tarfileset ...>
>    <url .../>
>  </tarfileset>
> </rat:report>

I hadn't even thought about that, but it's a nifty side effect.  :)

>> * Needed to add a new dependency (ant) to pick up TarInputStream and
>> friends.
> 
> For RAT's core commons-compress may be a better option, it is more or
> less the same code base (for tar, zip and bzip2) and has a lighter
> footprint.  Basically you'd have to change a few imports and class names
> (TarArchiveInputStream instead of TarInputStream).
> 
> http://commons.apache.org/compress/

Thanks for the suggestion.  I switched to commons-compress and uploaded a new version of the
patch: https://issues.apache.org/jira/secure/attachment/12425235/rat-tar-gz-v2.patch

Using commons-compress will actually make handling bz2 and zip files much easier, but I'm
going to wait on those until this patch is committed.

Cheers,
-Hyrum
Mime
View raw message