dev@glassfish.java.net

Re: [REVIEW] several pom changes necessary to integrate Grizzly 1.9.19-beta1

From: Snjezana Sevo-Zenzerovic <Snjezana.Sevo-Zenzerovic_at_Sun.COM>
Date: Tue, 26 Jan 2010 12:51:05 -0800

Jane,

those servlet-api/jstl-api dependencies were commented for a similar
reason, we were running into stack overflow error during dependency
graph resolution and the workaround was to exclude them using ant tasks
instead of direct pom dependency exclusion. I would leave them as-is for
the time being and I'll take another look when I have more time.

Thanks,

Snjezana


Jane Young wrote:
> Hi Ryan,
>
> Thanks for updating the Copyright dates to 2010.
> I reviewed the pom files and have one question (maybe the question is
> for Snjezana). In packager/glassfish-web/pom.xml, the artifacts
> servlet-api and jstl-api are commented. Is there a reason for doing
> this? It's was also previously commented. Can we remove it?
>
> The rest of the pom file changes look fine.
>
> Thanks,
> Jane
>
>
>
> Ryan Lubke wrote:
>> Hi Jane, Snjezana,
>>
>> I've had to add/duplicate a small set of exclusions to several of the
>> pom.xml files in the packager section of the tree.
>> This was necessary in order to integrate a new version of Grizzly
>> without the antrun-extended pluging to bottom out
>> with a StackoverFlowError.
>>
>> Could you please give your blessing on these changes?
>>
>> Thanks,
>> -rl
>> ------------------------------------------------------------------------
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>