« first day (3860 days earlier)      last day (1073 days later) » 
00:00 - 20:0020:00 - 00:00

8:00 PM
@NikiC to be more precise: -DAVIF_DLL -I/usr/local/include
 
@JoeWatkins Test added to the branch.
 
@FlávioHeleno I believe that indicates an issue on the pkg-config side. pkg-config is supposed to strip out system include paths (and libdirs) if PKG_CONFIG_ALLOW_SYSTEM_CFLAGS is not set
Though possibly it strips only /usr/include and not /usr/local/include?
 
@NikiC could be the case. the workaround right now is to install libavif on a different prefix
 
Yeah, looks like /usr/include gets stripped, but /usr/local/include doesn't, despite both being on the default search path
 
@NikiC The latter is not always on the default default search path. I believe RHEL and CentOS do not have it by default, for instance.
 
8:11 PM
we're running the build in macOS
not sure how to check the default search path though
 
8:45 PM
Someone over here with more experience than me in autotools is saying that it's possible to force the system to find gd.h locally, by including an -l flag. I don't know where that would appear, though
 
9:00 PM
@BenMorss just the flag?
 
cmb
9:14 PM
unlikely, since I mostly work on Windows. :)
Anyhow, the discussion went on, and the general problem that libavif, libjpeg etc. must not be installed in the same directory as gd has been identied.
Simple solution: use Windows /runs/
 
That's generally solid advice IMO
 
@cmb use Linux /var/runs
 
@FlávioHeleno he's remained mysterious about where this flag would be deployed
He's one of the core maintainers of libgd and he's good at autotools :)
 
