struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Huber <gregh3...@gmail.com>
Subject Re: [GitHub] struts pull request: loadedFileUrls only contains the last one of ...
Date Thu, 20 Mar 2014 19:45:08 GMT
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 <git@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 infrastructure@apache.org or file a JIRA ticket
> with INFRA.
> ---
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message