That part was already being checked so we should be ok.

Chris

--
Chris Custine
My Blog :: http://blog.organicelement.com
Apache ServiceMix :: http://servicemix.apache.org
Apache Directory Server :: http://directory.apache.org


On Mon, Mar 2, 2009 at 12:33 AM, Guillaume Nodet <[email protected]> wrote:

> Should we also avoid starting bundle fragments from the FileMonitor ?
>
> On Mon, Mar 2, 2009 at 06:56,  <[email protected]> wrote:
> > Author: ccustine
> > Date: Mon Mar  2 05:56:32 2009
> > New Revision: 749199
> >
> > URL: http://svn.apache.org/viewvc?rev=749199&view=rev
> > Log:
> > SMX4KNL-215 - Don't attempt to stop fragments in filemonitor
> >
> > Modified:
> >
>  
> servicemix/smx4/kernel/trunk/filemonitor/src/main/java/org/apache/servicemix/kernel/filemonitor/FileMonitor.java
> >
> > Modified:
> servicemix/smx4/kernel/trunk/filemonitor/src/main/java/org/apache/servicemix/kernel/filemonitor/FileMonitor.java
> > URL:
> http://svn.apache.org/viewvc/servicemix/smx4/kernel/trunk/filemonitor/src/main/java/org/apache/servicemix/kernel/filemonitor/FileMonitor.java?rev=749199&r1=749198&r2=749199&view=diff
> >
> ==============================================================================
> > ---
> servicemix/smx4/kernel/trunk/filemonitor/src/main/java/org/apache/servicemix/kernel/filemonitor/FileMonitor.java
> (original)
> > +++
> servicemix/smx4/kernel/trunk/filemonitor/src/main/java/org/apache/servicemix/kernel/filemonitor/FileMonitor.java
> Mon Mar  2 05:56:32 2009
> > @@ -397,7 +397,9 @@
> >         if (bundle == null) {
> >             LOGGER.warn("Could not find Bundle for file: " +
> file.getCanonicalPath());
> >         } else {
> > -            bundle.stop();
> > +            if (!isBundleFragment(bundle)) {
> > +                bundle.stop();
> > +            }
> >             bundle.uninstall();
> >         }
> >     }
> >
> >
> >
>
>
>
> --
> Cheers,
> Guillaume Nodet
> ------------------------
> Blog: http://gnodet.blogspot.com/
> ------------------------
> Open Source SOA
> http://fusesource.com
>

Reply via email to