« first day (4278 days earlier)      last day (653 days later) » 

12:14 AM
Oops
 
@IGP the parameter type is int, you're passing array in two. The second one you're doing, I think the named argument has to appear after
Not sure exactly if variadics are supported or not, but there are other issues with your example
 
IGP
@Tiffany Yes, that is true. From what I can tell there is no way of specifying a variadic argument. I read the documentation for function_arguments and the rfc for named params but it's not really explicit.
about that bit.
And the "what does X mean in PHP" question in SO even has that bit left out. stackoverflow.com/questions/3737139/…
 
@bwoebi It does. I'm not familiar enough with gdb to know what I'm doing, though. :-)
 
12:39 AM
Speaking of IDEs, I think github.com/php/php-src/pull/8912/… somehow garbled odd characters.
 
IGP
12:58 AM
That's pretty much what I wanted to see. Thanks Danack. I feel like that should be stated explicitly somewhere. Maybe here in an example 21 https://www.php.net/manual/en/functions.arguments.php
Is there a way to directly contribute to the php docs?
 
@IGP it's all in github.com/php/doc-en but @Tiffany might be able to point you to detailed instructions.
 
IGP
Cool. I'll take a look
 
 
11 hours later…
12:04 PM
Is it possible a http "Fire & Forget" request with Fiber?
I want to send a http request to a 3rd party but dont want to wait for response
 
12:16 PM
I could be wrong, but I think all you need for that is a non-blocking socket
 
PHP Tests: Dependent on having trailing whitespace at the end of lines
IDE: So you have chosen death.
2
 
 
2 hours later…
cmb
2:02 PM
Does anybody know which PHP version runs on php.net? The README states: "Code must function on a vanilla PHP 7.2 installation." But do we really still run 7.2 there?
 
Wes
can i typehint this in phpstorm somehow? /** @var Tuple<int, float, string> */
meaning array based tuples [12, 12., "12"]
 
cmb
not sure about PHPStorm, but PHPStan supports array-shapes, e.g. array{int, float, string}
 
2:17 PM
@cmb Psalm supports the same. I believe PhpStorm supports it, as well
 
Wes
2:30 PM
that works, more or less. at least i am getting autocompletion
ty muchly
 
2:50 PM
@Danack maybe I should type up a bulleted list of where and how to help.... though that should be present in doc.php.net... I guess I revise something present and PR it in
@IGP that's where docs can be cloned. Depending on how much you change, it may be preferable to set up a local environment to test the changes. The local environment uses doc-base to validate the XML, then uses phd to render the docs into a viewable format (e.g. XHTML or PHP, but XHTML is easier to deal with since it doesn't require a web server)
@IGP Info can be found doc.php.net/tutorial/local-setup.php on setting up local environment
You'll probably want to fork the repos to your own GitHub, and update the commands you run accordingly
@cmb I thought it was 7.2?
well, we have the powered header turned off :P
It might be 8.0, but I can't remember if it was updated for sure
 
@Tiffany In unrelated news, I just made it to level 100 on co-op \o/ Zeratul OP
 
cmb
3:24 PM
@Tiffany Possible; wiki.php.net/systems/php-web2 doesn't have any info about that.
I find it strange that we avoid third-party packages to not endorse these, but stick with old PHP versions, which we (passively) endorse. :p
 
@cmb Hear hear!
 
4:05 PM
@cmb 💯
@MarkR nice!
 
4:44 PM
@cmb Wasn't it @Derick who moved all our infra and killed mirrors to make https everywhere? Pretty sure we updated the de-facto PHP.net version when we did that to whatever was the latest at the time.
@IMSoP Yep, that's the one.
 
@LeviMorrison Maybe Derick led that effort, but I think Sascha's company hosts everything
@Derick More errors again. Probably also because I'm trying to mix C and C++, but I can't figure out what I need to do to solve this.

Code: https://github.com/ramsey/ext-ecma_intl/compare/more-testing

Error messages: https://gist.github.com/ramsey/6e9322b0fb7ceaea839834ea75b52a40
 
