« first day (3510 days earlier)      last day (1424 days later) » 
00:00 - 23:0023:00 - 00:00

12:00 AM
I'm trying to build an example, and wanted to show method chaining for funsies, it gives an error ... this works (no method chaining), this doesn't (method chaining). What am I doing wrong, or is what I'm intending not possible?
 
@Tiffany you can only chain method if the method returns the object
And a fluid pattern is usually best used to "configure" the object instance
A typical example is a SQL query object
 
@Girgias so it would need a return $this->blahblahblah; or something like that?
 
return $this;
 
darn
 
Basically if you can declare the return type as self, then you've got a fluid method
But the design pattern can be pretty abused and not clear
On this note I'm off to bed, I need to wake up in 6h to prepare for a project meeting...
 
12:08 AM
o_O
good luck
 
12:31 AM
question regarding docs XML syntax, I'm using doc-en>language>functions.xml as an example to build a table. Does the table need to be wrapped in a <sect#> tag? (where # is a number)
 
 
2 hours later…
2:03 AM
readdir skips entries ・ Directory function related ・ #79633
 
2:44 AM
Hi all, I have a `git cherry-pick` question as it relates to my PHP codebase. I realize this room is directly for PHP questions, but would a git question be allowed?

I'm happy to be redirected but yeah... couldn't find a git room.
 
> Don't ask to ask, just ask.
 
Lets say 1 company is using AutoCad system and they have 2 licenses
they are 20 systems in the company
Is it possible to share 2 licenses on all the 20 systems?
 
(from what I understand, off-topic questions are okay, as long as it isn't disruptive, and if an RO asks you to stop, then stop)
@ILoveStackoverflow possibly, but I doubt it's legal
 
but question is how?
 
2:49 AM
How...what?
 
I mean how is it possible to share licenses across 20 systems?
 
How to use two licenses across 20 systems?
 
because only 2 system would be able to use 2 licenses right?
 
It depends on how the software checks licenses
It may allow reuse, but doesn't mean it's legal
 
Is it possible to use any cloud technology that would allow to run this software from 20 machines with just 1 or 2 licenses?
 
2:51 AM
If the software does any kind of audit check, the company is opening themselves up for a lawsuit...
 
I am just trying to find potential solutions
 
That's a question for AutoCAD...
 
I am working on case study where it says the company doesn't want to spend licensing for all 20 systems
So I have to figure out solution for them
 
I'm not giving you advice on how to do something illegal
 
Actually its just a case study
This is just in practice not an actual implementation
 
2:55 AM
Hi All,

My (git) question is that I know how to use `git cherry-pick` for when I branched off of the wrong branch, but this current one I'm doing has very, very complex diffs along the way (conflicts with each commit). I basically would like to simply use the current code checked in for my files during cherry-pick but have the very last commit be where I apply my changes in by hand.

I'm sure this isn't a good practice, but is it reasonable for my situation and will it mess with the commits at all?
 
lol
I have been cooking lately.
 
@MatthewVita most of the more experienced members live in EU, so you may not get an answer tonight, but possibly something early in the morning, if they're feeling up to it...
 
@Tiffany oh thanks for that - I appreciate it!

Got myself in a pickle xD
 
EU/UK I should say
 
 
1 hour later…
4:15 AM
morning all
Looking at the current status of PHP namespace in core vote it looks like there's quite much of people who would like to see PHP namespace in action and the same against it.
I think we're not gonna get to any consensus within next years in this topic.
But well it goes the other way I imagine should evolve all the time.
I know that it is a completely different topic. We're adding more and more things to standard library which is bundled and even if someone doesn't want it it's not possible to build just PHP interpreter, mostly naked cause there's a lot of dependencies between exts.
Last week I was wondering if it is possible to build small PHP interpreter like Python has the microPython which can fit into small microcontrollers, but it looks like SPL, Date, Session etc. are coupled with the Standard - I'd love to loose those dependencies and have a naked interpreter able to interpret the language and provide a replacement for Standard build around vendor SDK like for eg. the SDK for ESP microchips.
And why such a crazy idea? I was thinking of building an ESP image with PHP interpreter bundled so I can for instance create a separate PHP file and in few lines implement what I want the ESP to do, nothing more than just couple of lines. The SDK already ships with a http and lot of functions which could be used to build an standard replacement.
 
 
2 hours later…
6:46 AM
Could not open password protected zip archive ・ zip ・ #79634
 
@MatthewVita git tries very hard to give you a useful history - which is why you shouldn't just try to stuff the current state in the repo. What you might be looking for is to rebase your branch on top of the changes done in another branch
 
cmb
7:16 AM
@Tiffany no, not necessarily; see tdg.docbook.org/tdg/4.5/informaltable.html#d0e98899 for details
 
7:52 AM
@brzuchal You missed (I did too but not that much) a major discussion which happened here, and @Crell made a Google doc summing up some of the opinions: docs.google.com/document/d/…
 
@Girgias Well it didn't help to convince others who voted no :(
 
I mean sure, but at least we are having a discussion where multiple people try to find a common ground, much better than everyone doing stuff on their own and trying to convince them
We didn't get scalar types in one night, might be the same for a namespace usage
 
Yes, I agree
More people involved works always better
 
Not in meetings...
 
8:12 AM
Hey there, I'm stuck with Symfony and easyadmin. Maybe someone can push me in the right direction and solve my (probably easy to do) riddle. I have an entity for User and another entity for UserAccounts. I try now to Add/Remove and Edit UserAccounts right in the User Form in the easyadmin backend. For that I do believe I need a new UserAccountType where I build the Form for the UserAccounts inside the User Form. But I just don't really understood where I went wrong. pastebin.com/ThmvxrwT
Within the UserAccountType the entry_type does make no sense, also the reason I guess why I just get stuck right now.
 
Good morning all, I just joined this chat and I beginner to php programming.
 
8:48 AM
@NikiC wait, how can you even commit into my repository? i just saw the commit
 
cmb
"Allow edits by maintainers" checkbox
 
@MatthewVita any offtopic stuff is allowed, so long as it isn't too offensive and isn't disrupting more ontopic discussions.
 
Hi all!
I have a question regarding PHP and MySQL. How can I update a sale page with multiple products? if there would be a single product then I just need to fire an update query to update it but I don't have any idea how to update multiple products
 
9:03 AM
@Danack thanks
 
@cmb interesting, i thought this only relates to pull request description :D
 
@Tiffany sorry, will reply to your messages soon.....still feeling shite.
 
@MuhammadSameer are you familiar with SQL?
 
@Tiffany Yeah but I am still learning
 
@Danack it's cool, it's more important that you feel better
 
9:06 AM
@Tiffany Can I post my code snippet here?
 
@MuhammadSameer I just woke up, but I believe you can add commas in the update part... maybe googke for something like "how to update multiple items SQL PHP" or something
If you're getting an error, you should've added that in your original message :P
@MuhammadSameer preferably post it in something like gist.github.com or pastebin.com
 
@Tiffany No problem, I've tried googling but didn't get the answer required answer
 
Does anyone know of a stack overflow chat group for Google Cloud Firestore questions?
 
@beberlei I hacked github, of course
 
9:15 AM
@NikiC i knew it ;)
 
