« first day (3511 days earlier)      last day (1423 days later) » 

12:07 AM
@pmmaga I've fixed it. It ain't pretty, but it works ;)
 
:)
 
The main issue I see here with this code, is that the worker code is embedded inside the run-tests.php code, so they're all convoluted together, and are stepping on each other's toes. run_all_tests() code is expected to work either as the main php process or as one of the worker processes.

Thus all the mental shit going on.
 
12:31 AM
@Tiffany I saw that, and what @LeviMorrison wrote, this morning both at breakfast.
 
@Derick if I added a class file named TestUtil.php, and imported it into run-tests.php, would people be unhappy? :)
 
1:17 AM
We've added a class inline, inside run-tests.php, "RuntestsValgrind", and I'm looking to make an additional class, but have it as a separate file, instead of inside. What's the deal here? AFAIK the next master target is for PHP8, and we don't have overhead issues anymore, like we used to.
master target = master branch
 
 
2 hours later…
3:31 AM
run-tests doesn't pass all info down to workers ・ Testing related ・ #79639
 
3:58 AM
Morning, I got the following failing test output - there's no explanations at this summary part. Is there a way I can uncover more thorough details as to what specifically failed? (assertions that didn't match: expected vs actual) ? Thanks - gist.github.com/dragoonis/f21dce17748b71d45102e42cf331c31a
It's good to be back :) I've made my first PR - expect followups - github.com/php/php-src/pull/5632
Going to sleep now, bye!
 
 
2 hours later…
5:44 AM
Annabug ・ *General Issues ・ #79640
 
 
1 hour later…
cmb
6:53 AM
@PaulDragoonis see qa.php.net/write-test.php#analyzing-failing-tests. You can also pass --show-diff to run-tests.php, to show the diffs in the output. Or set TEST_PHP_JUNIT=report.xml to generate a JUNIT report (also includes diffs for fails).
 
 
1 hour later…
8:02 AM
@NikiC While reading nikic.github.io/2020/05/10/Make-LLVM-fast-again.html a while ago I thought: Should we not have compile-time performance benchmarks (how long does it take to compile a piece of PHP code) and runtime benchmarks (how long does it take to execute a piece of PHP code) as part of CI? Probably not for each push, but maybe nightly? weekly? for tags?
Having official benchmarks to link to would be nice to show improvements (and of course for detecting regressions).
 
cmb
With the current CI, this might not make much sense; it seems to me the performance of these VM varies too much.
FWIW, for Windows there are perf benchmarks available at windows.php.net/downloads/snaps/ostc/pftt/perf
 
@cmb That seems not to have been updated since 2018?
Nevermind, I was looking at the wrong things.
 
cmb
I'm doing these benchmarks for every release (QA and GA). :)
 
8:17 AM
What was WinCache again? Windows-specific addition to OpCache?
 
cmb
In the olden times it was a Win specific alternative to OPcache; now it offers some additional caching capabilities (e.g. file caching), but I think it mostly abandoned.
 
@SebastianBergmann Would certainly be nice...
But sounds pretty icky to set up
 
@NikiC still working on PVS-Studio, but I will need to setup some private variables for the license key
 
@SebastianBergmann Probably the most valuable starting point would be to just have an automated way of setting up all the necessary test applications
Like, I'm too lazy to even install wordpress, don't want to figure out how one does a proper mediawiki setup
 
8:27 AM
@PaulDragoonis Depends on what it does and whether it messes things up
 
@Girgias Do we actually want to run that in CI?
 
@PaulDragoonis You certainly should not make run-tests require extra files. It should be one .php file.
 
@NikiC also a good question, I'm still strugling with some exclusions because the docs are not that clear, but I can ignore warning codes so the report should be more useful, and I also think that due to WSL1 I'm getting some bogus warnings
 
8:45 AM
@cmb Do you publish those numbers in machine readable format anywhere? Just slapping a json file somewhere would be nice...
@Crell turn off the file cache for the routing? see if that makes a difference maybe.
 
