Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geronimo.apache.org
Received-SPF: pass (idunn.apache.osuosl.org: domain gmail.com designates
 64.233.166.176 as permitted sender)
DomainKey-Status: good
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:user-agent:mime-version:to:subject:references:in-reply-to:content-type:content-transfer-encoding;
        b=r1+Pwi1Xo8XmCteEneNyLrawQP42SHxdhCvhe9qqm+3oJtGZVdzpwFzAYreKan5kXhHqJ3qjI4TlZ8mXadlgGGSjxQ06HLvYvNrKuVrZeUhF5X06mxUIFGasXtaWX6R92SIG37URy1TwdUL+nQpEI1j2lqTQPBEL5PACDzMgud0=
Message-ID: <4508FA83.6080001@gmail.com>
Date: Wed, 13 Sep 2006 23:45:23 -0700
From: "Christopher M. Cardona" <chris.m.cardona@gmail.com>
User-Agent: Thunderbird 1.5.0.5 (Windows/20060719)
MIME-Version: 1.0
To: dev@geronimo.apache.org
Subject: Re: Dojo Toolkit inclusion to Geronimo
References: <450763E0.9070202@gmail.com>
 <19e0530f0609121853o77ff6cd1ia1e17260dcb01852@mail.gmail.com>
 <45076812.4000507@gmail.com>
 <9F6BEE5F-6D5B-4669-B0E6-F07087961157@optusnet.com.au>
In-Reply-To: <9F6BEE5F-6D5B-4669-B0E6-F07087961157@optusnet.com.au>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 8bit

Gianny,

Thanks again for finding time to look at the patch. Sorry to hear that 
it didn�t worked out smoothly in Safari. I agree with you that we should 
find a better way to include/checkin Dojo in G. I decided to include the 
Dojo source in the patch to make it easier for people like you who want 
to look at it without doing additional work like downloading some 
distribution. Your suggestion of expanding the Dojo files upon build is 
fine but I think checking in Dojo as a separate module (webapp) as 
suggested by Paul has an advantage of being reused by other webapps 
deployed in G simply by making the webapp�s parent the Dojo module. 
Furthermore, I was able to verify that this works. I�m still open for 
other suggestions but if we are left with these 2 options I�ll give +1 
to checking in Dojo as a separate module. Any thoughts?

Best wishes,
chris


Gianny Damour wrote:
> Hi Chris,
>
> The JMX Viewer portlet is finally working for me. Actually, it seems 
> that due to a Dojo known issue, this portlet does not work with Safari 
> :(; having said that, it works really nicely, and I really mean really 
> nicely, with IE.
>
> Regarding your patch, I believe that this is a large piece of work; 
> unfortunately, I cannot appreciate it as this is the first time that I 
> am seeing dojo in action. Also, I think that instead of checking in 
> the dojo files directly at the right location, we should check in a 
> tar.ball of these files and expand it upon build of the module. I 
> think that this is better because this way we do know which files are 
> dojo specifics (this is a minor detail). What do you think?
>
> It would be cool if other people could have a look to this patch; for 
> sure, it really deserves it!
>
> Thanks,
> Gianny
>