« first day (4437 days earlier)      last day (502 days later) » 

12:14 PM
\o
Is there a document somewhere containing a list of what PHP considered a constant expression?
 
 
2 hours later…
2:19 PM
Morning
 
Isn't there a document somewhere which lists the "probably maintainer" for each extension?
At least each extension in ext/ in the src
 
cmb
3:04 PM
That's not really up to date, though.
 
Personally as one of the persons listed for ext/random would be interested in CODEOWNERS: github.com/php/php-src/pull/8670
I've also wondered before, whether it would make sense to create a GitHub team for each team of release managers, I believe teams can be requested for review as well. It would be easier to request review from "release-managers-8.2", than each of the three folks individually.
 
cmb
3:47 PM
@TimWolla I'm not against that, although I'm slightly concerned about the term code owner (nobody should own any parts of php-src in the sense that they decide what may change there and what not).
Also, the basic problem is that we have only a few real extension maintainers.
@TimWolla IMO, unnecessary overhead. I usually only ping RMs if something is actually a RM decision, but not for any PRs which target the respective branch.
 
@StatikStasis Sinking ship or wreckage. :D
 
@cmb Yeah, CODEOWNER is the terminology GitHub uses, personally I'm rather interested about the automated review requests (not the branch protection part). As a ext/random maintainer, I'd like to take a look at the PRs at the very least (so that I can keep myself up to date with whatever happens) and for for other code it's not always obvious who would be a good person to review request.
So I'd be happy with a “self-service” solution where folks add themselves for parts they are interested in.
 
cmb
4:06 PM
@TimWolla Just replied on the PR. :)
 
@cmb New comment on bugs.php.net/80047 ...
 
cmb
4:22 PM
replied :)
 
4:59 PM
https://github.com/php/php-src/commit/ac3ecd03af009d433d4b75d570b3b0f0a3fc0ff7
Ugh. Is the [ci skip] detected *anywhere* in the commit message, instead of just the subject line?
 
cmb
@TimWolla I think that depends on the CI provider. AppVeyor only recognizes it in the title (and possibly only at the beginning of the title).
 
Ah. GHA appears to detect it anywhere, I would've liked the tests to run after the merge, but oh well. They'll be included with whatever push comes next, then :-)
 
cmb
5:19 PM
@TimWolla, Translation can be hard. :p
 
It really surprises me that heise has such a big ass GDPR violation.
THey're requiring me to opt in into trackers, or subscribe. That's not allowed.
 