@MuhammadSameer read about binding parameters and using prepared statements. As it stands currently, your code is vulnerable to SQL injection
(I doubt you want some bored kid dropping your database)
 
@NikiC i'll talk to martin about the other todos, specially the self:: seems wonky after testing a little yesterday. do you think the ZEND_ACC_ATTRIBUTE is useful before merge to master? otherwise i get back to you once all comments are adressed, so that we don't take too much of your time
 
@MuhammadSameer you can try googling for something like "using prepared statements example mysqli" or some other variation. There's a lot of stuff out there. I can try to help more when I'm actually caffeinated.
... and not tired
 
@Tiffany Thank you! have a good day. I am going to google you recommended question now
 
@beberlei nah, not necessary
If you're not particularly concerned about performance there
 
9:24 AM
@MuhammadSameer if you get an error, I'm sure you know to google the error, but if you still don't understand how to fix it, ask a question here with what you've tried, the error you receive, and explain what you don't understand with the error after having tried looking it up yourself
... Now I'm going back to sleep for a bit ...
 
@Tiffany Thank you!
 
9:55 AM
@brzuchal a key point is that the votes against the current RFC aren't necessarily against using the namespace in general, they're just against adopting the particular policy wording of this RFC; what we need to find some consensus on is a policy of when and how to use it
which as I said on here the other night is fundamentally writing a new set of naming conventions, because a namespace is, ultimately, just a way of naming things
 
Anyone happen to know Arkadiusz Kondas irl?
 
@IMSoP I tried to write it as much abstracted I could, given examples were only to show potential places and the test was about allowing to use only in future RFC's. Maybe you're right but I wish we could find a consensus and make a final statement in near future, cause otherway it's gonna come back like a bumerang in next RFC's session for next minor release
@Danack I heared of him, somewhere on conferences I guess, I might even have him on twitter
 
