You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by Jason Dillon <ja...@planet57.com> on 2007/03/01 22:32:23 UTC

Re: mx4j dependencies

I think we are free from mx4j deps now... only 2 references of 'mx4j'  
in the project now:

  * modules/ge-activemq-rar/pom.xml (dependency excludes)
  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/jetty6/ 
ClassLoaderTest.java (attempts to use "mx4j.MBeanDescription" to test  
classloading stuff)

Not sure what to do about the later... or even if anything needs to  
be done.

I've removed all other references, including the LICENSE.txt/ 
NOTICE.txt bits.  I didn't find any \*mx4j\* files in the jetty  
assembly, so unless mx4j is hiding in a jar/car/whatever I think it  
should be gone now.

--jason


On Feb 28, 2007, at 1:30 PM, Sachin Patel wrote:

> should work, as i think i remember it working from the eclipse  
> plugin also, but will verify.
>
> -sachin
>
>
> On Feb 28, 2007, at 4:27 PM, Jason Dillon wrote:
>
>> What about remote deploy?
>>
>> --jason
>>
>>
>> On Feb 28, 2007, at 1:22 PM, Sachin Patel wrote:
>>
>>> Trying now, I know I removed my dependencies on mx4j on devtools  
>>> and rely on the jmx stuff in the vm and it works perfectly.
>>>
>>> -sachin
>>>
>>>
>>> On Feb 28, 2007, at 4:16 PM, Jason Dillon wrote:
>>>
>>>> Does the system blow up in strange ways if you do?
>>>>
>>>> --jason
>>>>
>>>>
>>>> On Feb 28, 2007, at 1:09 PM, Sachin Patel wrote:
>>>>
>>>>> The following still have dependencies to mx4j..
>>>>>
>>>>> applications/console/geronimo-console-core/pom.xml
>>>>> configs/client-system/pom.xml
>>>>> configs/j2ee-system/pom.xml
>>>>> configs/jsr88-cli/pom.xml
>>>>> configs/jsr88-deploymentfactory/pom.xml
>>>>> configs/shutdown/pom.xml
>>>>> maven-plugins/geronimo-maven-plugin/pom.xml
>>>>> modules/ge-activemq-rar/pom.xml
>>>>> modules/geronimo-deploy-jsr88/pom.xml
>>>>> modules/geronimo-jmx-remoting/pom.xml
>>>>> pom.xml
>>>>>
>>>>> Can we go ahead and remove these?
>>>>>
>>>>> -sachin
>>>>>
>>>>>
>>>>
>>>
>>
>


Re: mx4j dependencies

Posted by Jason Dillon <ja...@planet57.com>.
Why not just change these to use org/apache/geronimo/system/main/ 
Daemon, which should be on the parent's classloader?  Or org/apache/ 
log4j/Level ?

Anyways... I don't know enough about what this is really testing to  
just nuke it.  Seems a tad safer to just pick a different class to load.

--jason


On Mar 2, 2007, at 3:53 PM, Anita Kulshreshtha wrote:

>  Go for it...
>
> Anita
>
> --- Jason Dillon <ja...@planet57.com> wrote:
>
>> Ah... I didn't notice that ;-)
>>
>> Maybe it should be nuked then...
>>
>> --jason
>>
>>
>> On Mar 2, 2007, at 4:49 AM, Anita Kulshreshtha wrote:
>>
>>>   The tests have been disabled (the names starts with xtest)...
>>>
>>> Thanks
>>> Anita
>>>
>>> --- Jason Dillon <ja...@planet57.com> wrote:
>>>
>>>> On Mar 1, 2007, at 4:04 PM, Dain Sundstrom wrote:
>>>>>>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/
>>>>>> jetty6/ClassLoaderTest.java (attempts to use
>>>>>> "mx4j.MBeanDescription" to test classloading stuff)
>>>>>>
>>>>>> Not sure what to do about the later... or even if anything needs
>>>>>> to be done.
>>>>>
>>>>> I'd just say remove the test.
>>>>
>>>> This test doesn't actually fail with the mx4j deps removed from
>> the
>>>> project.  So either is picking up mx4j somewhere (that isn't
>> getting
>>>>
>>>> included in the assembly, cause I can't find it there... though
>>>> didn't look too hard for it), or the test is disabled, or its not
>>>> testing anything useful... or its a broken test... or i've been
>>>> sucked into a biizzzzaaaroooo universe.
>>>>
>>>> --jason
>>>>
>>>>
>>>>
>>>
>>>
>>>
>>>
>>>
>>
> ______________________________________________________________________
>>
>>> ______________
>>> Need a quick answer? Get one in minutes from people who know.
>>> Ask your question on www.Answers.yahoo.com
>>
>>
>
>
>
>
> ______________________________________________________________________ 
> ______________
> Bored stiff? Loosen up...
> Download and play hundreds of games for free on Yahoo! Games.
> http://games.yahoo.com/games/front


Re: mx4j dependencies

Posted by Anita Kulshreshtha <a_...@yahoo.com>.
 Go for it...

Anita

--- Jason Dillon <ja...@planet57.com> wrote:

> Ah... I didn't notice that ;-)
> 
> Maybe it should be nuked then...
> 
> --jason
> 
> 
> On Mar 2, 2007, at 4:49 AM, Anita Kulshreshtha wrote:
> 
> >   The tests have been disabled (the names starts with xtest)...
> >
> > Thanks
> > Anita
> >
> > --- Jason Dillon <ja...@planet57.com> wrote:
> >
> >> On Mar 1, 2007, at 4:04 PM, Dain Sundstrom wrote:
> >>>>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/
> >>>> jetty6/ClassLoaderTest.java (attempts to use
> >>>> "mx4j.MBeanDescription" to test classloading stuff)
> >>>>
> >>>> Not sure what to do about the later... or even if anything needs
> >>>> to be done.
> >>>
> >>> I'd just say remove the test.
> >>
> >> This test doesn't actually fail with the mx4j deps removed from
> the
> >> project.  So either is picking up mx4j somewhere (that isn't
> getting
> >>
> >> included in the assembly, cause I can't find it there... though
> >> didn't look too hard for it), or the test is disabled, or its not
> >> testing anything useful... or its a broken test... or i've been
> >> sucked into a biizzzzaaaroooo universe.
> >>
> >> --jason
> >>
> >>
> >>
> >
> >
> >
> >
> >
>
______________________________________________________________________
> 
> > ______________
> > Need a quick answer? Get one in minutes from people who know.
> > Ask your question on www.Answers.yahoo.com
> 
> 



 
____________________________________________________________________________________
Bored stiff? Loosen up... 
Download and play hundreds of games for free on Yahoo! Games.
http://games.yahoo.com/games/front

Re: mx4j dependencies

Posted by Jason Dillon <ja...@planet57.com>.
Ah... I didn't notice that ;-)

Maybe it should be nuked then...

--jason


On Mar 2, 2007, at 4:49 AM, Anita Kulshreshtha wrote:

>   The tests have been disabled (the names starts with xtest)...
>
> Thanks
> Anita
>
> --- Jason Dillon <ja...@planet57.com> wrote:
>
>> On Mar 1, 2007, at 4:04 PM, Dain Sundstrom wrote:
>>>>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/
>>>> jetty6/ClassLoaderTest.java (attempts to use
>>>> "mx4j.MBeanDescription" to test classloading stuff)
>>>>
>>>> Not sure what to do about the later... or even if anything needs
>>>> to be done.
>>>
>>> I'd just say remove the test.
>>
>> This test doesn't actually fail with the mx4j deps removed from the
>> project.  So either is picking up mx4j somewhere (that isn't getting
>>
>> included in the assembly, cause I can't find it there... though
>> didn't look too hard for it), or the test is disabled, or its not
>> testing anything useful... or its a broken test... or i've been
>> sucked into a biizzzzaaaroooo universe.
>>
>> --jason
>>
>>
>>
>
>
>
>
> ______________________________________________________________________ 
> ______________
> Need a quick answer? Get one in minutes from people who know.
> Ask your question on www.Answers.yahoo.com


Re: mx4j dependencies

Posted by Anita Kulshreshtha <a_...@yahoo.com>.
  The tests have been disabled (the names starts with xtest)...

Thanks
Anita

--- Jason Dillon <ja...@planet57.com> wrote:

> On Mar 1, 2007, at 4:04 PM, Dain Sundstrom wrote:
> >>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/ 
> >> jetty6/ClassLoaderTest.java (attempts to use  
> >> "mx4j.MBeanDescription" to test classloading stuff)
> >>
> >> Not sure what to do about the later... or even if anything needs  
> >> to be done.
> >
> > I'd just say remove the test.
> 
> This test doesn't actually fail with the mx4j deps removed from the  
> project.  So either is picking up mx4j somewhere (that isn't getting 
> 
> included in the assembly, cause I can't find it there... though  
> didn't look too hard for it), or the test is disabled, or its not  
> testing anything useful... or its a broken test... or i've been  
> sucked into a biizzzzaaaroooo universe.
> 
> --jason
> 
> 
> 



 
____________________________________________________________________________________
Need a quick answer? Get one in minutes from people who know.
Ask your question on www.Answers.yahoo.com

Re: mx4j dependencies

Posted by Jason Dillon <ja...@planet57.com>.
On Mar 1, 2007, at 4:04 PM, Dain Sundstrom wrote:
>>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/ 
>> jetty6/ClassLoaderTest.java (attempts to use  
>> "mx4j.MBeanDescription" to test classloading stuff)
>>
>> Not sure what to do about the later... or even if anything needs  
>> to be done.
>
> I'd just say remove the test.

This test doesn't actually fail with the mx4j deps removed from the  
project.  So either is picking up mx4j somewhere (that isn't getting  
included in the assembly, cause I can't find it there... though  
didn't look too hard for it), or the test is disabled, or its not  
testing anything useful... or its a broken test... or i've been  
sucked into a biizzzzaaaroooo universe.

--jason



Re: mx4j dependencies

Posted by Dain Sundstrom <da...@iq80.com>.
On Mar 1, 2007, at 1:32 PM, Jason Dillon wrote:

>  * modules/geronimo-jetty6/src/test/java/org/apache/geronimo/jetty6/ 
> ClassLoaderTest.java (attempts to use "mx4j.MBeanDescription" to  
> test classloading stuff)
>
> Not sure what to do about the later... or even if anything needs to  
> be done.

I'd just say remove the test.

-dain