Ok, there was a bug in my code. When i was writing the bytes to the output
stream, i was writing some extra bytes. here is what i had earlier
byte buffer[] = new byte[4096];
for (int c = jis.read(buffer); c != -1; c = jis.read(buffer))
{
jos.write(buffer); // i was writing all 4096 bytes here
}
And here is what i have changed it into
byte buffer[] = new byte[4096];
for (int c = jis.read(buffer); c != -1; c = jis.read(buffer))
{
jos.write(buffer,0,c); // just writing the number of
bytes read from the inputstream
}
Works fine now. I will submit a patch
On 1/17/07, Karan Malhi <[EMAIL PROTECTED]> wrote:
David,
1. you sure you're using compression?
--I am using the JarInputStream and JarOutputStream. I also tried to set
the compression level to 9 but didnt acheive anything.
2. if suggestion #1 doesn't lead anywhere, feel free to take the
chunk of Ant code you need and trim it down, clean it up and use that.
--I was thinking that before digging into ANT, if somebody could look at
the code and see whats wrong, then that could save some trouble. I am
probably missing something in the code. This is the first time i have worked
with Jar input and outputstreams. Dain did send me a link to the
DeploymentUtil of Geronimo, I will further look into that and see how it was
done in DeploymentUtil.
The code behaves a bit differently with different jars. For example, i
took ant.jar and tried to add files to it. even if i dont add a file and
keep running my program on ant.jar, it keeps increasing its size (but
stops increasing the size after 4 or 5 runs). however, if i pick another jar
and try to add a file to it, the code works fine (size is the same as
produced by the "jar" command). I probably need to do some more testing
before i can come up with a definitive conclusion, but for now i am guessing
maybe it is something with ant.jar or something wrong with my code.
karan
On Jan 17, 2007, at 8:39 AM, Karan Malhi wrote:
> Please do not review the patch yet. i did not svn update before
> creating the
> patch, will do it tonight and submit the correct patch. Apologize
> for the
> confusion
>
> On 1/16/07, Karan Malhi <[EMAIL PROTECTED]> wrote:
>>
>> I have attached another patch, without the ANT libraries. Please
>> review
>> it. It works fine, I am not happy with the size of the jar it creates
>> though. Any thoughts or input would be a good learning experience
>> for me.
>>
>> https://issues.apache.org/jira/browse/OPENEJB-434
>>
>>
>> On 1/16/07, Jacek Laskowski < [EMAIL PROTECTED]> wrote:
>> >
>> > On 1/16/07, Karan Malhi < [EMAIL PROTECTED]> wrote:
>> >
>> > > I then tried to use the Jar class from ANT and it works perfectly
>> > (same as
>> > > the jar command) and is much simpler to use. Do you think we
>> could use
>> > the
>> > > ant classes for openejb project.?
>> >
>> > I think Ant's fine as long as we don't use it as if we're
>> executed it
>> > from the command line, but using its classes. I'm not convinced,
>> > though, it's the best bet as Ant is more than copying jars and for
>> > such a small task we should not need it at all.
>> >
>> > Jacek
>> >
>> > --
>> > Jacek Laskowski
>> > http://www.JacekLaskowski.pl <http://www.jaceklaskowski.pl/>
>> >
>>
>>
>>
>> --
>> Karan Malhi
>>
>
>
>
> --
> Karan Malhi
--
Karan Malhi
--
Karan Malhi
