Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-04 Thread Christoph M. Becker
Hi Kalle! On 04.05.2019 at 13:33, Kalle Sommer Nielsen wrote: > Den lør. 4. maj 2019 kl. 14.19 skrev Christoph M. Becker : >> >> On 04.05.2019 at 10:56, Christoph M. Becker wrote: >> >>> I have submitted PR #4112 to bring back the common PDO tests, which are >>> the majority of existing tests

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-04 Thread Kalle Sommer Nielsen
Hi Christoph Den lør. 4. maj 2019 kl. 14.19 skrev Christoph M. Becker : > > On 04.05.2019 at 10:56, Christoph M. Becker wrote: > > > I have submitted PR #4112 to bring back the common PDO tests, which are > > the majority of existing tests for pdo_firebird. Three tests are > > failing (which

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-04 Thread Christoph M. Becker
On 04.05.2019 at 10:56, Christoph M. Becker wrote: > I have submitted PR #4112 to bring back the common PDO tests, which are > the majority of existing tests for pdo_firebird. Three tests are > failing (which should be investigated), but these tests already failed > in PHP-7.3 and maybe before.

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-04 Thread Christoph M. Becker
Hi Kalle! On 03.05.2019 at 19:00, Kalle Sommer Nielsen wrote: > Den fre. 3. maj 2019 kl. 12.02 skrev Christoph M. Becker : > >> While running the tests with a PHP debug version, I've noticed that >> bug_aaa.phpt is causing a memory leak. Maybe someone proficient with >> Firebird/Interbase will

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-03 Thread Kalle Sommer Nielsen
Hi Christoph Den fre. 3. maj 2019 kl. 12.02 skrev Christoph M. Becker : > While running the tests with a PHP debug version, I've noticed that > bug_aaa.phpt is causing a memory leak. Maybe someone proficient with > Firebird/Interbase will want to have a look at it. Thank you for testing, good

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-03 Thread Christoph M. Becker
On 02.05.2019 at 22:21, Kalle Sommer Nielsen wrote: > Den tor. 2. maj 2019 kl. 22.55 skrev Lester : >> The problem with including a database is it's dependent on the version >> of FB running. This will only run with FB3 while FB2.x is still in >> common use hence it being preferable to use

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Kalle Sommer Nielsen
Hi Den tor. 2. maj 2019 kl. 22.55 skrev Lester : > The problem with including a database is it's dependent on the version > of FB running. This will only run with FB3 while FB2.x is still in > common use hence it being preferable to use something supplied with the > FB installation. > >

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Lester
On 02/05/2019 19:51, Kalle Sommer Nielsen wrote: It also includes a 'php.fdb', which I'm not certain whether or not we should keep as its quite large, but at least it makes the tests run for me with 15/15 pass. The problem with including a database is it's dependent on the version of FB

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Kalle Sommer Nielsen
Hi Christoph Den tor. 2. maj 2019 kl. 16.57 skrev Christoph M. Becker : > > Hi Kalle, > > On 02.05.2019 at 15:35, Kalle Sommer Nielsen wrote: > Indeed, it seems that pdo_firebird requires to connect to an existing > database, contrary to e.g. pdo_mysql which supports DSN like >

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Lester
On 02/05/2019 18:58, Christoph M. Becker wrote: Glancing that the ext/pgsql and ext/pdo_pgsql tests, though, it looks like we just require the user to have run "createdb test" before running the test suite. Indeed, see .

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Christoph M. Becker
On 02.05.2019 at 18:58, Rowan Collins wrote: > On Thu, 2 May 2019 at 14:57, Christoph M. Becker wrote: > >> Indeed, it seems that pdo_firebird requires to connect to an existing >> database, contrary to e.g. pdo_mysql which supports DSN like >> `mysql:host=localhost`, to my knowledge. > >

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Rowan Collins
On Thu, 2 May 2019 at 14:57, Christoph M. Becker wrote: > Indeed, it seems that pdo_firebird requires to connect to an existing > database, contrary to e.g. pdo_mysql which supports DSN like > `mysql:host=localhost`, to my knowledge. > Postgres works the same way - a connection is always to a

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Lester
On 02/05/2019 15:24, Christoph M. Becker wrote: A patch would be welcome! Looks like every file needs modifying ... I'll see what I can do, but my local php-src is broken and will not sync currently :( -- Lester Caine - G8HFL - Contact -

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Christoph M. Becker
On 02.05.2019 at 16:04, Lester wrote: > On 02/05/2019 14:35, Kalle Sommer Nielsen wrote: > >> Yeah that is one thing that is kind of unfortunate and a bad practice. >> However since interbase was disabled for AppVeyor, these tests had no >> chance to run in a while. Looking at the actual code in

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Lester
On 02/05/2019 14:35, Kalle Sommer Nielsen wrote: Hi Christoph Den tor. 2. maj 2019 kl. 14.56 skrev Christoph M. Becker : A problem is that the ext/pdo_firebird tests rely on ext/interbase to create a test database[1], which would require everybody who intends to run these tests to install

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Christoph M. Becker
Hi Kalle, On 02.05.2019 at 15:35, Kalle Sommer Nielsen wrote: > Den tor. 2. maj 2019 kl. 14.56 skrev Christoph M. Becker : >> A problem is that the ext/pdo_firebird tests rely on ext/interbase to >> create a test database[1], which would require everybody who intends to >> run these tests to

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Kalle Sommer Nielsen
Hi Christoph Den tor. 2. maj 2019 kl. 14.56 skrev Christoph M. Becker : > A problem is that the ext/pdo_firebird tests rely on ext/interbase to > create a test database[1], which would require everybody who intends to > run these tests to install PECL/interbase package now. It would be > great

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-05-02 Thread Christoph M. Becker
Hi! On 23.04.2019 at 18:55, Kalle Sommer Nielsen wrote: > Den tir. 9. apr. 2019 kl. 20.02 skrev Kalle Sommer Nielsen : > > The two weeks voting period for this RFC has passed and the RFC has > been passed with 46-0 in favor. I will prepare relevant practical > tasks for this RFC throughout the

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-23 Thread Kalle Sommer Nielsen
Hi Den tir. 9. apr. 2019 kl. 20.02 skrev Kalle Sommer Nielsen : > > Evening Internals > > Apologies for the one day delay, but the voting for the RFC "Unbundle > ext/interbase" is open[1]. There is only one voting choice, which > decides if the extension should be moved to PECL. > > Since this

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Pierre Joye
Hi Martin, On Mon, Apr 15, 2019, 5:08 PM Nikita Popov wrote: > On Mon, Apr 15, 2019 at 11:51 AM Köditz, Martin > wrote: > > > Hello everybody, > > > > I want to give you the general state of the Firebird driver. At the end > of > > January, I agreed to continue developing the Firebird driver

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Dan Ackroyd
Hi Martin, On Mon, 15 Apr 2019 at 10:50, Köditz, Martin wrote: > I agreed to continue developing the Firebird driver as a maintainer. As mentioned previously, possibly the first thing you should do is to make clear if you're maintaining it as something that is compatible with Firebird, or