cmb
@Danack I'll have a look later; currently benchmarks are running.
 
For the record, I have a strong belief that getting better at having distributed systems, that can be plugged together in interesting ways is a really useful thing to have.
Which reminds me... @PeeHaa @PeeHaa @PeeHaa how is it going?
 
9:14 AM
@cmb Regarding you comment "What about dropping f and q altogether?". Do you mean to only drop these variables or to drop the parameters themselves? I've already managed to drop the f variable.
 
cmb
I meant dropping these variables.
 
OK, I hoped so :D Can I use zend_long and int interchangeably? If so, then I can also drop q. Nevertheless, I can figure this out
 
Isn't zend_long a long int?
 
cmb
It's long long on LLP64 (aka. Windows 64bit).
 
but no matter if I remove the variables, the problem of finding the right condition remains. :D
 
cmb
9:17 AM
@MátéKocsis in this case, yes, since there are no casts (or even checks) anyway.
 
ok, thanks!
 
cmb
Can't you just check for quality_is_null/basefilter_is_null?
 
Yes, that's what I have now, but Nikita asked if we can make -1 the default value in both cases. So I'm currently trying to achieve this
 
cmb
9:51 AM
ah, yes, makes sense
 
thanks, I will use show-diff to get a better idea of why tests are failing.

A question to everyone is: for the list of failing tests I linked to, maybe I don't have system libs or exts installed? Is --show-diff be the best way to track this down, or is there a more specific way? Thanks.
 
cmb
@PaulDragoonis if a required ext is missing, the test should be skipped (respective check in SKIPIF section). So yes, I suggest you have a look at the diffs; might be just some system settings (on Windows I get several failures due to German locale and missing priviledges, for instance).
 
why? I'm not seeing a technical justification for it so far. It's 2020 now, and with PHP8 we don't have the perf issues we had with PHP4. Explain to me what I'm missing. Thank you

/cc @NikiC - I see you commenting on my PR.
 
@PaulDragoonis Because it is used for pecl extensions, and you suddenly have to think about paths, and autoloading. You change it from a self contained people can just run, to something more complicated.
 
are you saying people take run-tests.php out of /php-src/ and use it elsewhere? i.e: steal it and move it around?

In terms of paths, we can identify the current absolute path of the php-src DIR, and use /test-utils/SomethingHere.php
 
10:01 AM
@PaulDragoonis It looks like tests using the builtin server fail for you
@PaulDragoonis Yes, most PECL extensions use run-tests.php for their own tests
 
@NikiC can you link me to an example of this please, so I can better understand the context, before I make my next decision?
 
phpize will generate run-tests that you can use to test your PECL package
 
and it is also normally included in -dev packages (packages.ubuntu.com/groovy/amd64/php7.4-dev/filelist)
 
@Girgias Do you have a link to tell me how phpize "acquired" run-tests.php?
 
However one thing I'd love for run-tests to do for PECL extensions is to not ask you to send your report to qa.php.net cause I'm just testing local changes aaaaaaaaaaaaaaaaaaaaaaaaaaa
@PaulDragoonis Sadly I don't even know where the phpize repo is :/
 
10:12 AM
@Girgias also, the fact that people do think they go somewhere....
 
cmb
See pmmaga's comment above (I think run-tests.php is just contained in -dev packages)
 
@Danack Well they go into some mailing list void...
 
cmb
Same for Windows, by the way.
 
I'm here to re-evaluate the developer experience of our test suite. that's core dev, ext dev and standard users who wish to help in the running of the tests.

If you have a suggestion on how to better manage this situations, do reach out so I can think about it further.
 
cmb
nope, reports are all there (qa.php.net/reports); it's just that nobody looking at it :(
 
10:14 AM
Well my only complaint is that annoying prompt where you need to pass --quiet/-q
 
@pmmaga I found this file in php-src: "scripts\phpize" - is this the correct official phpize that we use all the time ?
 
I'd guess that it is used by package maintainers, yes. Not sure if that makes it "official" :P
 
