php73-apache2handler @7.3.16_1: error: too few arguments to function call, expected 3, have 2

[murrayE]

There are _many_ errors reported in main.log for building. the log is attached.

(Sorry, I don't understand enough of what's involved to pin this down further.)

[murrayE]

main.log

[ryandesign (Ryan Carsten Schmidt)]

Hmm, the first error is:

:info:build /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_lang_php/php73-apache2handler/work/php-7.3.16/main/reentrancy.c:125:23: error: too few arguments to function call, expected 3, have 2
:info:build         readdir_r(dirp, entry);
:info:build         ~~~~~~~~~            ^
:info:build /Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/dirent.h:110:1: note: 'readdir_r' declared here
:info:build int readdir_r(DIR *, struct dirent *, struct dirent **) __DARWIN_INODE64(readdir_r);
:info:build ^
:info:build 1 error generated.

I have not yet tried to look into what's going on here.

We were able to build this successfully on our build servers for all OS versions, so whatever's happening is not a general problem but is specific to your system.

[ryandesign (Ryan Carsten Schmidt)]

Several differences between the configure output from your log and the ​build server log:

Ours:

checking for broken libc stdio... no
checking for getaddrinfo... yes
checking for type of readdir_r... POSIX
checking for asm goto... no
checking whether pwrite works... yes
checking whether pread works... yes
checking for standard DES crypt... yes
checking for extended DES crypt... yes
checking whether sprintf is broken... no
checking how many arguments gethostbyname_r() takes... cannot find function declaration in netdb.h

Yours:

checking for broken libc stdio... yes
checking for getaddrinfo... no
checking for type of readdir_r... old-style
checking for asm goto... yes
checking whether pwrite works... no
checking whether pread works... no
checking for standard DES crypt... no
checking for extended DES crypt... no
checking whether sprintf is broken... yes
checking how many arguments gethostbyname_r() takes... can't tell

The config.log file might tell us more details about why these various checks returned different and I suspect incorrect results on your system. Could you attach it? It will be in the directory /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_lang_php/php73-apache2handler/work/php-7.3.16/

[murrayE]

config.log

[ryandesign (Ryan Carsten Schmidt)]

The config.log shows the tests are failing because of implicitly declared functions, for example the "broken libc stdio" test:

conftest.c:103:5: error: implicitly declaring library function 'exit' with type 'void (int) __attribute__((noreturn))' [-Werror,-Wimplicit-function-declaration]
    exit(2);
    ^
conftest.c:103:5: note: include the header <stdlib.h> or explicitly provide a declaration for 'exit'
conftest.c:111:3: error: implicit declaration of function 'unlink' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  unlink(filename);
  ^

I know that implicitly declared functions will be errors in Xcode 12 and later, so we definitely need to fix them, I just wasn't expecting them to show up in Xcode 11 already, and I can't explain why you're seeing these errors with Xcode 11.6 on macOS 10.15 when we did not on the same system.

Maybe we have different versions of the command line tools. On our server we have:

% pkgutil --pkg-info=com.apple.pkg.CLTools_Executables | grep version:
version: 11.5.0.0.1.1588476445

What version do you have?

[murrayE]

pkgutil --pkg-info=com.apple.pkg.CLTools_Executables | grep version

version: 12.0.0.0.1.1595091463

There were no CLTs for Xcode 11.6, only for Xcode 11.5 and Xcode 12 beta.

[ryandesign (Ryan Carsten Schmidt)]

That explains it. So to avoid this problem for now, please install the Xcode 11.5 command line tools. Using Xcode 12 or its command line tools will cause you to run into this problem with lots of ports.

[murrayE]

I deleted Xcode 11.6, re-downloaded to re-nstall it. But that did *not* get rid of the 12.0 CLT: The pkgutil command still shows version: 12.0.0.0.1.1595091463.

[murrayE]

Fixed CLT by downloading CLT 11.5 from developer site and installing. That automatically changed active version:

$ pkgutil --pkg-info=com.apple.pkg.CLTools_Executables | grep version
version: 11.5.0.0.1.1588476445

I'm making this comment because there seems to be a lot of obsolete information available about dealing with installing CLTs. E.g., some sources say one can change the version from within the Xcode app's Preferences, but I don't see any such place in Xcode 11.6.

That solved the immediate issue: php73-apache2handler upgraded without error (as did inkscape).

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #61168.

[ryandesign (Ryan Carsten Schmidt)]

Filed upstream bug report: ​https://bugs.php.net/bug.php?id=80176

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #61364.

The developers have committed a fix which should be in the next version of php73 but I haven't verified the fix yet.

[ryandesign (Ryan Carsten Schmidt)]

In 04a5fc5efac6611dce738d39b2fe659f37d48e4e/macports-ports (master):

php: Update php73 to 7.3.24

See: #60988

[ryandesign (Ryan Carsten Schmidt)]

In ffa632a3b93568fb316bf08530639cdeda6358f3/macports-ports (master):

php: Update php74 to 7.4.12

See: #60988

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #61517.

[ryandesign (Ryan Carsten Schmidt)]

In b3e55a05aa4ad86271d4f21dc6634fb8eda7c27a/macports-ports (master):

php72 and older: incompatible with Xcode 12+

See: #60988

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #61844.

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #61900.

If you use Xcode 12 or later, you'll need to use php73 or later for now.

If you want to use php72 or earlier, you'll need to use Xcode 11.x or earlier for now, which means using macOS Catalina 10.15.x or earlier.

[ryandesign (Ryan Carsten Schmidt)]

In 08cd4de291ce854357a05e4f2fdba4267a405b6a/macports-ports (master):

php70, php71, php72: Fix implicit function decls

The modified m4 files get installed, so the revision is increased.

See: #60988

[ryandesign (Ryan Carsten Schmidt)]

In 59ecb2c7041ba6287606f5fa440135a24e223f15/macports-ports (master):

php-1.1.tcl: php70 and later are fixed on Xcode 12

See: #60988

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #62888 reminding me I still need to backport the fixes to php56 and earlier.

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #63225.