yeah, keeping it abstract may seem like it's avoiding the bikeshedding, but really it's just putting it off until later
 
context - he just copied some code from another ML repo. He is just digging himself in deeper and deeper trying to explain why what he did wasn't wrong. All the other person wants is to have his name attached to his copyrighted files...
 
Oh is that the PHP-ML drama thing?
 
10:01 AM
I think it is..
Well, after checking my tw, no I don't know him, only from hear. I didn't meet him, I was wrong
:D
 
if all you agree is that the bikeshed definitely needs some paint, people still can't go and buy the paint without settling the argument about what colour it should be
 
@Girgias yeah. I still need to find a better way to something. Saying to someone "you are in a different reality from the one that everyone else exists in" either doesn't get understood, or gets ignored.
aka the guy appears to be a bit nuts, and it would be better for someone who knows him to have a chat.
 
similarly, if all we agree is that using the PHP namespace is a good idea, people still can't write an RFC using it without settling questions of what the classes inside it should actually be called; so either they'll avoid mentioning it (status quo), or it will de-rail an unrelated RFC
 
@Danack I do know only his employer CEO - he's my colleague from studies
@IMSoP yes, my intention was to cut off bikeshed when someone discuss about it in subject of a RFC
 
"if all we agree is that using the PHP namespace is a good idea," - nope. If we say "the root namespace is PHPs', every project needs to use their project name as a root namespace for all their stuff." that avoids all the discussion and avoids a problem.
 
10:07 AM
.
 
yes, well that's why I don't agree with them.
 
right, so you're agreeing with me: we need a policy
 
"the root namespace is PHPs', every project needs to use their project name as a root namespace for all their stuff." - I'm saying that's the policy we already have in practice. Maybe it could be said more clearly.
 
that's not enough, though
"we own it" doesn't answer "what do we do with it?"
 
@brzuchal to be clear, that wouldn't be the right person....it needs to be someone they go drinking with....
 
10:09 AM
oh, sorry, I see what you're saying
you're saying abandon all claims on the PHP namespace
that's certainly an alternative
 
...well it shouldn't be used for projects. But there could eventually be userland code shipped by the PHP core team, that could use PHP.
but for PHP core itself, I just can't actually see a problem with what has been done with namespaces so far.
 
well, one problem is that it forces everything in core to be flat
we can't use sub-namespaces, because there's nothing for them to be sub- of
 
"that it forces everything in core to be flat" - while that's true, you're assuming that I agree that it's a problem.
 
I'm not assuming anything
 
sorry -
 
10:13 AM
@Danack I've found an article where he describes what happened and that parts of his code got into RubixML arkadiuszkondas.com/dmca-php-ml-and-copyright-boundaries
 
I'm presenting a possible problem that I see
 
I mean, just by saying that, doesn't actually say a problem that needs fixing to me. why is " forces everything in core to be flat" a problem?
 
why do namespace need to exist at all?
or rather, why are namespaces hierarchical at all?
clearly, designers of namespacing schemes in many languages saw that as a necessary or at least useful feature
 
@brzuchal yeah......I'll probably keep my drama on twitter, but that blog post is full of lies and contradictions.
@IMSoP yes. Java and C++ in particular. And those are great languages that we want to copy. /s
 
and Perl
and .net
and people seem quite happy using them, rather than having all their files in one directory
 
10:17 AM
@GabrielCaruso When do you want to start the vote about your magic method RFC? :)
 
@IMSoP I'm going to go do some work.....but I've said this a couple of times in a couple of different conversations. Explaining why something is a problem first is a good step in getting people to agree to 'fix' it. Just saying that "that it forces everything in core to be flat" doesn't explain why that's a problem. And comparing something to other languages, also doesn't explain why something is a problem.
There is a much higher chance that you mistaking an aesthetic choice you personally prefer, for something that is actually a problem.

I did a talk that covers parts of that before: https://www.youtube.com/watch?v=TS9BNa1MDW0
Apologies for the quality of my voice. Fun-fact, it turns out I have a problem with my nervous system which was intefering with my ability to speak that day.
 
oh woah, that is a coincidence, but i had a rather unplesant experience on reddit with ripping something else off with them
 
@Danack Well I don't know specifics but from the source code examples it's obvious who was first with some code who was next
 
@beberlei same person? links to drama please.
@brzuchal I'm not going to go into the details here, as it is too offtopic, that blog post contains many half-truths and some outright lies. I do my drama on twitter.
 
OK
 