Every time a member of the php community compiles an extension for their project, they run "phpize" in a extension's src dir. Is this the same script? That's what I'm asking.
 
@PaulDragoonis yes
 
As per this link - https://github.com/php/php-src/blob/master/scripts/phpize.in#L14

it's already doing run-tests*.php, which tells me there is an opportunity here. Does anyone know of any examples where we have more than run-tests.php, such as run-tests-somtehing.php?
 
10:26 AM
@PaulDragoonis Yes. People do that, and modify them, and phpize also does it.
@Girgias You can set an env var for that.
 
@PaulDragoonis I don't believe so, probably a leftover
 
I could
 
In any case, please do feel free to refactor run-tests.php to reduce global state, but keep the class declarations in the same file. It's really not that hard.
 
@PaulDragoonis I am very much not happy about splitting run-tests.php up into multiple files.
yeah, what Nikita just said.
 
(The fact that PHP inherited the Java-ism of "one class per file" is a real shame, actually.)
 
10:28 AM
@NikiC Want to chat about named arguments/stable sort somewhere next week?
 
@Derick ok
 
@NikiC The primary factor is the autoloader I think. If you have a good idea how we can autoload arbitrarily named classes from differently named files you're welcome :x
 
@NikiC Which day?
 
@bwoebi Yeah, the autoloader really drives that design
 
Mon, Wed, or Thu, at 10:00 BST/11:00 CEST?
 
cmb
10:35 AM
/W1 /WX debug build:
C:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt\crtdbg.h(303): error C2220: Die
folgende Warnung wird als Fehler behandelt.
C:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt\crtdbg.h(303): warning C4005: "_malloca": Makro-Neudefinition
C:\Program Files (x86)\Windows Kits\10\include\10.0.18362.0\ucrt\malloc.h(122): note: Siehe vorherige Definition von "_malloca"
uh-oh
 
@NikiC at least I sometimes wish there were easy ways to tell the composer autoloader to just find MyNamespace\*Exception in src/Exceptions.php
 
won't the classmap cover that usecase?
 
@pmmaga IFF you define every single class by name, yes
 
@bwoebi getcomposer.org/doc/04-schema.md#classmap sounds like it will scan those for any class. So I don't think you have to be explicit about every single class
 
eih maybe I misunderstood classmap
@pmmaga how does it perform the search?
 
I'm not opposed to putting the utility classes inside run-tests.php itself. If we need to do that, we will.

There's clearly two separate use cases here.
a) the creation and maintenance of our test suite: clean separated maintainable code.
b) the usage of our test suite for extension developers: a simple process to copy over the test suite files and execute this. phpize already does this for us with its "cp php-tests*.php".

I don't think B needs to compromise on the abilities of A. Nobody is forcing this upon us.
 
10:59 AM
anyone know what the correct annotations are for doctrine when I am using uuids for identifiers, rather than autogenerated ids?
using
/**
 * @Id
 * @Column(type="string")
 */
private string $id;
 
@pmmaga so eih, it walks all the files in that dir at runtime?
 
It seems to not insert the ID, which gives an error.
I thought it'd be one of @GeneratedValue(strategy="NONE") - or similar, but it's not obvious.
 
@bwoebi I think it does it during composer's runtime (ie. composer install) to generate the class map which is then used by the autoloader
 
Can someone point me to the script which generates our releases / release candidates? I'dl ike to see the process it goes through when putting everything together. Thanks
 
@Danack there is a UUID strategy
 
11:02 AM
@brzuchal that appears to give the same error.
 
@pmmaga ah okay
 
@Danack the type in @Column should be uuid as well
there is a package ramsey/uuid-doctrine which has a proper UuidType for doctrine to understand it, you have it?
 
cmb
 
@Danack TBH I always generate the ID in constructor and all required fields first, that way I'm always at the end with proper object
 