cmb
Well, disable JS, and try again. (I haven't tried there, but what happens elsewhere is sometimes ridiculous.)
 
5:35 PM
@LeviMorrison I'm thinking on doing some work on Auryn and doing a release of it. Do you have any opinions on dropping support for PHP < 7.3 ?
 
What's Auryn?
 
A really bad name for a project as no-one knows how to pronounce it.
 
uh... how is that hard to pronounce? :-D
 
Which one did you go for? Ow-run, Or-run, Ow-rian? or something else?
 
ow rin
"yn" is used a lot in welsh (it means "the") and is pronounced "in" so that's what I was going with
 
Yes, I was familiar with the word :-þ
that's not quite a 100% match though. It's just "aur"
but google translate is silly
 
@Derick yep, that's how I always pronounced it too
 
6:31 PM
@cmb At first I thought: It's not too bad. Then I saw the headline.
@Derick You're seeing this type of banner for roughly every online German news magazine.
And now I made the mistake of clicking into the Heise Forum comments for the article …
 
 
1 hour later…
7:53 PM
@Danack it's become "injector" in Amp
 
8:15 PM
@Tpojka A little bit of both, plus ships with holes in sails but happy faces
 
8:43 PM
@Danack Definitely can/should drop PHP 5.
I haven't had any contact with Daniel. I've reached out but never heard back :(
 
@LeviMorrison I've actually never communicated with him other than here.... He's been active on twitter recently, so I'll try there.
 
Hi, What do you think var_export(new class{}); should be returning? Currently this edge case produces an invalid code. I'm looking for opinions :) github.com/php/php-src/issues/10066
 
@Danack Probably should start with the PHP 8 PR that's already open? Dunno.
 
@LeviMorrison And okay to drop the 7.0, 7.1 and 7.2? The reason is that the current PHPUnit runs on 7.3+, so it would take effort to continue to support those versions.
@LeviMorrison I'm too 'tired' to push now, but I've got a dev branch that fixes all known issues, and has CI. github.com/Danack/Auryn/actions/runs/3659982748 I didn't actually look at that PR before doing the work, but it has some silly style changes which would have been annoying to revert.
 
How hard would it be to use PhpUnit 8 instead so Ubuntu 18.04 LTS is still supported?
 
8:50 PM
Probably the only thing that you might have strong thoughts about is the words I modified for the maintenance status.
@LeviMorrison I'll check tomorrow but not hard - there's a symfony bridge library for installing appropriate phpunit versions: github.com/Danack/Auryn/commit/…
 
I'm fine with the language, assuming you don't hear from Daniel.
 
So to run the tests locally you need to install then run phpunit, rather than just single command.
@LeviMorrison thanks.
 
I do prefer that we maintain compat with Ubuntu LTS releases which are still in regular support (not the extended commercial version). This is generally a virtue, though sometimes you really do need something from newer PHP versions.
And, of course, bump it to 2.0 since we're dropping so many PHP versions.
Well, that may not be "obvious" I guess.
 
Honestly, I think if I had more energy that could be something we could argue over. But I seem to recall there being quite a few numbers between 1 and the maximum allowed in semver, so sure. Assuming I am able to tag it....
 
9:45 PM
Hope everyone has a great weekend!
 
9:57 PM
@StatikStasis Hah.
Have a great weekend!
 
10:11 PM
@Krzysiek anonymous class seems correct to me
ah, nevermind, I am misunderstanding how var_export() is supposed to work
I guess the problem is that it looks like var_dump output rather than giving valid PHP code? Haven't completely read through the proposed solutions in one of the comments, but trying to form my own to see if I have the right idea.
 
10:34 PM
@Tiffany Yes, probably fixing the name would be sufficient, though you can import these object only in specific conditions. On the other hand you can't import Closures/Generators at all, but these are exported anyway. var_export is somewhat broken in the core concepts, so I'd love to hear what people would expect in this scenario as the output. Thanks for the response :)
 
10:52 PM
typing my thoughts into a chatbox:
returning a stdClass object seems incorrect. while a stdClass object is a generic class object, in my mind, it's distinct from an anonymous class.
thinking through another possibility ... having var_export output "new class{}" would be... a solution I don't want to see...
at first, I was in disagreement, but after reading through @cmb's points, it makes sense... but I'm not sure if the right way to fix this is updating the docs
at minimum, docs should be updated though
 
11:13 PM
@Krzysiek IIRC the ::__set_state() issue was already before PHP 7. IMHO it's fine you can't import the export as not possible. this prevents the __PHP_Incomplete_Class situations that are far older (so fails earlier which I think is a good approach). What you suggest looks a bit like you want to eval that code which I would to restrain from and instead prefer that if eval'ed throw a parse exception instead.
 
11:41 PM
@Krzysiek and you can't even import classes, even if they extend stdClass, if their definition is not available. The level of error/exception is pretty much the same as of PHP 7 wich introduced anonymous classes you're picked as an example class.
So while the root cause to trigger the exception might be different, the outcome is of the same kind, an exception you can catch since PHP 7 which is the version introducing anonymous classes. So since PHP 7 this is valid PHP code as PHP does not exit with status 255 uncontrolled from the code being executed (to give it a bit of a stretch, take it with a grain of salt).
 

« first day (4437 days earlier)      last day (502 days later) »