10:26 AM
@Danack I almost agree, but I'm not sure "problem" is always the right frame; it can also be about _benefit_
and my initial aim wasn't to compare to other languages but to PHP itself - we already have hierarchical namespaces; for that matter, we have _namespaces_, plural, and currently everything in core must ignore that fact, and use PrefixConventions
 
@Danack are you ok with me not linking it? :D i don't want to fuel this
 
Did I hear something about drama?
 
@beberlei sure.....though if the guy gets too abusive, I might consider you my strategic weapons reserve.
@NikiC not going to link it, but look at twitter.
26 mins ago, by Danack
aka the guy appears to be a bit nuts, and it would be better for someone who knows him to have a chat.
 
@Danack why are you so invested? :D
 
@NikiC Didn't know you were such a drama stalker ;-)
 
10:30 AM
@Danack ah i didnt see it on your twitter, because you replied to someone i dont follow
 
@NikiC The yield from yield from bug is fixed, right? I'm about to roll 7.4.7RC1
 
okay, at the risk of revealing info about myself, some of my family show the same symptoms as this guy, of just telling stories that make themselves look good, but that actually twist reality a little bit. Due to too many years of that, I'm good at recognising that behaviour and calling it out.
Also, (and it might surprise some people here for me to say this) but I do restrain how I speak to other people in this room, so it's nice to have a 'clear villain' to go after.
Also, it appears the guy he copied the code from has not enjoyed the experience, so there is a 'kinship of the maintainers' thing going on.
 
@Derick yes
 
Excellent. I thought so, but thought it wise to confirm
 
Howdy! Trying to compile PHP 8 on MacOS, and the first thing it stumbles over is the ancient bison version shipped with the MacOS developer tools. Homebrew carries a newer version, but doesn't allow linking it to not conflict with the system default. Is there a configure flag I can use to tell make which bison to use? Don't know if AC_CHECK_PROG supports something like that
 
10:37 AM
Just make sure it comes first in the path.
 
I did that to work around the problem, but that doesn't work when trying to create a formula for homebrew
 
— Macro: AC_CHECK_PROG (variable, prog-to-check-for, value-if-found, [value-if-not-found], [path = ‘$PATH’], [reject])
so it looks like it can do that
but that'll also mean you'll have to add a new configure flag
 
Yep, I'll have to do that regardless of my path hack, as it doesn't work when bison 3 comes first in $PATH
The strange thing is that it works great when configuring directly from source but fails when doing it through homebrew. Ah the joy of package "managers"
 
@alcaeus I can only say "have fun"
 
@NikiC getting paid to do this makes it less bad...
 
?? ??? ????, PHP 7.4.6
how did that end up in NEWS?
 
@IMSoP the key for me, is that if you identify the problem first, quite often someone will come up with a different solution than the one you like. Sometimes this is annoying, as it means my wonderful idea gets thrown away. But most of the time, it at least avoids the problem of people speaking past each other, which appears to be happening for this discussion imo.
 
@NikiC Do you have an idea as to where I need to look into if (it looks like) the parser is getting confused with list()?
 
Morning
 
@NikiC Thanks! Homebrew seems to mess with the PATH, so I'll have to try and work around that.
 
10:49 AM
@alcaeus Alternative is probably to do something like ./configure BISON=xxx. I think that also works
 
@NikiC And include the full path to the binary?
 
@alcaeus yes
 
@NikiC I'll give that a shot, thanks for the suggestion!
 
cmb
@Derick which branch?
 
@Danack yes and no; people also disagree on the relative merits of different solutions - given the problem statement "we need to assign names to core classes that are clear and unlikely to conflict with each other and userland classes", some people consider current prefix- and suffix-based class names a sufficient solution, some see namespaces as a better solution
 
10:53 AM
php-7.4
 
@cmb the sect tag that surrounds the table in functions.xml, is it there so that the value types table can be linked to easily?
 
@cmb You did it for 7.4.5 and I did it for 7.4.6 - both fixed
 
cmb
ah yes; not super important in the PHP 7.4 branch, though, but still good to have that fixed :)
 
it makes NEWS in the cut branches wrong too
I spotted it because the NEWS in 7.4.7RC1 had "May 28th" 3 times
 
@NikiC For the record, ./configure BISON=/usr/local/opt/bison/bin ... does not work. I'll just prepend the path in the formula
 
10:59 AM
@alcaeus That's missing one more /bison I think
In this case it would need to be the path to the binary, not to the directory with the binary
 