@brzuchal no. I have the id already. I just want doctrine to insert the id value that I have generated. The data is actually going to be reused elsewhere in other storage systems, so I don't want to tie it to doctrine directly.
yeah, that's what I want also. it just isn't using my id property.
 
11:07 AM
Ok, separation of concerns but you have to tell doctrine to handle it not as a string but as a Uuid cause it's a Uuid object, right?
/** @ORM\Id() @ORM\Column(name="id", type="uuid", unique=true) */
public UuidInterface $id;
^^ this works for me and $id is instantiated like $this->id = Uuid::uuid4();
 
Hey guys, I could not find a xpath chat - so can anybody help me with a small issue in xpath?
 
I wonder why there is a unique=true, hmmmmm
 
No. it's just a string....
/**
 * @Id()
 * @Column(name="id", type="string", unique=true)
 * @GeneratedValue(strategy="NONE")
 */
private string $id;
 
> Don't ask to ask, just ask.
 
An exception occurred while executing 'INSERT INTO voting_motion (type, name, start_datetime, close_datetime, created_at, updated_at) VALUES (?, ?, ?, ?, ?, ?)'
The insert just isn't setting the ID...
 
11:11 AM
Okay, I have this xpat to find a element that contains a Text :
//span[text()[contains(., 'MyText')]]
But if my Text on the website is case sensitive - i.g. mytex - this cannot be found with my approach. How to ignore case sensitivity? I tried "match" but this does not work in browser console for a tes
test
 
@Danack Ok, I get it, I though it's similar to my case, but I've never delt with strings instead of Uuid objects, I suppose you've already tried without the @GeneratedValue
 
@SpaceToon there are xpath tester websites if browser console doesn't work
 
Personally I don't think using strings instead of Uuid objects has something to do with infrastruture independent storage mechanisms, I prefer to deal with objects
 
@brzuchal yeah, and trying the different possible values of generated values...
 
it doesn't appear that it's possible to make xpath to ignore case
 
11:13 AM
No, I mean my browser console works, but when i use match this element canno be found too
 
@SpaceToon do you have any sample XML input to test the xpath with?
 
@SpaceToon you probably can't. I guess you'll have to iterate over all the text elements....
 
Okay, thank you
 
@SpaceToon if you're using PHP, you can call a PHP function from inside an XPath query
 
12:11 PM
I cannot remember if this was in the original version of The Pragmatic Programmer or not, (I don't think it was) but good section of the revised version of the book nonetheless. media.pragprog.com/titles/tpp20/inheritance-tax.pdf
 
Type cast problem ・ *General Issues ・ #79641
 
Wes
guys i am having a brain fart, how do you call code paths that get executed a lot compared to the rest of the program? there's a common way to say it afaik
high something traffic something
 
hot paths
 
Wes
thanks, that's it :D
 
1:11 PM
is there a static initialiser for an empty hash table?
 
@Danack Does Slim have a file cache for routing? (I've not used Slim before.)
 
@Crell yes. it's just nikic fast route underneath so it has a setting somewhere.
 
Hm.
 
> Note that there’s no invalidation on this cache, so if you add or change any routes, you need to delete this file.
 
1:25 PM
@cmb CHeers, isn't .u.flags = HASH_FLAG_UNINITIALIZED, a problem though?
 
I'm not seeing a router cache setting at all in this app. Strange.
 
cmb
@Derick well, you cannot modify that struct anyways. If you need to, use array_init().
 
I don't want to modify it. I want it as a shoe-in for an empty hash table. I bet I can't zend_release_properties it though
 
cmb
Nope, you can't (it's a const HashTable).
 
yeah, I just saw that
 
1:36 PM
How can I reference a message I posted earlier, I can't reply to my own messages.
 
@PaulDragoonis the arrow-popup on the left should give you a perma-link option
 
Thank you.

As per my message here - https://chat.stackoverflow.com/transcript/message/49481148#49481148

I've identified some areas that could allow us to combine our test utility classes into a single file, for including and for easy usage by extensions.

One is in "phpize" when It copies files across to the /ext/ local DIR, so we can do the combining first, then copy over a single file into the /ext/ dir. Another is in our "make" file when it's installing phpize-type programs. Lastly there's a "scripts/dev/genfile" which we use in making our distributions (releases) and that could do a c
 
xml_parse() fails with XML_ERROR_NO_MEMORY with a mere 17M file ・ *XML functions ・ #79642
PHP with Opcache crashes when a file with specific name is included ・ opcache ・ #79643
 
1:52 PM
@PaulDragoonis No. Keep it all in one file.
 
Ok. I will. Thanks for at least considering.
What visualization/tools are we using to consume our JUNIT output right now, for local developers, extension developers, ci systems, members of the community?
 
@PaulDragoonis I use junit format on my Jenkins instance so I get nice test reports: php-ci.pmmaga.net/job/php-src/job/master/job/tests/4199/…
 
yeah, haven't given that a look in a while now :P but it seems to be more performant than before
 
2:04 PM
is it just me or is bugs terribly slow atm ?
 
@Sjon restarted apache
let me know if it's still slow
 
@Derick that fixed it, thanks
 
@Derick - in your script you are glob()ing over junit files, did you get them pre-created from somewhere? or are you executing run-tests yourself on xdebug code
 
Understood. Thank you. What about the php-src CI? is it using junit output ?
 
cmb
2:33 PM
@PaulDragoonis I think AppVeyor and Azure Pipelines do; Travis apparently not.
 
3:23 PM
Anyone ever worked with MultiDimensional eXpressions (MDX)? I don't have a question about them... just curious how often this query language is used among those in our Room 11 group.
 
@Danack slow :)
@StatikStasis Never heard of it before
 