cmb
@BenMorss -l is for including libs; unrelated to .h`
 
curses!
 
9:20 PM
@BenMorss no I don't think we have native support for that
 
@cmb I just noticed you've re-entered the room
 
-I specifies include directories on the commandline, you probably want something like AC_INCLUDE_PATH() tho
 
cmb
I'm not good at autotools; actually M4 gives me the creeps
 
@cmb more or less than the bastardised m4-like JScript?
 
@BenMorss Doesn't work on Windows
 
9:22 PM
Time for home... later!
 
Oh damn :(
 
cmb
@Sara there is pkg-config involved; it adds -I /usr/local/include, and there is an external gd.h, which is then included instead of the bundled gd.h
 
such a niche joke as well btw, I doubt most ppl under 30 have even heard of curses in that sense, prob a lot over 30 as well :-P
 
9:24 PM
heh... I just got it
 
when you have to explain tech jokes... those aren't jokes by any reasonable metric
 
I'm going to walk the dog, but will follow this discussion on my phone...
 
lol the adobe twitter social media person really is basically goading me into being rude to them :-P
(I am not gonna do it)
 
cmb
@DaveRandom actually, I prefer that (at least I understand the primitives of JScript) :)
 
@cmb ditto
I just have an urge to put semi-colons in is the main problem :-P
 
cmb
9:28 PM
use semicolons!
 
yeh but if you don't then sometimes you can legit copypasta between m4/js
 
.... ooooohh, I've unlocked a new git error message!
> fatal: bad tree object 00e9c9d22abfcba78dabe51e9f57361073d87b34
broken link from    tree d6abc93ea8e11d4a7324035856637a7209d03a97
              to    tree 00e9c9d22abfcba78dabe51e9f57361073d87b34
broken link from    tree c74980bba92337efeb4e40055d7014daff9fdffd
              to    tree 0021665212c9e7c2e1a1e91094b560802b4e4eb1
 
rename your current directory and do a fresh check out...
btw is this on windows with WSL?
 
9:45 PM
I'm trying to evaluate the difference in work load between doing that along with copying manually the content (sizeable) of the local branch I would like to keep, and trying to unlock my first git tree fix operation
 
it might be easier to figure out the syntax for exporting a patch, and applying it to the fresh checkout.
if it's not too borken to generate the patch.
 
@Danack yes I think I've twiddled with wsl at some point, I can access the ls command in powershell but not in the terminal from which I do these commands
 
cmb
@BenMorss ah, I think the problem is that three files in bundled libgd have #include <gd.h>; that should be #include "gd.h"
 
Yeah, I believe it is Truly Borken, the remote is downright refusing any kind of communication from that branch. I would have copied stuff around like in 1990
 
@cmb I was wondering whether I should try that...
 
cmb
9:48 PM
better get a smart phone ;)
 
I just figured it was in angle brackets for a reason
 
cmb
the reason was likely copypasta
 
lol
 
cmb
point is that #include "foo" first looks into same dir, but #include <foo> does not
please check it out; looks like bug we may want to fix for PHP-7.4+
 
9:52 PM
@FélixGagnon-Grenier yeah.
 
cmb
and of course, we need to get rid of bundled libgd ASAP; github.com/libgd/libgd/pull/692 might be the missing piece
@Crell, re new namespaces: see github.com/php/php-src/pull/6970 et al.
 
this deserved more than it got, and I got no problem with whoring it out twitter.com/_DaveRandom/status/1392182901668814850
 
cmb
they're still running PHP 7
 
10:08 PM
either that or they are free()ing sugar wrapped around the product
 
@cmb awesome!
 
@Danack (not that you asked, I'm feeling chatty as usual :-D) it appears I was quite thorough in corrupting the tree, format-patch wants nothing to do with reading it!
 
@cmb I'm checking if the gd related includes should be using <> or "" (cc @BenMorss)
 
cmb
ta
 
@FélixGagnon-Grenier maybe then if you don't care about history, get a fresh checkout, and use the compare directories tool in PHPStorm, to compare the directories, and you can also bring changes across.
 
10:22 PM
hmmm... yeah it may very well be possible that this is my only option... I'll try and pursue the gory way a bit before, I might get lucky hackernoon.com/…
 
lolflags
 
@cmb I've been following that PR with great interest! I figured I should still work on porting over the AVIF support now, since with luck that could get into usage in November
Though now I've seen there's plenty more to do to get a new function into PHP anyway :)
 
@cmb yay
 
is there a way to run only gd tests from source code root path?
 
Ok, I think changing the quotes to angle brackets works
I had to just change all the gd include files - the ones of the form gdfont*.h
 
10:33 PM
I did from angle brackets to quotes and it did not work for me, got stuck at the same point
 
DirectoryIterator show duplicate listing on mergerfs ・ *Directory/Filesystem functions ・ #81031
 
I got stuck with errors in my code, which is wonderful, because now I can fix those
right now, things I have to import into PHP's gd.h
 
@BenMorss awesome! :-D
 
Thanks for looking at this @FlávioHeleno!
 
cmb
@BenMorss yes, that makes perfect sense; even if GD would support custom allocators tomorrow, we'd still need to wait for widespread adoption.
And I now realize, that that PR doesn't play nicely with PHP debug mode, since in that case _emalloc() an friends also expect filename and line number for diagnostic purposes. No idea how to retain that info.
 
10:42 PM
@BenMorss got it working here by replacing the #include <gd*.h> with #include "libgd/gd*.h"
 
that's what I did too
great minds think alike
 
:D
it's probably worth fixing it in master I guess? (@cmb)
 
@cmb No idea here either, as I really haven't done much C in a long time. I know there were macros in gd that included filenames and line numbers, but I imagine the problem you're facing is deeper
 
if there are any mods floating about can you sort the flagging sitch out?
like 10 invalid flags in the last ~30mins
 
cmb
@FlávioHeleno ah, right! Please consider to file a bug report at bugs.php.net. This is not particularly related to libavif, but would likely happen with e.g. libjpeg headers installed in the same dir as system libgd headers.
 
10:53 PM
@cmb for sure! :D
 
11:05 PM
I'm propagating these include changes to other files - like gd_wbmp.c and gd_interpolation.c.
oddly gd_wbmp.c still uses gd_error()
wow, it builds now!

Wonder if I should change those `gd_error()`'s to `zend_error()` while I'm at it
heh, never mind; this is a project for another PR

Anyway, I've pushed the new files with new includes to my branch
 
11:21 PM
@BenMorss as much as I generally for macro normalisation across the board, error handlers like that tend to be a proxy in favour of hook for readability, so probably leave it
if the proxy does nothing other than call the underlying routines then kill them tho :-)
 
11:52 PM
GD install is affected by external libgd installation ・ GD related ・ #81032
 
@Jeeves cc @cmb, @BenMorss
I've submitted a patch with the bug report, not sure if I should do anything else..
 
cmb
@FlávioHeleno a PR is much more preferable to a patch :)
 
00:00 - 20:0020:00 - 00:00

« first day (3860 days earlier)      last day (1073 days later) »