What do you guys think should happen on $foo?->bar->baz = 'baz';? C# gives you a compilation error. Swift just ignores the whole assignment when foo is nil. We could skip also the whole assignment and make it evaluate to null (and probably the same for ++, +=, etc).
It always seems so simple at first, until you discover all the edge cases 😆
 
@NikiC You are correct, but that doesn't work as well. I've now got the brew formula to prepend the path to bison, which makes configure pass successfully
 
cmb
@Tiffany that section also contains the warning below the table
 
@IluTov I'd vote for skipping the assignment and evaluate to null. The user asks to conditionally assign if $foo is not null
@IluTov But then my opinion doesn't count much here
 
@alcaeus I'm leaning towards this side too +1
But I haven't completely thought it through. I might discover more edge cases.
 
11:08 AM
@IluTov We already have precendent what happens when the input is altered (assignment to typed properties) - we return the new value of the assignee. In this case the assignee is null. So the returned value should be null as well.
Same with assign-ops ($a = 1; var_dump($a += 2); returns int(3).) - always the new value of the LHS is returned.
 
For the record, got it to work on MacOS. Thanks for the help y'all!
 
@IluTov related thought, as doing $foo?[0] is an actual parser ambiguity, what about $foo[? 0]? (i.e. $foo[? expr] in general?)
 
@bwoebi But isn't $foo?->bar = 'baz'; somewhat different as the lhs might be null? The question is, should we error here or should we skip the assignment. Both are valid. It's entirely possible I completely missed your point.
 
@IluTov well, we return what the LHS evaluates to after the assignment. In this case, we skip assignment. The LHS still evaluates to null. So we return null.
 
one reason to make it consistent with method calls (i.e. skip not error) is maintaining the symmetry of __set and future property accessors
as in $foo?->__set('bar', 'baz') would work directly, so $foo?->bar = 'baz' should probably have the same effect
 
11:20 AM
@IluTov And I see no reason to error here, the point of ?-> is to not error, so it shouldn't.
 
ditto for longer chains: either $foo?->bar->setBaz(42) and $foo?->bar->baz = 42 both error or neither do
 
@bwoebi I think so, although it looks a little odd to me. We could make ?[ a symbol (which is a BC break of course). Annoyingly ?[ is often used in regexes which makes it hard to search. grep.app/search?q=%3F%5B&filter[lang][0]=PHP But I wouldn't think it's very common to not use a space right there.
 
if you treat it as ($foo?->bar)->... then the second -> should be ?-> in both cases
 
@IluTov you cannot due to regular ternaries returning arrays.
You probably won't find so much public code out there matching that as libraries on github tend to have a proper code style imposing whitespace.
but making ?[ a symbol is pretty much a no-go.
 
@bwoebi Ok :/
@IMSoP Yeah that makes sense.
 
12:25 PM
a blast from the past - in a time where pcre was still an extension
 
12:41 PM
Heads up I managed to get 30 licences for the PVS-Studio static analyser for the PHP project :)
 
@Tiffany I've tried searching google but still confused I've posted a question on StackOverflow and here is the link to that question: stackoverflow.com/questions/62020937/…
 
@MuhammadSameer your question currently reads like "will someone write my code". It is better to post a few examples of UPDATE queries you actually tried to run
 
@Sjon Sure thing I'll change it
@Sjon Done
 
1:01 PM
@MuhammadSameer so your question is how you would update the things you insert into product_inventory ? Don't they have a primary key?
 
They do but What if a user added more products in the update form?
 
you mean - what if an order is inserted completely but the customer changes the quantity of one of the products in his order?
 
@Sjon Yeah and also what if he adds more products that's why I deleted the old products and added again
 
@MuhammadSameer you could use UPDATE product_inventory SET qty_out=2 WHERE invoice_id = ? AND product_id = ? ? Does that not work?
 
@Sjon This works until the user doesn't mess with the number of products
 
1:13 PM
@MuhammadSameer what do you mean? Your whole code only works if the user doesn't mess with the input
 
e.g. if a user has three products each product can have the same or different quantity and then he want to add another product in his invoice. How do I update it?
 
@Derick the first words on their twitter banner are "Blockchain Development"; that tells me everything I need to know :P
 
:)
 
@bwoebi Is it? I think it's okay
 
@MátéKocsis Have you seen it externals.io/message/110276? :)
Time to share your work
 
1:22 PM
It could break a niche case in a niche style, but it's going to break it hard (parse error) so no problem
 
@MátéKocsis Hopefully today. I need to catch up with my OSS work
 
@IluTov I agree that the assignment should just be skipped
 