Interesting. May have to dive into it some for a work project.
 
.travis.yml and others are in /travis/
 
I didn't have "show hidden files" on. :)
 
Free Book from Microsoft on MDX download.microsoft.com/download/0/F/B/…
 
3:40 PM
do I have this correct? 3v4l.org/7RaLV with regards to using self and parent for typed properties?
it doesn't error, but want to make sure I'm doing it right ...
 
@Tiffany depends what you're trying to demonstrate? that particular example is the same as declaring both properties as type A, which in turn is the same as not re-declaring on class B at all: 3v4l.org/tjmRG
 
@IMSoP I don't plan to use it as an example, but I need to add a description for self and parent, and I realized I had no idea how they worked in the context of typed properties
 
as I understand it, self and parent are just aliases, resolved at compile-time to the appropriate class name
 
so I tried building something in 3v4l
 
3:54 PM
this example shows the difference between the two a bit more clearly: 3v4l.org/n820F
 
interesting
 
it's usually instructive to find examples that don't match a rule, rather than just ones that do
 
I was going to combine them on the same table row, but then decided to separate them, though your example makes me think I could probably combine them into the same row again... but probably better to keep them in separate rows so that I can be clear in the descriptions for either.
 
.
oh god I butchered it
 
4:03 PM
@IMSoP Can you only turn over one?
 
+--------+-------------------------------------------------------------------------------+
| self   | The property must be an instance of the same class in which it's defined.|
+--------+-------------------------------------------------------------------------------+
| parent | The property must be a parent of the class in which it's defined.           |
 
I was thinking 4 could tell us... but even if 4 had a vowel on the other side... the A card could still have an odd number on the opposite.
 
@StatikStasis there are two worth turning over, and two which would tell you nothing
 
Oh good- then I got it.
I was thinking only one... and that I was stupid. =) Jury is still out I guess.
 
.... I give up
 
4:06 PM
@Tiffany What's wrong?
 
@StatikStasis tried building an example of the two rows with the descriptions, but couldn't get the table formatting correct within SO chat... and I didn't want to edit the message a million times
 
Looks good =)
enough
 
how is the parent description?
> The property must be a parent of the class in which it's defined.
 
could be clearer that sub-classes are allowed in both cases - basically, anything where $foo instanceof parentwould return true
 
