> Ok well here is a patch to TldLocationsCache that should all Jasper to
> correctly process packaged tab libraries.  I tested on my system and it
> works well.  I changed processJars() to use the "getResourcePaths" method
as
> Remy suggested.  I pulled "tldConfigJar()" as it did a bunch of stuff with
> URL's that appeared to be overly complex.  I put the logic from that
method
> right in the "processJars()" and accessed the jar files directly (i.e.
w/out
> a url or connection).  I changed "parseTldForUri()" because it was looking
> for a child <taglib> and then a <uri> it should have just looked for a
<uri>
> right off.
>
> Lastly I'm not sure why the diff produced some 'goofy' stuff.  If somebody
> lets me know how to fix it I won't do it in the future.
>
> Really lastly - this is my first attempt at sending in a patch to an open
> source project so if somehow I've f'ed up the etiquette, then please be
> gentle.

Nice work, but you shouldn't use getRealPath() and access as a file, as
getRealPath() can return null.
Although it is indeed more complex, tldConfigJar() is the way to go, as it
should work in all cases.

Remy

Reply via email to