Created attachment 200938 [details] Patch adding powerpc and powerpcspe to libreoffice Only minor additions are needed on top of the powerpc64 patch to enable building for powerpc and powerpcspe. Test-built on powerpcspe. The files/patch-powerpc64 can/should probably be renamed to patch-powerpc since it now covers all 3 powerpc architectures.
It turns out gcc already defines __NO_FPRS__ for powerpcspe, so that part of the patch is redundant.
A commit references this bug: Author: lwhsu Date: Tue Feb 5 17:46:04 UTC 2019 New revision: 492260 URL: https://svnweb.freebsd.org/changeset/ports/492260 Log: Add support for powerpc and powerpcspe PR: 234776 Submitted by: jhibbits Changes: head/editors/libreoffice/Makefile head/editors/libreoffice/files/patch-powerpc head/editors/libreoffice/files/patch-powerpc64
(In reply to Justin Hibbits from comment #1) Do you mean this section in Makefile? CXXFLAGS_powerpcspe= -D__NO_FPRS Just wanted to make sure, gcc defines __NO_FPRS__ or __NO_FPRS ?
GCC defines __NO_FPRS__, and that addition was a typo (I had meant __NO_FPRS__ in that define, so that just goes to show that since it builds fine on powerpcspe anyway, it's unnecessary). That Makefile change was unnecessary, and I should've removed it.
A commit references this bug: Author: lwhsu Date: Tue Feb 5 18:04:34 UTC 2019 New revision: 492263 URL: https://svnweb.freebsd.org/changeset/ports/492263 Log: Remove unnecessary CXXFLAGS for powerpcspe gcc already defineds the correct one. PR: 234776 Submitted by: jhibbits Changes: head/editors/libreoffice/Makefile