It does fix the reloading issue, but now it reloads a whole lot more files.
I can see it checks the files per configuration, but it seems to check them
twice?  Whether it actually reloads them twice I will investigate.

If you log needsReload() {..} you can see, for me its gone from 4 to 11
files!

On 20 March 2014 17:49, lukaszlenart <g...@git.apache.org> wrote:

> Github user lukaszlenart commented on the pull request:
>
>     https://github.com/apache/struts/pull/6#issuecomment-38199004
>
>     You must have a strange setup, though. `XmlConfigurationProvider` is
> instantiated per each configuration file, i.e. `struts-default.xml` (one),
> `struts-plugin.xml` (many) and `struts.xml` (one). You probably include
> `struts-plug.xml` in `struts-biz.xml` or something.
>
>     In theory `FileManager` supports monitoring of file changes in jars
> but only in the simplest setup (under Tomcat or Jetty).
>
>
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
> For additional commands, e-mail: dev-h...@struts.apache.org
>
>

Reply via email to