<div dir="ltr">merged, thanks to everyone</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 14, 2013 at 9:49 AM, thomas bonfort <span dir="ltr"><<a href="mailto:thomas.bonfort@gmail.com" target="_blank">thomas.bonfort@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1 for merging<br>
<div class="HOEnZb"><div class="h5"><br>
On 14 August 2013 09:46, Umberto Nicoletti <<a href="mailto:umberto.nicoletti@gmail.com">umberto.nicoletti@gmail.com</a>> wrote:<br>
> On Wed, Aug 14, 2013 at 9:39 AM, Andreas Albarello<br>
> <<a href="mailto:andreas.albarello@tol.bz.it">andreas.albarello@tol.bz.it</a>> wrote:<br>
>><br>
>> Umberto,<br>
>><br>
>> thank you. I think this should be ok, and I agree with Thomas' comment to<br>
>> the original pull request that OCI_THREADED should require USE_THREADS to be<br>
>> defined.<br>
>><br>
><br>
> I have added that after I sent the first email, so I suppose it's good for<br>
> merging.<br>
><br>
> Umberto<br>
><br>
>><br>
>> Unfortunately, I lost my MapServer build environment in a HDD crash two<br>
>> weeks ago and haven't yet had time to put everything together again, so I<br>
>> can't help with testing this right now. I will say though that I have been<br>
>> compiling MapServer with OCI_THREAEDED in maporaclespatial.c for some time<br>
>> now and haven't noticed any adverse effects. As I said, I did find though<br>
>> that some weird crashes in conjunction with an 11g client (10g clients did<br>
>> seem to be a lot more stable in this regard) went away which I think is a<br>
>> good thing and what prompted my response in the first place.<br>
>><br>
>> Regards,<br>
>><br>
>> Andreas<br>
>><br>
>> On 14.08.2013 09:07,<a href="mailto:umberto.nicoletti@gmail.com">umberto.nicoletti@gmail.com</a> wrote:<br>
>> > I have added OCI_THREADED to the flags passed to OciEnvCreate as per<br>
>> > Andreas suggestion (and fixed a few compile warnings while at it).<br>
>> ><br>
>> > You can checkout the changes:<br>
>> ><br>
>> > <a href="https://github.com/mapserver/mapserver/pull/4729" target="_blank">https://github.com/mapserver/mapserver/pull/4729</a><br>
>> ><br>
>> > I'd like to merge this into master today, so I'd really appreciate it if<br>
>> > someone could test it right away. I'll be on #mapserver if you need<br>
>> > assistance.<br>
>><br>
>><br>
>> --<br>
>> --------------------------------------------------------------------<br>
>> Andreas Albarello<br>
>> Analysis & SW Development<br>
>><br>
>> Territorium Online srl/GmbH<br>
>> Via Buozzi Str. 12<br>
>> I 39100 Bolzano/Bozen<br>
>><br>
>> Phone:  <a href="tel:%2B39%200471%20068611" value="+390471068611">+39 0471 068611</a><br>
>> Fax:    <a href="tel:%2B39%200471%20068619" value="+390471068619">+39 0471 068619</a><br>
>><br>
>> email: <a href="mailto:andreas.albarello@tol.bz.it">andreas.albarello@tol.bz.it</a><br>
>> web:   <a href="http://www.tol.bz.it" target="_blank">http://www.tol.bz.it</a><br>
>> --------------------------------------------------------------------<br>
>><br>
>><br>
><br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> mapserver-dev mailing list<br>
> <a href="mailto:mapserver-dev@lists.osgeo.org">mapserver-dev@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/mapserver-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/mapserver-dev</a><br>
><br>
</div></div></blockquote></div><br></div>