« first day (4192 days earlier)      last day (756 days later) » 

12:22 AM
@Crell I'm okay with merging them if cmb approved, but not sure if they should be squashed before merging, and I need to distill a commit message which I don't wanna use brain power for, presently
 
1:10 AM
o/
 
 
3 hours later…
3:56 AM
\o o/ \o later...
 
 
5 hours later…
cmb
8:39 AM
@LeviMorrison there is no AC_RUN_IFELSE (or such) equivalent on Windows, and like @bwoebi already said, in practice you can rely on MSVC behavior (clang builds are broken for quite some time now, and I wouldn't know if anybody tried ICC for years).
If there's a strong need for something like AC_RUN_IFELSE, it might be possible to add support, though.
 
@TimWolla Merged the RFC. Congrats :) Don't forget to update the RFC status and RFC page.
 
@IluTov Already seen the email, thanks. I'm already in the Wiki and looking up how that update needs to look like :-)
wiki.php.net/rfc/…. Does this look right or did I miss something?
 
@TimWolla Nope, looks good!
 
Also in: wiki.php.net/rfc#php_82. At the top it says "Order in these sections: Language changes first, library changes later". Should "Locale-independent case conversion" be moved to the bottom in 8.2? Or what exactly is a library change?
 
@TimWolla We don't have rules on that. Most people put it at the top but it's fine either way.
 
cmb
10:07 AM
It seems the Windows PECL build machine died. :(
 
10:22 AM
Now that the SensitiveParameter implementation is merged, the attribute should be added to some native functions. Derick already suggested that support for attributes should be added to the stubs: github.com/php/php-src/pull/7921#discussion_r802682655, but this is definitely above my "pay grade" in PHP Internals.
zend_add_parameter_attribute(
		zend_hash_str_find_ptr(CG(function_table), "password_hash", sizeof("password_hash") - 1),
		0,
		zend_ce_sensitive_parameter->name,
		0
	);
That's what I'm doing currently, but it's awfully verbose. Is there some nicer way to get a specific function, other than zend_hash_str_find_ptr and then passing the same string twice (once for the sizeof)?
 
10:46 AM
@TimWolla I asked Mate about it.
(He does a lot of things with stubs)
 
o/
 
\o
 
11:02 AM
@TimWolla I discussed it with Mate, I'll attempt to support @sensitive-parameter in the stubs, then you can add it there.
 
11:31 AM
@IluTov Oh, not full attribute support, but rather a doc comment. That seems like a good short term solution.
 
@TimWolla Yeah, apparently that's easier according to Mate. Maybe we can switch all of them over to actual attributes at some point.
 
 
1 hour later…
12:45 PM
@TimWolla GitHub actions is something -.- The if is evaluated after matrix (which makes sense, as you could skip single jobs). The problem is that the matrix generate step generates an empty matrix when no branch has changed. And GitHub actions errors when a matrix parameter is empty. Are you aware of a workaround?
 
Do you have a link to the failing run?
 
Oh wait, maybe I just made a mistake.
Essentially, I tried skipping the jobs if the matrix is empty, but I don't think the evaluation order works that way.
 
I googled a bit, you technically could add a static job in the middle that is skipped if the build matrix is empty, so basically:

GENERATE_MATRIX generates the matrix. CHECKER depends on GENERATE_MATRIX and skips if the matrix is empty and then the actual BUILD jobs depend on CHECKER:
But that's certainly not pretty.
 
@TimWolla Hm... And that will be necessary for every job (only two atm but will be more)
Thanks for the research. Honestly, I should've just stuck with the generated workflow :P
 
Simple solution: Always rebuild 'master' every night.
Then the Matrix won't be empty :-P
No need to waste CPU cycles on rebuilding unnecessary stuff, but 'master' is likely to change regularly and rebuilding it even if it didn't change is not going to be the largest contributor to earth warming.
 
1:03 PM
Hm according to this it should work with != '[]', trying right now.
Nice, that worked
@TimWolla Does everything look good to you? I'll adjust the triggers before merging. Unfortunately I can't test the comment trigger since those are only run once on the default branch.
The comment trigger will create a lot of actions that do nothing, unfortunately. But I don't think there's a way to solve this other than use a bot (which I've never do and assume is going to be much more time consuming).
 
I'll have a look later.
Are you technically able to re-request a review from me now that I've reviewed?
 
Cool, thank you
@TimWolla Yes. But a message is more personal :P
 
Was just curious about the UI :-)
I like the review requests, though. Makes it easy to keep track of what's open (especially when combined with Slack Reminders which are natively supported by GitHub).
 
Hm, did you get an e-mail? The ui is bad, I click the icon, nothing happens. You might've just gotten two :P
 
1:19 PM
Nope, didn't. UI doesn't show me a review request either.
So it's broken for non-members of the org, I guess.
 
cmb
Apparently, I can't re-request a view either. Perhaps this just requires that the reviewer had "requested changes"?
or possibly GH has some outages
 
I'm so confused how missing ZPP checks in SPL could go unnoticed for years apparently
 
1:34 PM
@cmb It sends some ajax POST that returns a 200 response. So who knows, might just be borked.
 
1:45 PM
@cmb I can't even request changes, because:
Easy solution: Just give me access! :-p
 
cmb
@TimWolla I think in this case you also should have a php.net account: php.net/git-php.php
 
@TimWolla I think that's done because many people who haven't contributed would just approve random PRs they like.
 
cmb
yeah, that happened multiple times in the past
 
Ah, is this configurable per organization? I remember DefinitelyTyped/DefinitelyTyped making use of that (i.e. non-member approves).
@cmb I was partly joking there; though if you believe that it would be valuable, then I'm happy to fill in that form.
 
cmb
@TimWolla IMO yes. @IluTov, thoughts?
 
1:57 PM
@cmb Sure, many people got their account after an RFC. No objections.
 
Initial Karma would be "PHP group"?
 
cmb
@TimWolla yep
 
@cmb Okay, filled in the form (hopefully correctly :-) ).
 
