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

Piotr Kubaj pkubaj at anongoth.pl
Thu Sep 30 21:23:52 UTC 2021


No, I don't have a Google account.

On 21-09-30 21:50:21, Jan Beich wrote:
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/dev-commits-ports-main/attachments/20210930/a68df064/attachment.sig>


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