« first day (4351 days earlier)      last day (821 days later) » 

00:12
@TimWolla The irony of the PHP License being so bad, that it turns out to be too scary to fork, which turns out to be a really useful property of an open-source project, is beyond irony.
@Sara there will be a security thing
 
3 hours later…
03:26
@FĂ©lixAdriyelGagnon-Grenier thinking about this again, in fairness, if someone is going to be a jerk, regardless of their "importance" or believed "importance" ... well ... they deserve the kick :P
or message deletion, etc
TL;DR, don't be a jerk
I want to revise that, but can't think of the right word right now. "don't be a jerk" is ambiguous in the sense of the person on the other end... and there are thresholds for different people/cultures... but like, if called out on being a jerk by an RO, an apology is a good idea, and not continuing the behavior that was called out is also a good idea
04:02
lol so many ros here
 
3 hours later…
06:56
I think we have a major ABI/API breakage in 8.2.0RC2, ping @bwoebi
07:43
also ping @SergeyPanteleev
@RemiCollet I'll retag after the issues are fixed, I'm reading the email now
Mmx
Mmx
08:14
@Danack Maybe useful use queue - like a rabittMQ, create Microservices with many workers? Or React or other JS Framework will works after, better?
@RemiCollet pong, how exactly?
@bwoebi see my emails, it is about github.com/php/php-src/commit/…
@bwoebi He wrote on the ML
an, not ML, release manage "list"
uh, the change is not supposed to break API, only ABI
let me check
@RemiCollet I could be wrong, but I reverted locally these 3 commits, and got the same tests results, as before the reverting :thinking:
08:30
@bwoebi Isn't the ABI break the increase of the size of the CE?
yes struct size is a problem
@bwoebi if you have to fixx all ext as in github.com/php/php-src/commit/…
this is a big issue, especially as RC1 should have final API/ABI
and also would have be better to also break API, ex removing/renaming foo->std.handlers so it cannot be set, and thus break the build of ext not using the "new" way
btw, IMHO, not suitable for 8.2 during RC phase
08:54
@Girgias yes, I expect that. but afaik abi freeze always only has been with release?
@RemiCollet I mean, the build should not have been broken. It just would need a rebuild due to the ABI break.
Or is there actually a crash when rebuilt?
@bwoebi main question what happen if you don't fix ext (github.com/php/php-src/commit/…)
@bwoebi Don't quote me on that but I think the assumption is that RC1 should be ABI stables so all extensions can make the necessary adjustments for GA
@Girgias API - for ABI you just need a rebuild, no adjustments.
@RemiCollet if you don't change ext, it'll just set its class handlers in create_object handler and continues to work happily
Well I know for ABI you just need a rebuild, but yeah again I'm not the RM process expert here, I'm just here to break stuff :P
cmb
cmb
09:00
@bwoebi If ABI changes after RC1, ZEND_MODULE_API_NO would need to be bumped at the very least.
That change is specifically designed to not require a change to code - because yeah, otherwise, it'd really need fixes to all exts, which wouldn't be great
@cmb oh, this is ... ABI? I expected it by name to be something API related
cmb
cmb
@bwoebi It's actually both,, I think. An extension built against previous API_NO should not be loadable for newer API_NO. See also phpinternalsbook.com/php7/build_system/…
@cmb Also I wasn't aware these API no's are supposed to be touched pre-GA. So, do you suggest I change it for RC3?
@bwoebi ABI changes also need the API_NO updated...
@bwoebi I think you should back these commits out of PHP 8.2.
+1
and document them in UPGRADING.INTERNALS for 8.3
cmb
cmb
09:07
@bwoebi I think we need to re-tag anyway. Either reverting the "offending" commits or bumping API_NO.
And what Remi said for master: "removing/renaming foo->std.handlers so it cannot be set"
@Derick I don't quite like that to just break every single extensions code for little gain.
better signal that something needs updating then silently ignoring it...
@Derick but it doesn't need updating
it's optional to
also notice, that bumping ABI wil mean mass rebuild.... >100 ext for me... probably 2-3 days of work (especially because of very slow aaarch64 build)
cmb
cmb
09:10
@Derick Better postpone that for PHP 9.0. :)
but yeah, then let me revert it on PHP-8.2 for now.
@cmb that was my intention
cmb
cmb
+1
assuming PHP-9.0 is going to bring a bit bigger engine changes anyway
09:22
@RemiCollet @cmb @Derick done
Whom do I need to annoy about the retag of PHP 8.2 RC2 then?
:+1:
@SergeyPanteleev ^^
@bwoebi if think you also need to revert 800c6672e57a01d68dcaba36f8f7d65c6871aafc
After reverting the 800c667, I'm ready to retag :)
github.com/php/php-src/commit/… => commit message looks very strange ;)
@RemiCollet yes, indeed. that confused me
@RemiCollet done \cc @SergeyPanteleev
09:29
@bwoebi thanks
@bwoebi Thanks! I start to retag
09:58
@SergeyPanteleev I see a new tarball in your directory, is it OK to pull it ?
@Girgias I do actually check CI output ;-) (saw your comment after I already fixed it)
I had just opened a PR and saw it was red :p
I like being able to run with -j64 for "make test"
TEST 15563/15456 [1/64 concurrent test workers running]
Somebody can't count...
Next up:

=====================================================================
EXPECTED FAILED TEST SUMMARY
---------------------------------------------------------------------
DateTime::sub() -- spring type3 type3 [ext/date/tests/DateTime_sub-spring-type3-type3.phpt] XFAIL REASON: Various bugs exist
DateTime::add() -- spring type3 type2 [ext/date/tests/DateTime_add-spring-type3-type2.phpt] XFAIL REASON: Various bugs exist
DateTime::diff() -- fall type3 type3 [ext/date/tests/DateTime_diff-fall-type3-type3.phpt] XFAIL REASON: Various bugs exist
which is going to be a major pain, but the last outstanding bug
cmb
cmb
@Derick known issue; the redirect tests are only counted once for total, but may run for several exts
10:48
It's still funny.
cmb
cmb
Well, where is 7.4.0beta3? :p
I went with powers of 2, duh.
11:44
class Foo {
    public function __construct(public int $bar, public string $foo) {}
}
$foo = new Foo(...['foo' => 'Fooo', 'bar' => 123]);
var_dump(
    $foo,
    new Foo(...(array)$foo),
    new Foo(bar: 321, foo: 'fOOOO'),
);
This is simple example of intantiating a Foo class object using unpack arguments with their names
I can do that using array easily, but why do I need an object to cast to array before?
if my public property names match in my DTO's I could avoid repacking properties into arguments when their the only source of arguments, like create a command from all or just subset of event properties.
Does that make sense to simplify the instantiation of one based on second object?
12:09
o/
12:20
\o
 
2 hours later…
14:04
docker-compose pastebin.com/tvtS2kpV
I'm tring docker for the first time, php-fpm + nginx, But nginx complaint about "Connection refused while connecting to upstream, client, fastcgi://172.18.0.4:9000" (502 Bad Gataway)
can anyone give me hint ? ty!
Looks like this is php-fpm problem, But it listening in the app container rn, fpm is running, pid 118
@Mwthreex I'm not on a good machine to check, but isn't nginx.conf in the directory /etc/nginx not in the conf.d subdirectory?
aka, I don't think you are mounting your nginx.conf file.
@Danack I have nginx.conf file in ./docker/nginx/conf.d folder, I don't think this is a problem because the nginx accept new changes from this file i tested it
14:24
I need a new trance mix to listen to. Help @MarkR you're my only hope. =)
@StatikStasis you man enjoy this one soundcloud.com/akbarw3x/trance-revulotion-radio-41-mixed-by-w3x I made it with virtualdj :)
Checking it out now...
Epic soundtrack feel... I like it so far.
@Mwthreex Very nice- thanks.
@Mwthreex if you put your stuff into a github repo, so it can be checked out and fiddled with easily, you are more likely to get help.
@StatikStasis Your welcome
14:43
@Mwthreex Are connection blocked on that port perhaps?
Do you have selinux enabled?
I'm in windows WSL 2 i don't know really
Check the audit log
The full nginx log:
22#22: *1 connect() failed (111: Connection refused) while connecting to upstream, client: 172.18.0.1, server: , request: "GET / HTTP/1.1", upstream: "fastcgi://172.18.0.4:9000", host: "localhost:8000"
where is 'audit log' ?
Wherever the rest of your logs are stored
Most likely /var/log
I can only see these inside the nginx container
14:50
no
You are in the nginx directory
What does sestatus give you
/bin/sh: sestatus: not found
Well you are probably not running selinux :P
Is this some minimal image like alpine?
14:55
You are on your own :D
Ok maybe i should install nginx from ubuntu base image
that is not a repo. And I strongly suspect it won't be working as it has COPY composer* . and there is no composer file.
Ah ok
15:17
So, from 5pm (in 45 minutes) you will be able to walk along the Queen's coffin to pay your respects. As we all know the Brits like queuing, DCMS now has a YouTube stream with the queue length... and map: youtube.com/watch?v=9NpZuGxSgZY
@Mwthreex I get a different error:
> nginx_1 | 2022/09/14 15:41:28 [error] 23#23: *1 FastCGI sent in stderr: "Primary script unknown" while reading response header from upstream, client: 172.25.0.1, server: , request: "GET / HTTP/1.1", upstream: "fastcgi://172.25.0.2:9000", host: "localhost:8000"
Yeah, I noticed it now, What is problem then ?
wait, do you mean you get that error now or do you still get the one you had earlier?
I had that first error in my real laravel project actually, This is a fresh laravel app but it has some problem too
> it has some problem
1 min ago, by Danack
wait, do you mean you get that error now or do you still get the one you had earlier?
15:45
I was on the master branch and I executed git pull origin new_branch .. Suddenly the new_branch changes got merged with the master branch
any idea how can I rollback ?
I have the issue you mention rn, I want to see what is the problem then apply it to my real project @Danack
@Shafizadeh Check the git reflog, then git reset --hard to the old state. Keep in mind that git reset --hard will remove all your local uncommited changes (and also all changes in commits you made after the commit you roll back to). If you are inexperienced with git, I recommend making a backup copy of the full directory (i.e. the entire working copy) in case something goes wrong.
I see .. ok thx
@Derick WOW! 2.5 miles long!
@StatikStasis They're expecting it to be up to 10 miles...
that's just three hours of walking it
My wife is planning to attend tomorrow. I say good luck to her.
16:03
@Mwthreex I'm reasonably sure the problem is that the default fastcgi_params file on that docker box doesn't appear to set fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
I have the exact line in the nginx.conf, What should i do now ?
Do you mean the docker i have in is no the same the docker you have ?
oh wait - yeah. I have it in the fastcgi_params file instead....but I guess it's fine in your 'nginx.conf' file.
16:27
@Derick "Give my regards to the Queen when you go..."
I am not going, i've better things to do
16:43
@Mwthreex so...I think it is due to the nginx + php boxes not agreeing on directory structure. You have your files mounted in /var/www/ on nginx and in /app on the PHP box.
So changing the line:
fastcgi_param SCRIPT_FILENAME $document_root/$fastcgi_script_name;
to
fastcgi_param SCRIPT_FILENAME /app/public/$fastcgi_script_name;
appears to make php-fpm find the script at least.
btw, is this docker compose config from an example or you've written from scratch?
There's two things that seem a bit sub-optimal. First compiling pecl extensions from source is kind of slow. If you use Debian as the base box, you can install them as packages.
Also, you shouldn't need to copy the files in, so the lines:
COPY composer* .