The property must be a parent or sub-class(?) of the class in which it's defined.
...I don't like that
 
4:11 PM
some more examples: 3v4l.org/2qeIS
 
what's the difference between child and sub-class? wouldn't B be a child of A?
 
B extends A and C extends B, C is a subclass of A not a child of A
 
ah
 
The property must be defined in a parent class, which the sub-class is extending...? @Tiffany
Eh...
There is a better way of saying this clearly.
 
I'm thinking more along the lines of, "The property must be a parent of the class in which it's defined. Optionally, the property can be a sub-class."
 
4:17 PM
I'm usually too verbose when describing these things... but I'd rather be verbose and understand it than to attempt to be concise yet too ambiguous.
 
but optionally could be misconstrued as "inclusion of parent and sub-class" which is... not right
@StatikStasis I prefer concise and clarity. Remove any extraneous words that don't offer anything to explain the situation, while also being as clear as possible.
 
Well yeah... as long as the clarity part is there.
 
I'll need to write it to where it specifies it's an XOR
 
I try to explain things to where a 5 year old can understand, where applicable and within reason.
There have been many pages within the PHP Docs that I have read and thought "...do what now? O_o)
 
you should revise them :D
my current workflow is writing stuff out on paper, scratching out words I don't like, rewriting them on paper, until I find something I like... but I'm beginning to run out of scratch paper
 
4:25 PM
LOL! I do the same thing. I like using the large Ledger paper when I am in a meeting because it gives me plenty of room to write for a long time while they're talking during meetings.
I do the same thing for coding. Always on paper first.
 
for code, it depends if I understand it already
 
Mainly talking about a new feature.
I flesh it out on paper first; a wire frames on paper sort of thing. Then I do a database schema. Then I jump in my IDE.
 
yeah
just realized, it can be an inclusive-OR, so writing "optionally" may work
 
cmb
@Tiffany maybe "the value of the property must be an instance of the parent of the class in which it is defined"?
 
OpCache opcache.max_wasted_percentage cannot be set over 0.50% ・ opcache ・ #79644
 
4:40 PM
> The property must be a parent of the class in which it is defined. Optionally, the class may be a sub-class of the parent instance.
@cmb if I add "value of the property," I'd have to add that for each description, right?
unsure if that's too wordy or the right amount of clarity
 
cmb
I guess it depends :)
 
Would there be any reason that my login fails on apache?
 
yes
 
What could be the reason?
 
Who knows, with no information not gonna go far
 
4:58 PM
@earlyriser01 you need to give some specifics to your apache configuration
as your question stands, we would need to be psychic to understand the problem
 
Okay. Here's my apache configuration: codeshare.io/5QEeQQ
Someone else is telling me to put a no override?
 
NoOverride is always a good idea
 
@Girgias LOL!
 
Many thanks, I was a comedian in a past life
 
So my project is located in /var/www/html @DaveRandom
but I only see <Directory "/var/www/html/cgi-bin">
 
5:04 PM
@Girgias I can see that. Perfect amount of subtleness and excellent delivery.
 
if you don;t have it then a request for /foo/bar/baz/quz.html will generate stat("/foo/bar/baz/.htaccess"); stat("/foo/bar/.htaccess"); stat("/foo/.htaccess"); stat("/.htaccess"); for every request (maybe others)
 
could this be a problem?
 
@Girgias who died in poverty, wierdly
 
@DaveRandom o/
 
it certainly could be a problem, I have no idea whether it is a problem without more infop
and unfortunately I'm about to leave
@StatikStasis yo
fyi everyone I am still alive
10
 
5:06 PM
That's okay
 
