git: fc75da3a3b2a - main - graphics/libjxl: fix build on powerpc64*

Jan Beich jbeich at FreeBSD.org
Thu Sep 30 19:50:25 UTC 2021


Piotr Kubaj <pkubaj at FreeBSD.org> writes:

>     The include in cpu.cc is not actually necessary, so I did not include anything for FreeBSD.
[...]
> +--- tools/cpu/cpu.cc.orig	2021-09-30 15:38:45 UTC
> ++++ tools/cpu/cpu.cc
> +@@ -19,7 +19,9 @@
> + #endif
> + 
> + #if JXL_ARCH_PPC
> +-#include <sys/platform/ppc.h>  // __ppc_get_timebase_freq
> ++#if linux
> ++#include <sys/platform/ppc.h>  // NOLINT __ppc_get_timebase_freq
> ++#endif
> + #endif

Can you file a bug via https://github.com/libjxl/libjxl/issues/new ?
That include is probably unused even on Linux after 5175d11717f3
removed __ppc_get_timebase_freq.

Note, if you can't sign Google CLA don't submit a fix. Otherwise, upstream
will refuse to read the rationale (PR description), let alone the patch e.g.,
https://github.com/libjxl/libjxl/pull/193


More information about the dev-commits-ports-main mailing list