@NikiC really var_dump($foo?[0, 1]:[2, 3]); - making that syntax error because ?[ is a token?
 
@bwoebi sure
 
wtf
 
1:26 PM
@bwoebi You might not have noticed, but whitespace sensitive syntax is super common when it comes to such things
Like, you could say that "WTF is ++1 an error, it could be interpreted as + +1!"
 
@NikiC because that code makes no sense to write it as such
then it's sort of okay
I mean, why would you ever write numbers that way
but a ternary returning array is a totally common expression
 
Well, why would you ever write ?[]:[] rather than ? [] : []?
 
why not? it's an inline operator, those don't normally requires spaces around
 
@NikiC often - normally before formatting my code for the push then
i.e. while testing locally
also for quick testing on cmd line etc.
 
You'll have to add a space then...
Or just not have null-safe array fetch, your call :)
 
1:31 PM
@GabrielCaruso Yes, I saw that. TBH I don't really like when people are day-dreaming on the mailing list. :/ I mean, this is not that difficult to implement even for a beginner like me, so then they could just try to come up with a proposal/partial fix on Github if they really wanted this change.
 
@NikiC I prefer not having it with that syntax (or not at all if there's no alternative) over that restriction.
 
fair
 
I wish the language to not be whitespace sensitive except for extreme edge cases.
 
$?foo
$foo->?bar
$?foo->?bar
 
Question to folks who have can generate API token for the php-src repo, can i get one so that I can setup a GitLab CI pipeline?
(Probably need to go through GitHub to get the status but not totally sure about that)
 
1:33 PM
a read token?
 
Yeah, well they say "repo" permissions
 
@Girgias Let's maybe not add a fourth CI provider?
 
@Girgias why a gitlab ci pipeline? instead of travis / what we already have?
 
It looks like the appveyor webhook is broken actually
 
@NikiC why not? Or do you prefer GitHub actions? I'm trying to setup a static analyser build using PVS-Studio, probably should be scheduled tho
 
1:35 PM
It says "Last delivery was not succesful. Invalid HTTP Response: 404"
 
@bwoebi Because Travis and co are already testing various things and being congested from what I saw
 
a quick search of the code base in front of me shows a lot of ?1:0 and a few longer examples with no spaces around the ternary punctuation like $startdate = ($historic===true)?2002:date("Y") and $_REQUEST['isbookingongo']?'':'disabled'
so ?[...]:[...] doesn't sound particularly unlikely to me
 
@Girgias Can you put it on Azure?
 
@NikiC could try yeah
 
@MuhammadSameer you can update or insert another row in product_inventory (which totally has the wrong name btw)?
 
1:40 PM
I'm still trying to set it up locally as my setup is whack, but I'm also thinking how to generate a report
 
@Girgias Does it generate anything useful?
I've found static analyzers to be waaay to noisy for php-src
 
They made 2 articles, one for PHP 5.6 and another one for PHP 7.0 when it came out and it did catch some legit bugs viva64.com/en/b/0392
 
okay
well, doesn't hurt to try it
 
Will try on Azure first but I'm not super familiar with the configuration so may run into issues
 
God thank you, it was driving me crazy
 
2:19 PM
@cmb Do you think it's worth to wait with the unbundling of ext/xml-rpc until some of my related changes are merged? These include a few default value fixes (UNKNOWN to null) and addition of a few mixed types.
Or we should just let this ext go into the void? :'D
 
When does that vote end again?
 
it just did :)
 
yeah just saw that
 
!!rfcs
C'mon @Jeeves
 
cmb
@MátéKocsis yeah, sure, no problem to wait a bit. Is this about PR #5598, or are there more open PRs?
 
@cmb Yes, that's the only one AFAIR.
 
Well there is my -Wundef one but I'm explicitly ignoring that extension as I knew it was going away
 
@Girgias Haha, I fixed those stuff in ext/xml-rpc because I knew it will go away.
 
:p
 
Fixing compiler warnings is much more useful inside php-src, so that's also a reasonable decision that you left it out :)
 
2:49 PM
ReflectionMethod returns wrong method name for aliases ・ Reflection related ・ #79636
 
@Sjon any suggestions on how can I do that?
@Sjon How?
 
@MuhammadSameer I have a feeling you are modifying code that someone else wrote. If you wrote the linked code, executing another UPDATE query should not be a problem
 
@Sjon I wrote this code but I don't have any idea about inserting newly added products after the user updated it
 
