« first day (3399 days earlier)      last day (1773 days later) » 

00:54
@bwoebi What the heck is a mysqli? Is that contagious?
01:14
Q for the room, here on github.com/php/php-src/pull/5143/… can I get away with just a RETURN_STRING?
I'm not familiar with how the counting operates within strings of ce's
 
1 hour later…
02:35
If I feed a man a fish, they'll be full till tomorrow. If I teach a man to fish, they'll be full as long as they have access to fresh water. If I build a man a fishing boat, they'll be full until the boat breaks down. If I teach a man to build a fishing boat they'll be full until the Oceans run dry.
So I should probably just teach them how to farm, amirite?
03:27
@MarkR no, current code is correct
class names are interned, but it doesn't matter, there is nothing saying the names have to be interned so they must be treated like refcounted strings
04:01
Thank you Joe.
 
1 hour later…
05:11
I love how simple blogs were back in 2001
 
2 hours later…
07:01
o/
cmb
cmb
07:35
\o
08:25
o/
08:40
@Sherif you use mysqli when you need functionalities the generic PDO driver doesn't have
09:07
Yay @Ekin
@heiglandreas Does the karma work today, or still issues?
09:33
Stil Issues @nikic
But I sent @salathe an SSH key so that I can then use the bot for pushing.
And so far I still need to redo the clone with the correct authors file
BTW: Has anyone here ever envountered the message "fatal: name consists only of disallowed characters: " when running git svn clone?
@heiglandreas hrm... Can you try a dummy commit to web/doc.git or web/doc-editor.git?
(whitespace change or something)
To check whether the karma propagation is broken, or it's something else
@heiglandreas nope
Will check soonish and let you know!
@nikic web/doc.git works as expected. doc/es.git still not...
okay
I'll have to remember how to authenticate on the git server and check the cronjob...
09:58
@heiglandreas Give it another try
Turns out the svnroot cronjob was just running super rarely. I've changed it to hourly now and executed it manually
@Tiffany yay! what are we happy about? :D
@heiglandreas Does that happen when using the author map?
10:25
@Derick NEWS on PHP-7.4 seems out of date?
I added a new section, but probably some NEWS entries are in the wrong section now
Why would it be out of date?
Looks like I forgot to push yesterday after branching :-/
... why was there a 7.3.15 and 7.3.16 in there?!
cmb
cmb
@Derick in 7.4/NEWS?
yes
cmb
cmb
where? Did you already fix that up?
no, Nikita just did
@cmb Looks like you might have in d9f51e0e498abfb79fe23296f3761153ffce0c66
cmb
cmb
10:31
but that was an empty merge
at least it was supposed to
how odd
looks like it too
@cmb this is an odd one though: github.com/php/php-src/commit/… - already shows 7.3 was in 7.4?!
it's a mystery for which I don't have time now
cmb
cmb
github.com/php/php-src/commit/… went into PHP-7.3. The following upward merges are empty. Don't understand either, but if it's fixed, let's go on. :)
@Ekin your doc patch was committed :D
@Derick I didn't fix anything
@Tiffany oh nice, yay indeed :)
yeah I thought that'd be the one, cool
glad it's going to be out there soon
Making your mark in the docs :)
nevermind that, it's good to be contributing in some way again :p it's been a while
I get way too sucked into work these days
@NikiC Yay! Worked! Thanks!
@NikiC Looks like it. When I ran it without an author map everything worked file. But I also have an author prog set that should take care of "unknown" authors and the authors file does not contain empty rows....
10:57
@heiglandreas And of course it doesn't print which name it is :(
That would be faaaaar to easy, wouldn't it?
It stops at the commit nr. 137810 which was done by "didou" but I'm not sure whether that is the problematic one or the commit after....
@heiglandreas next one is by abies. Both don't seem to have any weird names...
Yeah....
These names I remember all from way in the past!
11:01
I'll leave that simmering for a bit and have a look later today....
@Derick Might be due to the fact that those commits happened 16 years ago.... :-)
you calling me old?
It was one commit further
@Derick Me???? Not at all!
Author map entry: changelog = <[email protected]>
That one is missing a name
:facepalm:...
I'll fix that manually. Thanks
I was only looking for missing "full" names but not whether the poart in front of the "=" was empty....
 
2 hours later…
12:37
I want to know which git branch is on Kubernetes. I have a git commit hash or a tag (I get it from Kubectl). How can I get this? Git branch --all --contains can return another branch that contains this commit so this hack doesn't really work
Problem is that commit hash can be in multiple branches, even newer ones than the one deployed in the container
git status? the first line should be the branch
Kube doesn't work with git, works with containers. So a container is created with the git commit hash in the name and that is what is on kube. Git status would show me what's on my local, and there is no git in the container
tag the container with the branch and the commit hash (you can tag them more than once). then do the deployment based on the branch tag instead of the commit. that should do the trick
13:03
I don't control the ops processes unfortunately
Maybe getting the branch with the commit HEAD that has this SHA? How would that go
@Jimbo git for-each-ref | grep sha1-hash
13:27
o/
cmb
cmb
14:00
\o
o/
14:17
\o
 