AW: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Köditz , Martin
An: Köditz, Martin Cc: internals@lists.php.net Betreff: Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase On Mon, Apr 15, 2019 at 12:17 PM Köditz, Martin wrote: > Hi Nikita, > > > To update the documentation you can either use the editor at > edit.php.net (useful for smaller c

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Nikita Popov
On Mon, Apr 15, 2019 at 12:17 PM Köditz, Martin wrote: > Hi Nikita, > > > To update the documentation you can either use the editor at > edit.php.net (useful for smaller changes and does not require > permissions), > > or also request direct commit access to the documentation. > I've added the

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Köditz , Martin
I think it will be a good practice to request direct access. Thank you Martin -Ursprüngliche Nachricht- Von: Nikita Popov [mailto:nikita@gmail.com] Gesendet: Montag, 15. April 2019 12:08 An: Köditz, Martin Cc: internals@lists.php.net Betreff: Re: [PHP-DEV] Re: [RFC VOTE] Unb

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Nikita Popov
On Mon, Apr 15, 2019 at 11:51 AM Köditz, Martin wrote: > Hello everybody, > > I want to give you the general state of the Firebird driver. At the end of > January, I agreed to continue developing the Firebird driver as a > maintainer. In this regard, I first took care of getting familiar with

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-15 Thread Köditz , Martin
Hello everybody, I want to give you the general state of the Firebird driver. At the end of January, I agreed to continue developing the Firebird driver as a maintainer. In this regard, I first took care of getting familiar with the code and an important bug (# 72175). Obviously, this

Re: [PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-09 Thread Joe Watkins
Hi Kalle, You forgot to move to voting on the RFC index. Cheers Joe On Tue, 9 Apr 2019, 19:07 Kalle Sommer Nielsen, wrote: > Den tir. 9. apr. 2019 kl. 20.02 skrev Kalle Sommer Nielsen >: > > > > Evening Internals > > > > Apologies for the one day delay, but the voting for the RFC "Unbundle >

[PHP-DEV] Re: [RFC VOTE] Unbundle ext/interbase

2019-04-09 Thread Kalle Sommer Nielsen
Den tir. 9. apr. 2019 kl. 20.02 skrev Kalle Sommer Nielsen : > > Evening Internals > > Apologies for the one day delay, but the voting for the RFC "Unbundle > ext/interbase" is open[1]. There is only one voting choice, which > decides if the extension should be moved to PECL. > > Since this