dev@glassfish.java.net

Re: Build is failing in Admin GUI module ...

From: Lloyd L Chambers <Lloyd.Chambers_at_Sun.COM>
Date: Thu, 09 Nov 2006 09:28:49 -0800

Why don't we merge admin-ee into admin? And ditto for most of the
other "xxx-ee" modules?

For that matter, we should merge admin into admin-core, since the
code is already cross-dependent at runtime.

Life would be much simpler.

Lloyd

On Nov 6, 2006, at 9:02 PM, Anissa Lam wrote:

> Dinesh Patil wrote:
>> Anissa Lam wrote:
>>> Hi Dinesh,
>>>
>>> PortReplacedException is in appserv-se.jar, so we changed our
>>> build.xml to depend on appserv-se.jar during compilation.
>>> If we do 'maven checkout -Dmodules=admin-gui bootstrap build'
>>> then it is fine since appserv-se.jar is available. But looks
>>> like that 'maven checkout bootstrap-all build' will fail when
>>> compiling admin-gui. I am doing a bootstrap-all now to confirm
>>> this.
>>> If its because appserv-se.jar is not available when compiling
>>> admin-gui, can you suggest how to fix this ?
>> Anissa,
>> Yes, appserv-se.jar will not be available since this is created
>> from EE modules (*-ee) and admin-gui is earlier in the build
>> dependency order.. so this class which you are using or depending
>> on need to be moved from appserv-se.jar to other jars or change
>> the dependency order but that will require more verifications/
>> tests.. So its better for you to remove this dependency from admin-
>> gui code to fix this and look for another class or may be moving
>> admin-gui later part of the order.
> Hi Dinesh,
> It used to be that admin-gui will be part of the PE build, and
> admin-gui-ee will be part of the EE build. But now that we are
> moving to have only 1 version for both PE and EE, we start to
> introduce the EE dependency in admin-gui. PortReplacedException
> is just one of these EE dependency.
> So, we need to move the order of building admin-gui to the last
> part of the build. Can you help me move it so that admin-gui is
> build just right before admin-gui-ee ? If thats too much to
> change now, then i will comment that part of the code to get the
> build going, and work on this tomorrow with Senthil as i am not
> too familiar with changing the order of the build.
>
> thanks
> Anissa.
>
>
>> thanks
>> Dinesh
>>> thanks
>>> Anissa.
>>>
>>>
>>> Dinesh Patil wrote:
>>>> Ana/Anissa,
>>>> This is still failing.. can you please fix this quickly?
>>>> thanks
>>>> Dinesh
>>>>
>>>> kedar wrote:
>>>>> Can this be fixed soon?
>>>>>
>>>>> Thanks,
>>>>> Kedar
>>>>>
>>>>>
>>>>> [java] [exec]
>>>>> [apt] ^
>>>>> [java] [exec] [apt] /space/tinderbox/smoke/
>>>>> Linux_2.4_depend/workspace/glassfish/admin-gui/admin-jsf/src/
>>>>> java/com/sun/enterprise/tools/admingui/handlers/
>>>>> StandAloneInstanceHandlers.java:231: cannot find symbol
>>>>> [java] [exec] [apt] symbol : class
>>>>> PortReplacedException
>>>>> [java] [exec] [apt] location: class
>>>>> com.sun.enterprise.tools.admingui.handlers.StandAloneInstanceHandl
>>>>> ers
>>>>> [java] [exec] [apt] if (ex.getCause()
>>>>> instanceof PortReplacedException){
>>>>> [java] [exec]
>>>>> [apt] ^
>>>>> [java] [exec] [apt] Note: Some input files use or
>>>>> override a deprecated API.
>>>>> [java] [exec] [apt] Note: Recompile with -
>>>>> Xlint:deprecation for details.
>>>>> [java] [exec] [apt] Note: Some input files use
>>>>> unchecked or unsafe operations.
>>>>> [java] [exec] [apt] Note: Recompile with -
>>>>> Xlint:unchecked for details.
>>>>> [java] [exec] [apt] 2 errors
>>>>>
>>>>>
>>>>> ------------------------------------------------------------------
>>>>> ---
>>>>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>>>>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>>>>
>>>>
>>>
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>