1 hour later…
15:19
o/
15:39
o/
\o
15:56
/o/
jesus, I can't possibly send a kitten to the docs list :-P
I might have emailed you many times with the same kitten @Sherif :D
sorry about that
@Ekin Well, you're off the hook for now.
:-)
oh I see why... I tried to subscribe to the list and it failed. So now it's not letting me post there
> Sorry, only subscribers may post.
KITTEN
does that mean it finally worked? :P
16:10
Yes
ah great :-)
Any long time Laravel users have any advice for me? stackoverflow.com/questions/60007712/…
@bassxzero run like hell?
16:51
I'm getting this compiler warning (added some new flags) and I'm not sure how to solve it and even if it can be:
https://gist.github.com/Girgias/a25c36cdb789585023af68035bf18b7b
@Girgias Looks like false positive. Try replacing unsigned short length with size_t length and remove the size_t casts.
Okay will do, cause that's the last one of that type
Seems like that fixes it indeed :)
FInally fixed the issue with "name consists only of disallowed characters" when cloning the repos. Had to fiddle with the Perl code of git-svn to get the actual revision git was complaining about. Again an empty name in the authors file. I need to fix that....
@heiglandreas Can you share the entries you modified (and how)? So I can update our reference mapping
I'll share the script I used to fix it.
17:06
I guess it was ali? ^^
It was indeed :-)
@NikiC I also updated my sockets warning promotion PR, mind having a look again at it? :)
ANd ali and changelog where the only ones with an empty name. So now it should run smoothly
So I won't post a script as that'S easier hand-coded ;-)
@salathe ^^
17:23
hullo, I was wondering what order can/should I put session_start() vs ini_set()?? I always read people say put session_start before EVERYTHING, but not sure exactly what "everything" means in that context
ini_set first
@agentprovocateur everything meaning output
The only reason why you shoul put session_start before anything else is that it's easy to forget to access the session before initializing it or doing output
As the headers will be flushed the moment you have output (response body)
oh i see
thats very different from what I thought lol
thank you guys!
17:32
YW :p
17:44
Erf, I forgot the standard ./configure doesn't enable everything we test on CI
18:13
@cmb (because I see you work on libgd a bit too) how hard would it be to upgrade the bundled libgd for PHP 8?
cmb
cmb
@Girgias for years now the plan is to unbundle libgd, but the external libgd doesn't support custom memory allocators, so I wouldn't do it yet. Also, external libgd didn't have a release for years, and I'm not sure whether this will change in the next months.
Furthermore there are some minor differences, so BC break would be inevitable (mostly for Windows users).
@cmb github.com/php/php-src/pull/5151 mostly because libgd throws a -Werror=type-limits, so can I patch our version directly?
But seems like a tough call :-/
cmb
cmb
ah, gd_interpolation.c (special favorite besides gd_imagecrap.c)
if you have a good idea how to solve this, great! Otherwise I'll try to have a look.
@cmb I like the honesty in that last name
It's just removing a condition
/home/travis/build/Girgias/php-src/ext/gd/libgd/gd_interpolation.c: In function ‘gdImageRotateBilinear’:

/home/travis/build/Girgias/php-src/ext/gd/libgd/gd_interpolation.c:1777:11: error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits]

if ((m >= 0) && (m < src_h - 1) && (n >= 0) && (n < src_w - 1)) {

^

/home/travis/build/Girgias/php-src/ext/gd/libgd/gd_interpolation.c:1777:42: error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits]

if ((m >= 0) && (m < src_h - 1) && (n >= 0) && (n < src_w - 1)) {
cmb
cmb
18:25
@Girgias there might be deeper issue there
Well if it is an unsigned integer it must always be greater or equal than 0
cmb
cmb
@PeeHaa well, it's not that bad per se, but it's unfinished (and differs from external libgd quite a lot)
I was planning on only removing the (m >= 0) and (n >= 0) conditions
Not the whole bloc
cmb
cmb
maybe it's not intended to be unsigned
well
cmb
cmb
18:26
afk for now
I'm still trying to get most of the extension that CI tests to work locally
Ok :)
 
2 hours later…
20:19
YAY @Ekin
Lol
what are we happy about this time @Tiffany :D
oh I think I know?
:D :D
\o/
ViCiOuS cAt TrIeS tO cLaW yOuR eYeBaLlS oUt
20:23
:D
It is snowing hard today
Someone take it away
I guess we're due for our two feet of snow
Welcome aboard @Ekin and @SorinNunca :-)
\o/ thank you :-)
21:08
thanks a lot
 
2 hours later…
23:01
@Derick shots fired!
23:35
:-D
So brave @Derick
And I'm the only one who could possibly benefit from it if I use it for mathematical computations :')
23:39
no?
No!
I did some benchmarks back in the day for a pure php impl of redis in reactphp beberlei.de/2019/03/23/playing_with_the_php_jit.html
Not impressive
I'm off to bed. Oidhche mhath!
おやすみ
Gute nacht
23:43
Buenos noches
@beberlei huh, the thing I'm wondering is if it can improve pure math computation like Matrix multiplication
But we don't really have good math libs so
Yeah
Maybe composer has one?
Yeah I know there is math lib on packgist
there's the one... a dude I can't remember
This weekend my plan is to try to do my Numerical Analysis coursework in PHP
(just to confuse the GTAs)
But will see how far I can push that
23:46
that logo
@Derick This is why I was one of only two "no" votes on the RFC. Was kind of astonished there were only two of us...
Until it can neatly solve inlining I bet it will remain mostly useless.
Tbh I didn't vote on the main vote as I thought I had no say as I have not the technical knowledge
But the proposal for inclusion in 7.4 was pretty dumb
cmb
cmb
neither of these functions are in external libgd, btw

« first day (3399 days earlier)      last day (1773 days later) »