Hi,<div><br></div><div>After doing some research, I found that there is a way to use multiple instances of Velocity. It's actually pretty simple. It is explained here: <a href="http://velocity.apache.org/engine/releases/velocity-1.7/developer-guide.html#separate">http://velocity.apache.org/engine/releases/velocity-1.7/developer-guide.html#separate</a></div>
<div><br></div><div>I couldn't find an easy way to set up a jira account to log this issue, so I hope that one of you can log this issue for v3.</div><div><br></div><div><br></div><div>Regards,</div><div>Tom<br><br><div class="gmail_quote">
On Tue, May 1, 2012 at 12:00 PM, Tom van den Berge <span dir="ltr"><<a href="mailto:tom.vandenberge@gmail.com" target="_blank">tom.vandenberge@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Brent,<div><br></div><div>Unfortunately overriding initializeVelocity doesn't work since it is a static method, so the only option seems to be to override bootstrap() and leave out the initializeVelocity() method. Sadly, this also means that the static String[] xmlToolingsConfigs must be copied.</div>
<div><br></div><div>I fully agree that initially, this is a Velocity problem, since it seems that it can only be initializes in a static way. However the same is true for OpenSAML, although there is a relatively simple workaround for it.</div>
<div><br></div><div><br></div><div>Thanks,</div><div>Tom</div><div class="HOEnZb"><div class="h5"><div><br><br><div class="gmail_quote">On Fri, Apr 27, 2012 at 9:02 PM, Brent Putman <span dir="ltr"><<a href="mailto:putmanb@georgetown.edu" target="_blank">putmanb@georgetown.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br>
<br>
On 4/27/12 7:25 AM, Tom van den Berge wrote:<br>
><br>
> I think it would be far better to have this optional in OpenSAML<br>
> itself though. Duplicating code is a major sin ;)<br>
><br>
<br>
</div>Note that you don't have to literally copy/duplicate code to do this.<br>
You can just extend the DefaultBootstrap class and override the<br>
initializeVelocity() method to be a no-op. Then invoke that bootstrap<br>
subclass instead of ours.<br>
<br>
That said, I agree that it's not optimal for us to initialize Velocity<br>
like that. We probably can't fix this for v2, since we aren't planning<br>
on anything but patch releases for that going forward. But if you'd<br>
like to file a Jira issue against v3, we can examine whether we can<br>
rework how we use Velocity to avoid the static init. It would actually<br>
be more crucial to fix it there, because the custom bootstrap workaround<br>
won't be there in v3, the library init there is very different.<br>
<br>
Thanks,<br>
Brent<br>
<div><div><br>
<br>
<br>
--<br>
To unsubscribe from this list send an email to <a href="mailto:dev-unsubscribe@shibboleth.net" target="_blank">dev-unsubscribe@shibboleth.net</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>