You're compiling a C header file that has a "new" keyword in it.. .which is a restricted word in C++
Shouldn't you be using the C++ interface of ICU instead here?
 
5:00 PM
hmm... probably. When I was starting out, I didn't know the difference :-)
 
cmb
The mirrors have been retired early in 2019. At that time 7.3 was current, but I guess it's actually about some distro (Debian?).
 
Sascha's company does the cloud front ened... I have access to the actual servers
 
cmb
@Derick Maybe you can fill in some info at wiki.php.net/systems/php-web2. I'm particularly interested in the PHP version.
 
php-web2 is retired I think, and we use -web4 now
 
cmb
Would be nice to list it in the Wiki. :)
 
5:04 PM
it uses debian 10, using PHP 7.3.19
 
PHP 8.1 + Containers is where it's at, if we want to use our own industry best practices.
 
@Derick Actually, I think I am using the C++ API
 
i see icu4c
 
that's the only one they have... they mix the C and C++ APIs together
 
oh, right
i misassumed then
 
5:07 PM
But I do think my error has to do with using C in C++ or vice versa
I'm just not sure why it works in my extension code but I can't get it to work in the test
Other than that the test file itself is C++
 
sorry, don't have the time to look at it now
 
5:24 PM
@MarkR Using a stable, supported OS version is an industry best practice ^_^. We should probably migrate to Debian 11 soon though because 10 goes EOL in a few months, I think?
Debian 11 has PHP 7.4.21, for reference.
 
@Derick No worries. I'm not in a hurry. :-)
 
@LeviMorrison 10 goes into LTS in a month or so. After that it's 2 years of LTS.
 
@TimWolla Right. Still, probably should upgrade to Debian 11 around then.
Debian uses "EOL" for going into LTS BTW: wiki.debian.org/DebianReleases.
 
cmb
@Derick thanks
 
Yes, the LTS is not supported by the Debian Security Team, but instead by a private company. The updates are available using the standard package servers, though.
I can certainly recommend upgrading before the LTS comes into effect. In my experience the Debian updates have always been very smooth.
 
cmb
5:28 PM
@ramsey Have you tried including CppUTest/TestHarness.h later (after the other includes)?
 
@TimWolla I run Debian unstable, as that's much smoother to update. I can attempt a normal upgrade for this server, but I need to figure out how to get console access first.
 
Complexity depends on the amount of non-stock packages and whether or not there is any configuration management. Official release notes are here: debian.org/releases/bullseye/amd64/release-notes/index.en.html.
My opinion is, as long as the machine boots after the upgrade, everything is fixable :-D
 
yeah, hence the console access...
I doubt there are an non-stock packages
 
5:43 PM
@cmb Are you going to merge the finished PRs or are you going to make me self-merge? :-(
 
cmb
I've merged 1670 and 1671.
 
Yay. How about switch and str_split?
 
cmb
6:22 PM
I merged 1665, and left a comment on 1658.
 
7:04 PM
I'm good with whatever on mkdir (1666). Your call, make a decision. :-)
 
hah, I thought that 1666 was permissions to give to mkdir :D
 
cmb
7:32 PM
Merged 1666. We can still improve later. :)
 
7:45 PM
Whee!
 
Mornings / evenings
 
8:11 PM
I want the following to format the date like 3-7-2022
$sth->bindParam(array('Updated_at',$Updated_at=>date("Y-m-d"),PDO::PARAM_STR));
what to write here $Updated_at=>date("Y-m-d"),....,
$sth->bindParam(array('Updated_at',$Updated_at=>date("Y-m-d"),format(..........),PDO::PARAM_STR));
Oh
it formatted perfectly but
I get this error
Fatal error: Uncaught ArgumentCountError: PDOStatement::bindParam() expects at least 2 arguments, 1 given in
 
8:51 PM
@Derick @cmb I switched to using Criterion for unit tests (instead of Cpputest), and that seems to work much better for me. github.com/ramsey/ext-ecma_intl/compare/criterion-tests
 

« first day (4278 days earlier)      last day (653 days later) »