@DaveRandom How did you know :(
 
@DaveRandom \o/
 
cmb
+1
 
I will be back properly soon
(tomorrow or day after)
also hey @Tiffany I haven't been ignoring you, or not any more than anyone else :-P
might be back later, tomorrow deffo though
 
5:08 PM
@DaveRandom dork
 
I know you are, you said you are, but what am I?
 
-_-
not enough energy to be sarcastic today
 
(I might have had a beer)
gmfdi I want to go to a nightclub and dance around like a prick less than 2m away from other ppl
at least the pub in the village has starting selling pints again, albeit in plastic glasses and you have to go sit on the field over the road but it's better than nowt
 
now I'm off to write tests, yay
 
@DaveRandom ugh can't wait for stuff to open here again :(
 
5:49 PM
@DaveRandom do you want someone to run a hose over the field for a few hours, so you can recreate that classic glastonbury "what even is a solid surface anyway" vibe?
 
Apparently the LA Times believes PHP is terrible and is encouraging people to stop using it
12
 
@Machavity We should rebecca black them back
And by we I mean you obviously
 
;)
 
@Machavity well, they make some good points.
6
 
6:34 PM
Da hell is happening on GitHub
 
@Danack you seem to have forgotten where I live
 
@Machavity ヽ(ಠ_ಠ)ノ
 
 
1 hour later…
7:59 PM
@brzuchal it turns out I had a left over @generatedValue annotation on another property that was not needed, and generatedvalue annotations affect the class id, regardless of what property they are on. Deleting the bogus one has fixed the problem.
 
8:28 PM
Subject to squashing my commits, is there anything else I should be considering. This PR fixes a regression in functionality.

After this merge I have identified further areas for improvements, reducing scope issues and will make further "cleanup" PR thereafter.

I'm not so keen on doing cleanups and regression fixes in the same ticket.

Let me know. Thanks
s/considering/considering when making a PR ready as a merge candidate. I know the obvious one is squashing commits, is there anything else?
 
Make sure all discussion is resolved
 
You can always mark a PR as a draft worst case
 
8:49 PM
Noted. Thanks for the feedback guys.
 
@Girgias there is no spoon
 
I may need to ask someone to do a build not on WSL
Because it looks like WSL did dump a bazilion false positives
 
I'm on WSL1 and can't run Docker properly, so I'm about to reformat my machine and install latest Wind10 with WSL2.
 
Well I'm on WSL1 too :)
 
8:58 PM
Did you get docker working, inside WSL? (not docker desktop from win10)
 
Nah you can't, and I don't have Win Pro so
WSL2 will fi that because it's a full fledge VM
 
Fair enough. Win 10 Pro - costs £9 by the way, if you want a link to get the license.
 
Well, due to COVID I'm back at my parents and I don't have my desktop
So I'm using my mums laptop
and it's killing me
 
cmb
9:20 PM
@PaulDragoonis it's not strictly necessary to squash commits; the merger can do that as well. :)
 
@cmb thanks for this. I do prefer to squash my commits on this particular PR, since they're just duplicates :) If I was making commits across a few different contexts then perhaps multi-commits would be valuable :-)
 
@cmb yes. Though I like JSON_PRETTY_PRINT also.
 
cmb
Fine, I'll use JSON_PRETTY_PRINT and upload right besides the .html files. :)
 
Cool. I'll say more about the general idea another time....but as I said before, I think one of the things the PHP project has done wrong is just making it difficult to share information. Even just having things like "which votes are going to be opened soon" would be a useful thing to have as a datasource, and then have that info piped to other places (like this room), rather than having the info hidden away.
 
@cmb I may need your help to setup a GitLab CI pipeline for a Windows build to run PVS-Studio on it, as it looks like on the Linux build no issues are found (or you could run a build localy first before I try to go throught the hassle...)
 
cmb
10:01 PM
@Girgias no idea how to use PVS-Studio, nor would I know how to set up a GitLab CU pipeline, but setting up a minimal php-src build environment isn't hard :)
 
@cmb tbh until yesterday I didn't know how to use it either, but they have way more docs about using it on Windows than on Linux
Like I think it can automagically integrate itself into VC2019
You know that I stared that gist, just so that I can try it out when I'm back on my desktop
 
 
1 hour later…
11:05 PM
Presented without further comment
 

« first day (3511 days earlier)      last day (1423 days later) »