COPY . .
may be redundant.....they also mean that Docker has to completely rebuild the images each time you do docker-compose up, as they can't be cached reliably. Either removing those lines, or at least moving them after all the extensions are installed should speed up the docker-compose time.
For the directory stuff, I'd pretty strongly recommend mounting the files in the same directory on all containers. Having to remember which container you are in to find stuff gets quite annoying.
17:13
@Danack Thank you so much for helping me
The COPY composer* . you mentioned, I thought this is the right thing to do, It look into the composer.json if there was no change then it cache the layer right ?
I don't think it's smart enough to do that. I normally run docker-compose up --build and for me, if nothing has changed, that takes about 10 seconds as all the containers are cached. But your docker-compose config was recompiling extensions every time.
btw, putting all the container configs under a separate container directory e.g. is probably more sensible than having them distributed in different places in your project.
@Derick That was in quotes as I was imagining you stating to your wife.
 
2 hours later…
19:18
@Crell have you seen my twitter DM?
19:53
Man utterly pointless aggressivity
JRL
JRL
is there something i dont understand? it seems like the entire point of this comment is to basically criticize the contributors for occasionally having bugs in SPL?
Yes, and that nerging a bug fix in a patch release is bad
I'm really moving to the point of wanting to rewrite the whole IO layer of PHP just so I can get rid of this stupid piece of garbage which is called SPL]
Also "please write regression tests so the behaviour doesn't change"
JRL
JRL
well, that won't actually get rid of SPL, lol
How about, you contribute by writing them because apparently we break your application
like JEEEEEEEEEEEEEEEE
JRL
JRL
good luck getting that deprecated and removed
20:01
@JRL That is my ultimate objective, because it's pure utter garbage
I'll know it's time to retire from php-src the moment I get that done
@Girgias sending hugs your way, if that's your thing
@FĂ©lixAdriyelGagnon-Grenier I'll take them (>^w^)>
20:19
Relevant xkcd: xkcd.com/1172
Though I must admit that long-standing buggy behavior can be considered a "feature". It's similar to the ksort() bug that ultimately was reverted and the fix deferred to 8.2 (?).
@TimWolla I don't mind that it can be considered a feature. But if you are going to complain open a bug ticket ASAP instead of waiting for like 4 fucking months to whine about the behaviour
I get now why nobody likes touching SPL, the codebase is shit, and you try to fix it you get complaints that you fix the behaviour because it's been buggy for so long
Yeah, I totally get both sides here, because I've been on both sides myself. As a downstream consumer of PHP and as an upstream developer of an application where 3rd party developers rely on the API being stable.
FWIW: If you're curious, I had my own variant of "bugfix in patch release broke long-standing code of ours" here: github.com/php/php-src/pull/7839#issuecomment-1025983049
Fixed that, because the fix made the code cleaner anyway and moved on.
It's more I'm really wondering, what is the point of RC releases if nobody fricking uses them to test stuff
The majority of users are likely using distro packaged PHP (which doesn't necessarily even get bug fixes at all) or something like deb.sury (which only packages the stable versions to my understanding). To even make use of the RCs you also would need to have the necessary automated testing infrastructure in place, which likely often isn't the case even if you use some testable framework.
So the devs who actually use the RCs are probably just going to fix their code to work around the behavior change and move on.
20:36
@Girgias Yes, exactly.
Basically only fix serious issues, because then if you break something, you at least fixed something serious.
> if you break something, you at least fixed something serious
Perfect
@Girgias Some people test, others don't. Such is life.
JRL
JRL
i wish the Ds/ stuff was more widely used/supported
that's some genuinely useful stuff at times
@JRL Its latest release has 86k ish downloads in about 9 months on pecl. I'd say that's decent.
JRL
JRL
true, but it honestly should be part of core more than SPL should imo
20:47
It should, but that also has problems
@Girgias This kind of communication will always exist, not just in development, but all facets of product development, policy changes, government, etc. I hate it myself sometimes when it comes up for other projects where there is an end-user. You have to chew up the meat and spit out the bones and get to whatever the focus is of the rant and decide whether it's important. Teaching people etiquette is a much more challenging problem. =P
But I get it... it doesn't make it any less aggravating.
cmb
cmb
21:07
@Girgias Well, you always have the choice: either fix the bug, so people can complain about the BC break; or don't fix the bug, so people can complain that the bug still has not been fixed. ;)
Damned if you do or don't scenario.
I'll choose the latter, as that way I at least don't get an email sent to me by GH by someone ranting
And it's faster. =D
@Girgias Also saves a bunch of time :P
Yeah :)
JRL
JRL
21:12
not doing work to propose RFCs is also a huge time saver
:D
In other words, let's get paid doing nothing ^^
Who's getting paid?
I thought all of you internals devs were doing it out of the goodness of your hearts?
JRL
JRL
Ilu got grant money from the foundation i think?
wow, nice!
So no kindness of heart, just cold hard cash.
JRL
JRL
lol
21:17
=D
cmb
cmb
Re max_input_vars: did you know, that the limit is off by one? 1000 means that you get at most 1001 vars. See bugs.php.net/bug.php?id=60707. (maybe someone wants to fix that, causing a BC break)
3
JRL
JRL
haha
i still need to reimplement my PR that has been in limbo for months
it seemed like the consensus was that i should make an RFC for it, but also that php-src has never required an RFC for this kind of thing before that only affects internals and extensions in a non-breaking way
so i was confused, and then i forgot about it because i was trying to figure out what to do for so long, lol
@StatikStasis Yep! 6 of us since April. thephp.foundation/structure (I hope I still have a kind heart :P)
@cmb Is that still true? Dumping $_POST, $_GET, $_REQUEST, $_COOKIE with 2 query parameters results in:
Warning: PHP Request Startup: Input variables exceeded 1. To increase the limit change max_input_vars in php.ini. in Unknown on line 0
array(0) { } array(1) { ["foo"]=> string(3) "bar" } array(1) { ["foo"]=> string(3) "bar" } array(0) { }
Very much looks like 1 to me.
JRL
JRL
actually.... looking at my PR, it seems that I did reimplement it by adding a new object handler already...
im not sure there's any more work to do, other than testing to make sure nothing broke
i'll look at that this weekend
cmb
cmb
21:32
@TimWolla I just run the given test script with master, and got the reported actual result. 1 might be a special case.
JRL
JRL
actually, if anyone could review the PR within the next few days, that would be excellent. I think it's been changed quite a bit from the last time anyone looked at it: github.com/php/php-src/pull/7973/files
the current PR doesn't cause any issues for existing extensions, so i think the 'Requires RFC' tag can be removed
cmb
cmb
@JRL I think you should switch to "ready for review", and also leave a brief comment asking for review. :)
JRL
JRL
ahhhh... good point, i still have it in draft
i am a real developer, i swear
i know how github works and everything
@cmb A-ha! The off-by-one only exists for $_POST. $_GET works as expected (use method="get" to see).
cmb
cmb
Fascinating! :)
22:02
@IluTov From my perspective in this room, you do. It was just a jest. =)
22:36
Hey @JRL
JRL
JRL
hi
@bwoebi Just did and replied.
JRL
JRL
@SahilDev were you looking for me?

« first day (4351 days earlier)      last day (821 days later) »