The required fix has already committed to the repository[1], but a
new version has not been released since then.

[1] <http://svn.php.net/viewvc?view=revision&revision=341642>

So, can we please get a new version out?
This is the last ext that holding me down on 7.1.

Please?…

https://windows.php.net/downloads/pecl/releases/rrd/2.0.1/logs/

2.0.2 for 7.4+ did not build

https://windows.php.net/downloads/pecl/releases/rrd/2.0.2/logs/

Latest source is on https://github.com/php/pecl-processing-rrd/. It was migrated from svn, so I think patches from Christoph are included. 2.0.2 was released some days ago.

Christoph can you look at why windows rrd build for php 7.4+ doesn't work?

I'm changing the title - missing windows builds are for 7.4+ and assigning to Christoph for verifying. See my previous comment from 2021-04-22 10:44. Thanks.

See the linked pull request.

thanks cmb@php.net - I merged the pull request, there are some build warnings on linux


rrd.h:197:5: note: expected ‘long unsigned int *’ but argument is of type ‘zend_ulong *’ {aka ‘unsigned int *’}
     unsigned long *,
     ^~~~~~~~~~~~~~~

rrd.c: In function ‘zif_rrd_xport’:
rrd.c:418:70: warning: passing argument 6 of ‘rrd_xport’ from incompatible pointer type [-Wincompatible-pointer-types]
  if (rrd_xport(argv->count-1, &argv->args[1], &xxsize, &start, &end, &step,
                                                                      ^~~~~
rrd.c:19:
/usr/include/rrd.h:229:5: note: expected ‘long unsigned int *’ but argument is of type ‘zend_ulong *’ {aka ‘unsigned int *’}
     unsigned long *,
     ^~~~~~~~~~~~~~~

rrd.c:419:3: warning: passing argument 7 of ‘rrd_xport’ from incompatible pointer type [-Wincompatible-pointer-types]
   &outvar_count, &legend_v, &data) == -1) {
   ^~~~~~~~~~~~~


but if it's good for windows builds, go on, maybe some another fix these if necessary :-).

As I've seen, the 7.3 build is also full of similar warnings, but succeed.

All newly packaged archives are missing gobject-2.dll

I see there are windows builds already - https://pecl.php.net/package/rrd/2.0.3/windows. So I think bug is fixed and missing dll is another problem. But OK, I'm trying to change back to cmb.

Christoph, can you look at reported missing gobject-2.dll in latest generated packages on https://pecl.php.net/package/rrd/2.0.3/windows?

Oh, this turns out to be ugly.  Apparently, the build system[1]
isn't capable of adding the gobject-2.dll, so this has been done
manually in the past (rrd 1.1.3).  I did that now for rrd 2.0.3 as
well, but it would be good to have that automated.

[1] <https://github.com/php/web-rmtools>

> With a cherry on top.

Guess that was the manual adding of gobject-2.dll. :)

Thanks, I'm on PHP 7.4 now.

Have to wait for IMagick for next step, but that's a different problem.

Just a small note: The 7.3 and 7.4 packages (I checked TS ones) have gmodule-2.dll instead of gobject-2.dll

It can be retrieved from relevant 7.1 package, if anybody is stumbled upon this bug in the meantime.

> The 7.3 and 7.4 packages (I checked TS ones) have gmodule-2.dll
> instead of gobject-2.dll

Oh, right!  That should be fixed now.

> […], but it would be good to have that automated.

Obviously.