On 19/12/2009 10:45, André Warnier wrote: > Bill Barker wrote: > ... >> If you have a svn checkout, then the command: >> svn diff LocalStrings_fr.properties > LocalStrings_fr.properties.patch >> will do it. Alternatively, Cygwin has a Unix/Linux like diff utility. >> >> Also, for a translation, it's probably not too bad if you just attach >> the fixed file. The developer who grabs this can then run the "svn >> diff" command to review it. But if it is either Mark or myself, then >> we probably won't really be questioning the translation ;). > I tried to submit a patch file. > I actually have an SVN client on my pc and it has a "generate patch" > option. Neat.
Patch format looks good. No idea about the French though ;) > Bugzilla : Bug 48413 has been added to the database > > > To get back to the main issue, as long as I anway get the hang of this > stuff, and have checked out the SVN of Tomcat anyway, > where in the /valves stuff do I find where it actually checks the > "remote IP" against which RemoteAddressValve operates ? public void invoke(Request request, Response response) throws IOException, ServletException { process(request.getRequest().getRemoteAddr(), request, response); } It is the request.getRequest().getRemoteAddr() call. > And, supposing I were to tweak one or the other module, how do I create > a local version of Tomcat to test my changes on ? > This is exciting stuff... ;-) http://tomcat.apache.org/tomcat-6.0-doc/building.html should get you started. Ask if you get stuck. > If this is too vast as a question, just let me know and point me where > to get started maybe ? Not at all, although this is heading off topic. I'd start another thread if you have trouble with the build process. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscr...@tomcat.apache.org For additional commands, e-mail: users-h...@tomcat.apache.org