cmb
@TimWolla request approved; GH invite sent :)
 
@cmb Thanks. Quick question regarding the Wiki: Was the account merged automatically? I noticed that I got logged out and then for a short time neither the Wiki password nor the php.net password worked.
 
cmb
2:11 PM
@TimWolla that should have changed to phpcvs group; that means you should be able to login with the php.net account password (that might take a while, though)
 
Yeah, the php.net password worked after waiting a bit. I just wanted to make sure that this didn't result in two distinct accounts internally, possibly causing issues down the line (especially since Wiki and php.net used distinct email addresses; Wiki was my work mail).
 
2:45 PM
welcome to the club; now you get a fancy @php.net e-mail address :)
 
2:58 PM
@IMSoP Thanks! But me also being someone who deal with the various issues of "sending emails" by running my own mail server: Any type of email forwards to domains not under your control scare me :-)
If my mail server would reject the forwarded email for whatever reason, then php.net would need to send a bounce to a possibly forged sending adress (this is commonly known as "Backscattering"). So I'd rather shy away from using that alias to not cause any issues for the PHP.net mailserver.
 
@cmb Thanks. I'm still battling through the basics of Windows CI, so dunno yet ^_^
What license should I use on a new file, say zend_atomic.h that doesn't give anything to Zend because I wrote the thing, and they deserve no credit? No sure why fibers got ZE license also...
I'd prefer not to have zend in the name of the file as well...
 
So Dale's safe fruit spam gets through, but every time I try to make a release announcement, I roll the dice if it'll get bounced back
 
PHP License? then you give credit to the nebulous "PHP Group" instead
 
PHP Group has got to be better than Zend, I think?
 
3:14 PM
The upside of the PHP Group is that they can NEVER change the terms of the license or sell out. You'll literally never find half of them.
It is a little weird that stuff under Zend/ is specifically licensed to Zend the company. I'd probably put the PHP license on it and let someone explain why that's wrong.
 