3:16 PM
@NikiC made a minimal build of php-src with PVS-Studio and it did seems to find some legit issues already: https://gist.github.com/Girgias/dce42cecf17314b91e9b7f948374a93f
The log is ~3700 lines were 1400 lines are in the VM and a bazillion other ones are of the type:
V616 The '_flags' named constant with the value of 0 is used in the bitwise operation.
Should probably have excluded the bundled PCRE lib tho...
 
@Girgias The _flags warnings is part of fastzpp implementation
 
Yeah I thought that it was bogus, just need to configure it to ignore certain warnings
 
And all of those "potentially uninitialized" warnings are bogus as well
I mean, maybe some aren't put, it throws this for literally every variable that's part of fastzpp
 
TBH, I wasn't looking at those but more of "low hanging fruit" legit ones like:
/main/php_variables.c 171 err V576 Incorrect format. Consider checking the fourth actual argument of the 'php_error_docref' function. The memsize type argument is expected.
Well I need to check it first
Erf that's going to be annoying to find as it works with the files after they have been preprocessed
 
3:52 PM
@NikiC "An TMP/VAR may only be used once, unless the opcode is specd as non-freeing" Isn't this something that could be checked by the opcache optimizer relatively easily?
 
4:17 PM
@NikiC Can't you talk to Dmitry to check the CI before committing to master? :S JITed jobs fail again since github.com/php/php-src/commit/…
+ I'm wondering if it's worth to ask him to help you with the reviews. It's kinda unfair to leave everything for you :/
 
cmb
@MátéKocsis warnings seems bogus; should be fine to initialize old_res_info to 0, though
 
@IluTov I'm not sure what you're suggesting there
 
@NikiC The optimizer makes a few checks to make sure the SSA is correct. I was just curious if use after free isn't something that could be caught as well (if a TMP is passed to a consuming instruction is shouldn't be used again).
 
@cmb I see it now, thanks! Unfortunately, the effect is the same whether it is a bogus warning or not :( Our PRs have a red CI status and someone has to commit a fix or revert (and that someone is usually Nikita).
 
5:00 PM
@IluTov Ah okay. Yes, that should be possible
Walking the use list of tmpvar results should do it
 
5:33 PM
@MátéKocsis I'm going to propose a tuna sandwich.
 
6:12 PM
@Danack How wise! :D :D The funny thing is that his wish has already come true since we worked a lot on error messages (but most notably on the same type errors he mentioned)
 
6:31 PM
@cmb JIT is still not OK on MacOS. Also, I committed changes to phpdbg simultanously to you, so that might caused issues on Windows. But interestingly, only on the opcached one.
 
6:52 PM
(op2_info && MAY_BE_LONG) -> (op2_info & MAY_BE_LONG)
 
7:06 PM
Hello again :)
@cmb you about ?
xmlrpc RFC has passed, and I'm ready to begin the work now.
(for transparency, CMB and I have been talking in private about approach, steps to take and so on) .. which is what spurred on my discussions last time about better understanding pecl/pear (again). Cheers
Who's around and active these days, in the PHP team, to communciate with about me generating DDL's - is Kalle still working on this? Please update me. Thanks <3
 
7:30 PM
ldap_connect ignoring port ・ *Network Functions ・ #79637
 
7:43 PM
@PaulDragoonis fyi, chat is asynchronous.....if you give people a link to the things you need help with, that will probably be more effective...and for the record, I avoid xml.
 
8:21 PM
@Danack I've DM'd you on twitter, can you check it please?
 
Is it reasonable (or realistic) for PCNTL to intercept a signal when a socket is in blocking mode?
I've tried with both ticks=1 and pcntl_async_signals(true) and it isn't giving me much joy, but I haven't got to the point of reducing it to a simple test yet, in case this is never going to work.
 
ThW
@Danack Long live YAML? :-)
 
"me generating DDL's" <-- this should have been DLL's not DDL's :) Sorry! :D
 
@ThW csv 4 life.
@PaulDragoonis for imagick, I tend to work in a container github.com/Imagick/imagick/blob/master/docker/developing/… with the docker compose file in the root, and github.com/Imagick/imagick/blob/master/docker/installPHP.sh for compiling PHP.
I have zero knowledge of how to build on windows.
 
I have ubuntu (running in win10 with WSL1), so it's still native ubuntu. I'd like to get an environment setup to pull down the phpsrc, compile/run tests.

Are there any links/docs on achieving such a setup, and the system dependencies required? I'm happy to work in Docker if that's the way forward./
 
8:33 PM
@Stephen Generally yes. Assuming syscall restarting is disabled. You might be running into issues like bugs.php.net/bug.php?id=79564&edit=1 though. strace should tell
 
