On Tue, 9 Aug 2022 at 23:22, Philippe Mathieu-Daudé via <qemu-devel@nongnu.org> wrote: > > Commit 06680b15b4 moved qemu_*_exec_dir() to cutils but forgot > to move the macOS dyld(3) include, resulting in the following > error (when building with Homebrew GCC on macOS Monterey 12.4): > > [313/1197] Compiling C object libqemuutil.a.p/util_cutils.c.o > FAILED: libqemuutil.a.p/util_cutils.c.o > ../../util/cutils.c:1039:13: error: implicit declaration of function > '_NSGetExecutablePath' [-Werror=implicit-function-declaration] > 1039 | if (_NSGetExecutablePath(fpath, &len) == 0) { > | ^~~~~~~~~~~~~~~~~~~~ > ../../util/cutils.c:1039:13: error: nested extern declaration of > '_NSGetExecutablePath' [-Werror=nested-externs] > > Fix by moving the include line to cutils. > > Fixes: 06680b15b4 ("include: move qemu_*_exec_dir() to cutils") > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > Cc: Marc-André Lureau <marcandre.lur...@redhat.com> > Cc: Markus Armbruster <arm...@redhat.com>
I wonder why this doesn't show up with clang? Anyway, obvious fix. I'll take it via target-arm.next for my pull for rc3, unless anybody has a different preference. thanks -- PMM