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-all
mailing list