« first day (4280 days earlier)      last day (893 days later) » 

01:57
Hi
02:26
@M.K.Dan Hey:
Jun 15 at 2:27, by Danack
'ello. For the record there are normally more people talking during the UK day, and they are normally more sober. But feel free to ask questions.
@M.K.Dan this may be disheartening, but stackoverflow and wordpress.stackexchange aren't actually good to ask questions where the answer is in the manual. Searching for how to add a class to a theme wordpress appears to give some relevant results.
(also, I don't use wordpress so I have no idea).
03:01
The search results are adding class to menu
But div id = content is not a menu
Next result is adding class to body tag
04:00
Related to my earlier question about getting CLion to recognize test sources, etc.: by default, CLion uses the all target in your Makefile. If you set up a custom target that includes all and whatever test build targets you want, then you can tell CLion to use that target, and then it provides the proper code insight. github.com/ramsey/ext-ecma_intl/blob/…
I think I'm finally starting to understand and get the hang of things. I'm very close to grokking pointers. Maybe by the time I finish v1.0.0 of this project, I'll have them. ;-)
Wes
Wes
04:18
f me
public function __constructor(){
 
3 hours later…
07:19
Maybe I should also ask on the mailing list, but is there a secret reason why some native functions do not allow double-dot range notation in their character mask? Examples: stackoverflow.com/a/72865139/2943403
07:59
@mickmackusa looking at those examples, all of the ones that don't accept it are based directly on C functions (you can tell by the naming style)
so they'll likely either be implemented directly on top of a standard C library, or written based on the specification from one
that's not a reason why they shouldn't have extra functionality; but it's a reasonable guess as to why they don't already
 
1 hour later…
Wes
Wes
09:14
i wish anon classes could be named inline to facilitate static analysis

public function bar(): MyNewClass{
    return new MyNewClass extends Foo{
        // ...
    };
}

sounds stupid to name anonymous classes, i know... but it's for edge cases.
for example specialized exceptions types are often thrown from one place only, so rather than creating a new file for just that exception, you can create it inline, and without losing the ability to use it in type declarations:

throw new InvalidPermissions() extends Exception{};
09:24
I like the new "n" pattern flag coming with regex functions in PHP8.2. I assume it gets me closure to getting rid of unwanted fullstring matches. 3v4l.org/97lST Does this mean I can directly create indexes from zero? or at least numeric keys from zero?
I've been long waiting for something like PREG_MATCH_NO_FULLSTRING
09:51
@Wes my instinct was "WTF", but actually, new class{} does kind of declare a permanent class, because two objects created from the same line have the same class: 3v4l.org/RiFvX ... so maybe putting that class into the main symbol table isn't completely crazy
I still really want to find a solution to getting lexical variables into anonymous class definitions without a bunch of constructor boilerplate, but the two things would probably combine OK
public function bar(): MyNewClass{
    $thing = $this->createThing();
    return new MyNewClass extends Foo{
        public readonly $thing = lexical $thing;
    };
}
it only wouldn't work if we wanted lexical vars to go into the definition, requiring a different class each time, e.g.:
public function bar(): MyNewClass{
    $thing = $this->createThing();
    return new MyNewClass extends Foo{
        public function setThing($arg = lexical $thing) { ... }
    };
}
but that's probably a sensible restriction anyway
10:40
@Danack 1.1.1 is released now: openssl.org/news/openssl-1.1.1-notes.html: "Fixed AES OCB failure to encrypt some bytes on 32-bit x86 platforms".
Wes
Wes
11:19
@IMSoP ah yeah that'd be nice!
11:50
guys, i just upgraded the phpmyadmin from 4.7.4 to 4.9.10, everything works fine in the website towards the database, but im unable to access the webui localhost/phpmyadmin
Server Error in '/' Application.
Access is denied.
any idea ? :)
12:04
o/
\o
 
1 hour later…
13:18
o/
13:33
@Sara @GabrielCaruso @ramsey @PatrickAllaert I know I'm late but this only came up on Sunday that a bug fix I merged into SPL cause some drastic changes which have been present since PHP 5.
The commit in question is https://github.com/php/php-src/commit/6f87a5c6338d0d64d18e3a44b65c19acc9c529e8
Should this be kept or reverted (and sorry if therefore it needs retagging)
Mhhhh, I don't really know.
How "drastic" is it?
cmb
cmb
@Girgias This shouldn't matter wrt. the GA release which are tagged today, since those ignore the latest changes to the main branches, but instead use the release branches with added security fixes only.
oh, well, it would affect PHP 8.2
@cmb that commit is in the RC
> I have code that uses SplFileObject to iterate over the lines of a CSV file. That code has been working fine for years, from PHP 5.x to PHP 8.1. Now with current master it suddenly returns false instead of an empty array at the end of the file. Is that intentional?
Also seem to have broken iteration in a foreach loop if looking at: github.com/php/php-src/pull/8915
cmb
cmb
Ah, I see. Maybe reverting for now is the best solution. But RMs should decide.
Yeah, been told that by Derick
13:49
@Girgias 8.0.21 is aleady tagged by @GabrielCaruso, so If you choose to revert, this have to be done quickly
I know, I thought tagging was next week :|
also, some people may already use it (at least from windows.php.net/download/#php-8.0)
i know i know
but me breaking the git yesterday just made me not want to look at PHP for the day
cmb
cmb
I can remove it from there, but AFAIK some also use the tags directly.
having to fix test expectations in a commit should have raised some alarms...
13:51
@RemiCollet It's a bug fix, stuff is going to need to change because of it
And the behaviour it has now is the correct behaviour
It's just that SPL has been buggy for so long that people might rely on this and I didn't think about it because the bug it's fixing is not talking about this exactly
cmb
cmb
Anyhow, somebody should have filed an issue regarding this. This is why we ship RCs.
14:10
@RemiCollet That is their problem, using releases from before they were announced.
2
cmb
cmb
@Derick Maybe run a php -v on your machine and check the date. It's not unlikely that the version has been build between tag and announce.
sorry?
cmb
cmb
I've seen Debian PHP builds which have been build between tag and announce in the past.
Right, they shouldn't do that either.
but I build my own... and I'm definitely always running -dev builds :-D
14:25
@cmb I think most Linux distro use SOURCE_DATE_EPOCH for reproducible build
ex, we use export SOURCE_DATE_EPOCH=$(date +%s -r NEWS)
so whatever the build is run, displayed date will be NEWS date (not build date)
cmb
cmb
Ah, okay, then maybe that was not a real issue. :)
15:00
So... I'm officially addicted to paintball.
Oh man I loved playing paintball back in the day. I haven't played in years though
16:08
@Girgias @Derick @cmb So, what are we doing regarding the tags today? I think @SergeyPanteleev and @Pierrick are tagging 8.2 right now, and I was about to do the tags for 8.1.
cmb
cmb
Well, the RMs need to decide whether to revert the respective commit. However, if PHP 8.0.21 won't be retagged, I don't think it makes sense to revert the commit in the newer versions. cc @GabrielCaruso
Well 8.2 is fine as that change needs to land there anyway, but the question is, should it be a "traditionnal" bug fix even if that means changing long standing behaviour for the proper behaviour or just leave it in master
what's the impact if we don't accept the change?
i.e., if we revert it, what's the impact for users?
is the commit in the PHP-8.1.8 release branch, or is it in the PHP-8.1 development branch only?
cmb
cmb
@ramsey It's about fixing github.com/php/php-src/issues/8563. The commit is already in the release branches (was shipped with RCs).
gotcha
So, the question is whether to revert it for 8.1 and 8.0?
16:15
Yes :)
I should have brought this up yesterday as I've been told this on Sunday, but the whole me fucking up git made me want to not look at php-src for the day and I thought taggin was next week
Hmm... seems like it's definitely a bug that should be fixed in 8.0 and 8.1, IMO, and since @GabrielCaruso has already tagged 8.0, I don't think we need to do anything
the "RETURN_THROWS() assertion failed" only affects DEBUG build...
btw, it raise an inconsistency about 'failure' in which case an exception is expected
also should it return NULL or false (for consistency with other stream, ex fopen, fgets...) ?
so, obviously, the fix is NOT ready, so better to revert it, and work on another fix which will go in next version, so in next RC
(anw whatever it returns, doc is wrong... as doc states it returns a string)
php.net/manual/en/splfileobject.fgets.php public SplFileObject::fgets(): string
php.net/manual/en/function.fgets.php fgets(resource $stream, ?int $length = null): string|false
IMHO: revert everywhere and retag 8.0.22 and 8.2.0alpha3
16:31
@RemiCollet It returned NULL in 8.0 but that was changed to return false consistently in 8.1 (by Nikita)
To be clear, I don't mind reverting
@ramsey FWIW, I don't agree. This is a subtle behavioural change that might end up breaking people's code - like it did for Sebastian
But SPL is a nightmare and the test coverage is piss poor
cmb
cmb
If @GabrielCaruso is unavailable, maybe @Sara can retag php-8.0.21?
But yeah, I don't want to go and revert stuff without having RM approval
@Girgias You can't revert things in release branches yourself, really (or rather, you really shouldn't touch these)
16:33
Right, that also lol
cmb
cmb
Yep, release branches are for RMs only. :)
perhaps should raise this on RM mailing list ?
(don't know who can mail to the list, only RMs or everybody ?)
cmb
cmb
I think everybody can mail, if the now the secret email address. ;)
I guess we better reach Sara here than on the RM list, and Gabriel likely called it day already.
16:53
sorry, have to go, will check tomorrow what happened.
So, if we want to revert it, it needs to be reverted in PHP-8.0, merged to PHP-8.1, then merged to master, and then the revert needs to be reverted in master. At that point, I can cherry-pick the revert from PHP-8.1 into PHP-8.1.8.
yes
@CharlesSprayberry I've avoided it for a long time because I knew it would suck me in... aaaand it has.
although you don't really need to do " then merged to master, and then the revert needs to be reverted in master", you can just merge with --strategy=ours once instead
17:04
i need to run too though
It sounds like it would be less harmful overall to revert it, so I just need to hear from @GabrielCaruso or @Sara that they are okay with retagging php-8.0.21
I'll send an email to RMs mailing list, @Girgias, since that might be the best way to reach everyone right now
Sorry, just got the email - give me a sec to read everything
I didn't send it yet, so not sure what email you just got. :-)
Actually, I just sent it, but after you posted "just got the email" :-)
From Remi, to RMs
Hmm... I didn't get it
weird
17:17
Alright, I'm fine with the revert - I'm finishing dinner, I'll re-tag
did you get my message, though?
cmb
cmb
@ramsey We need to revert; the assertion error is actually a guard to prevent a segfault, on code which previously worked fine.
@Girgias or better, can you let us know once you revert?
And thanks for bringing the attention to it
I don't understand why I didn't get Remi's email
cmb
cmb
spam folder ?
17:19
nope... checked
Oh... I see. It was in reply to "PHP 8.2.0alpha3 build"
I was expecting a new thread
@cmb Thanks. I didn't see the segfault comment
@ramsey sorry, I should've said that :p
cmb
cmb
@ramsey That comment was nonsense. Sorry.
17:45
@Girgias It sounds like @GabrielCaruso is on-board with re-tagging php-8.0.21. Can you revert in PHP-8.0 and merge up to PHP-8.1 and master (using --strategy=ours when merging to master, as @Derick suggested)? Let us know when you've be able to do that, please.
@cmb No worries
@ramsey I don't know if @SergeyPanteleev is still there but I can retag and rebuild 8.2alpha3 sometime today.
@Pierrick Retagging 8.2.0alpha3 isn't necessary. @cmb said there is no segfault
Oh ok ! Then I'll do nothing ! I'm good at it !
2
So, the bugfix should be in master and 8.2.0alpha3. When the revert is merged up into master, it should be done in a way that the commit hash ends up in master but it doesn't actually revert it, so just double-check the merge commit, when that happens
cmb
cmb
Actually, I think it's good to ship that with 8.2.0alpha3. That might trigger further bug reports which may help to resolve the issue (at least increase test coverage).
18:25
@GabrielCaruso Ditto okay with revert. Better a minor bug for a little longer than a big wtf.
@GabrielCaruso In unrelated news, would you mind taking 8.0.22 (I'll do 23 and 24 in exchange). Turns out I have some scheduling conflicts.
@Sara Don't mind at all - I'll handle it
@GabrielCaruso <3
We're nearly out from under regular releases. Soon it'll be security fixes only and sailing off into the sunset.
18:51
@Sara It's been a journey - thanks for all the guidance
Alright, I'll handle this tomorrow morning first things, I need to shut down for a bit
@Wes yeah, that should produce a big flashing syntax error in PHPStorm:
19:04
@TimWolla "32-bit x86". Ok then.
@Danack But also "failure to encrypt some bytes". I dunno about you, but I want my encryption function to encrypt.
how many 32bit x86 machines do you run?
@Danack "Php Inspections (EA Extended)" has you covered :)
I've never figured out what the "EA" stands for, but that plugin picks up a lot of useful stuff
me neither.
19:26
it is indeed a mystery
I thought it was a recursive thing, like Extended Analysis , but it's entirely possible I made that up
it's ok, that's basically how humans create meaning anyway :)
Wes
Wes
19:42
@Danack only a slight gray zig-zag underline for me
completely missed it
it complains about void not being specified :P
@Danack Zero, but can I guarantee none of the SSL connections I have open are to 32bit x86 machines?
It looks like "EA" might stand for "Enterprise Applications." Source: reddit.com/r/PHP/comments/7ihobm/comment/dqzk2yn/… Cc// @IMSoP @Danack @MarkR
20:21
@Girgias how are things? Need any help with that revert commit?
@ramsey I wasn't at home this evening just got back
I can revert now
So where do I do it? on the PHP 8.0 branch?
Or on the 8.0.22?
@ramsey Ah ok that's the process let me do that
20:46
@GabrielCaruso @ramsey should be pushed now
and I hope for the love of god that I didn't mess something up
21:01
@Girgias looks good on PHP-8.0 and PHP-8.1. Can you confirm that when you merged into master, you made sure that you used --strategy=ours (or something like it)? github.com/php/php-src/commit/…
that commit makes it look like it's also reverted in master
@ramsey git didn't like the file renaming, so manually
@ramsey yeah it is reverted also on master, as apparently there is a segfault
oh, I see
@Girgias @cmb clarified that he was wrong
But I can re-do the change on master
Oh, well let me redo it and do something in UPGRADING
wrong about the segfault, that is
:-)
I gathered that
21:04
At any rate, looks like PHP-8.0 and PHP-8.1 branches are good. Thanks! @GabrielCaruso, you can re-tag
(after cherry-picking that, of course :-) )
@Wes here's the PhpStorm plugin that gave me that extra inspection; there are a few times I've had to silence it, but mostly I find its suggestions and fixes helpful: plugins.jetbrains.com/plugin/7622-php-inspections-ea-extended-
22:00
@StatikStasis Nice find
22:45
Voting for the trait constant RFC is now open.
And since this is a proposal that hasn't had much discussions in the ML, I suspect a typical outcome for such proposals that critical issues will be pointed out during the voting phase.
"Always has been."
Man, Ocramius really hates everything, doesn't he. :-)
JRL
JRL
@Crell what thread is that from?
23:01
Re "And since this is a proposal that hasn't had much discussions in the ML, I suspect a typical outcome for such proposals that critical issues will be pointed out during the voting phase."
Because the whole world is deadline-driven :)
@Sara OCB isn't used in TLS. From the advisory at: openssl.org/news/vulnerabilities.html#CVE-2022-2097 "Since OpenSSL does not support OCB based cipher suites for TLS and DTLS, they are both unaffected."
JRL
JRL
oh, lol. @Crell i thought your two messages were about the same thing. I was trying to see where ocramius had used a similar phrase on the ML
Ha. No, separate. The second comment was an observation that he'd already voted no on trait constants, as the only vote.
If it's about traits, then ocramius won't have a very good impression even for that alone.
JRL
JRL
23:05
tbh, i think his opinion is kind of useless. not because he doesn't have technical contributions to make in discussions and code, but because he seems (by his own arguments) to come from the position of "if you want to do [thing], use a language that's better suited for it than PHP".
which is a really.... counter-productive position to take when you are voting/helping control the direction of the language
it makes it hard to take his contributions and suggestions into the design, because you know it won't affect the outcome at all
which reduces his actual contribution to the language and project as a whole
which is not a thing that is limited to him, i think
But he often posts his reasons for voting in the ML, which I appreciate, so I hope it will be the same this time. :)
23:25
I have a feeling he's been having a bad day. twitter.com/Ocramius/status/1544369270456565762
Followed by Jordi having an equally bad day. twitter.com/seldaek/status/1544408358970613760
Wishing everyone a good day.

« first day (4280 days earlier)      last day (893 days later) »