@PaulDragoonis you may find the scripts in github.com/php/php-src/pull/4781 helpful.
they were written for a pretty vanilla Debian from memory, but should work on Ubuntu too.
 
@cmb looks like phpdbg tests on windows are broken on master
 
@NikiC thanks for the tip. So it seems that it does eventually catch the signal. when I send the signal I get the first two lines of the following, and then 60s later the last line (plus a heap more as the user land signal handler causes it to exit):
--- SIGTERM {si_signo=SIGTERM, si_code=SI_USER, si_pid=15834, si_uid=1000} ---
rt_sigreturn({mask=[]}) = -1 EINTR (Interrupted system call)
poll([{fd=11, events=POLLIN|POLLERR|POLLHUP}], 1, 60000) = 0 (Timeout)
 
@Stephen That does look like you're hitting the issue I linked
The syscall gets interrupted successfully and we just ... ignore the interrupt
 
8:50 PM
right glad im not going crazy.
so I can work around this, ish. It seems like it recognises the signal after whatever value is set for stream_set_timeout
 
RFC - Namespace in core - wiki.php.net/rfc/php-namespace-in-core - I'm noticing a lot of the core team are voting against this, is there a mailing list entry I can look at to understand this better, and where the conversation is happening ?
 
@NikiC am I reading that git blame correctly? That code hasn't change in mostly 16-18 years, but we both found the same bug this month? Am I missing something external that would suddenly be causing this to surface? (This is just idol curiosity now)
 
@Danack ಠ_ಠ
 
@Danack github.com/Imagick/imagick/blob/master/docker-compose.yml - with this volume, what should be checked out locally? php-src, and should I do that from github or from git.php.net ? I have karma on git.php.net and not on github. Should I be making PR's from my own fork of PHP-src repo, or can I have my own branch on php-src repo? Thanks
 
You'd probably be better following nikic's link. I'm don't have access to git.php.net, on the grounds that the chances of borking it, are zero.
And other's will be able to comment on your work flow question better than I can.
But I just checkout php-src into a directory....and it gets mounted into the container.
 
9:04 PM
@NikiC oh scratch what I said about the timeout. It's not the value of the timeout that matters. Having any timeout set will cause the signal to interrupt, it seems.
 
@Danack @NikiC - a better way to phrase my question is - how are you guys currently cloning down the php-src, is it from github? or from git.php.net? How are you pushing up your changes are you using a branch of the php-src repo or are you maintaining your own forks? Thanks! (EDIT: The latter being a github specific approach)
 
again, I don't write much core code, but doc here wiki.php.net/vcs/gitworkflow say "but developers are encouraged to fork php on github and start implementing the feature there on the respective branch."
 
cmb
@NikiC Thanks for the ping! That is a JIT issue (can repro the 17 test fails locally). Will try to narrow it down.
 
9:21 PM
o/ @cmb
 
cmb
@PaulDragoonis thanks! It seems to me that the actual unbundling is trivial. I have a patch ready (delete ext/xmlrpc, and some xmlrpc related info), but has to wait for PR #5598. Preparing xmlrpc for PECL may not be that hard either, but I won't have time before end of the week. :(
BTW, if you like to set up a php-src build environment on Windows, see gist.github.com/cmb69/47b8c7fb392f5d79b245c74ac496632c
 
I figured it would be trivial which is why i wanted to pick up this low-hanging-fruit as a way to get back into contributing to php-src (thus my initial msg to you).
 
@PaulDragoonis If you want to push directly upstream, you need to use the git.php.net remote. Otherwise use a fork on github as usual
 
@NikiC what are you doing? github?
 
Both ^^
It's really the usual setup of having an upstream remote and a fork remote ... just that in this case upstream is on git.php.net, while the fork is on github.com :)
 
cmb
 
10:31 PM
I've went with github approach this time around, rather than using git.php.net. I'm compiling php-src on my ubuntu machine (no docker).

It took 54-seconds `./configure --enable-debug`
I'm doing `make -j8` now
make test - does this report get saved anywhere? or do all I have to work with is the STDOUT output ? I want to investigate the failures, but the scrollbar doesn't go back that far.
 
It does get saved into a file at the root of your php-src normaly
 
I'm reading the Makefile, trying to figure it out. run-tests.php takes a --html argument
 
10:52 PM
I used: make TEST_PHP_ARGS=-j8 test --just-print, to see what the makefile was upto :)
 
00:00 - 23:0023:00 - 00:00

« first day (3510 days earlier)      last day (1424 days later) »