@Sara It gets especially fun with all the different jurisdictions. IANAL, but here in Germany I don't think that you grant Zend-the-company any special rights, just because there's a "(c) Zend" line in the file, as the copyright law is pretty strict with regard to procedure.
 
Thank God this is an Open Source project and nobody wants to be the asshole who tries to sue for ownership only to end up with half a language (only the engine is licensed that way, and the much larger language runtime is.... weird.... )
 
cmb
@Sara on the flipside, Zend might cater to license violations of third-parties, the PHP group is unlikely to do so ;)
 
4:04 PM
@Tiffany Squashing for merging seems to be the standard for php-docs. It's what I've been doing lately. ("Fixed the indent again, damnit" is not a helpful commit message to retain.)
 
4:22 PM
Amended the true type RFC to include the compile time redundency checks
 
4:46 PM
@NikiC Here's a PR that uses C11's <stdatomic.h> and falls back to non-atomics, which is what we do today so it shouldn't be worse. Windows uses C++'s <atomic> instead, since their recently added C11 support omits atomics. github.com/php/php-src/pull/8327
Tests passed before rebasing, so if any failures pop up they are hopefully trivial to fix.
There is an expected tiny perf hit to using atomics, but it's probably slightly bigger than it needs to be except when using LTO due to the functions not being inlined.
Maybe all the files could be merged into a .h with macros and such but I didn't see an obvious way to put it into C++ mode for Windows.
 
5:20 PM
@Crell merged 1493, will look at the others until my I get a call about my cat
 
@Tiffany Kitty!
I think 1492 is the only other one I still have outstanding.
(Though there's ample from other people that are also pending and could use attention if you have bandwidth.)
 
Vet appointment for his ear :( might have another ear infection
 
Poor kitty. :-(
 
Usually need to pull out a whiteboard for reviewing. But I can give it an attempt until I hear something. Nothing else useful for me to do while sitting in my car.
 
5:38 PM
There was an attempt to enforce line length at some level in doc-en, did that ever fruition? Or is it still up to reviewers to enforce?
 
I'm not aware of any specific rules on that.
 
I remember you made an attempt to enforce it through a GitHub workflow I think, but IIRC it didn't get merged, but I think that was before the complete migration to GitHub
Looking at a PR from my phone and doing a lot of scrolling left to right. Not sure if character length is followed...
 
cmb
doc.php.net/tutorial/style.php: "Please aim to keep lines in an XML file around 80 characters long or less. This is a loose requirement and 100 is probably acceptable as a maximum length."
 
I tried to add auto-enforcement of no trailing whitespace. That got shot down because of the noise it would cause for translators. :-(
 
5:53 PM
Ahhh, fair
I need to verify from a PC that it's within character limits, but checking visually, it doesn't seem to be
 
6:15 PM
Definitely too long.
Especially as the previous line is still there, so it's one hand-broken line and one no-break line. Do you want to respond or should I since I'm on a real keyboard? :-)
 
 
1 hour later…
7:21 PM
@LeviMorrison Why are you declaring atomics implementation out of line?
On platforms we care about these are just plain loads/stores, they shouldn't be out of line
And there shouldn't be a need to use c++ with msvc, they have their own functions for that. InterlockedXYZ iirc
 
 
2 hours later…
9:04 PM
Can anyone remember what the crypto function is that deliberately accepts a mixed so it can throw errors on anything that isn't explicitly a string (to avoid coercion)?
 
9:24 PM
@MarkR hash_equals does this. Maybe others?
 
 
1 hour later…
10:53 PM
@DaveRandom Are you buying weekend tickets or just the ticket for Kate's day?
 
Wes
\o
 
11:29 PM
\o
 
Heya guys o/
 
Wes
how ya doing phpellow phpriends
 
Tis the weekend aye \o/
 

« first day (4192 days earlier)      last day (756 days later) »