Den tir. 26. jun. 2018 kl. 22.01 skrev Nikita Popov <nikita....@gmail.com>:
> Thanks for adding it. Unless there is a particular reason to rush here, I'd 
> suggest to follow the usual process and formally deprecate the option prior 
> to removal.

I don't think theres a need for a rush here, tho I wasn't sure how you
prefered we handled the ini directive deprecation, I thought adding it
to main.c would be too "noisy".

> As a clarification, what is the alternative to using this ini setting? I 
> guess you can just explicit set the DB2INSTANCE env var, but that probably 
> has the same issues. Is there a "correct" way to do this? (Sorry if it's 
> obvious, I'm not familiar with ODBC.)

I'm not too familiar with it either, I just remember it from way back
then when I began the crusade to kill old functionality in the 5.3
development days. I don't think there is any other alternative than
setting it manually, which still creates the same issues as what it
already does not but forces our users to be explicit about it. I
personally don't like such magic behind the scenes.

As for any correct way, I don't think there is any other, given how
the ibm_db2 pecl package mimics the same behavior as pdo_odbc does,
but then again the IBM folks have not been the fastest to update their
extensions, so maybe there there is a new way to do it now but I doubt
it. I don't really have the interest to dig too much into it either.

-- 
regards,

Kalle Sommer Nielsen
ka...@php.net

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to