On Thu, November 23, 2017 18:13, Jeremie Courreges-Anglas wrote: > On Wed, Nov 22 2017, "Kirill Bychkov" <[email protected]> wrote: >> On Wed, November 22, 2017 18:06, Jeremie Courreges-Anglas wrote: >>> On Wed, Nov 22 2017, "Kirill Bychkov" <[email protected]> wrote: >>>> On Tue, November 21, 2017 19:19, Jeremie Courreges-Anglas wrote: >>>>> On Tue, Nov 21 2017, Sol?ne Rapenne <[email protected]> wrote: >>>>>> Building games/tome4 on powerpc (macppc kernel) fail >>>>>> >>>>>> [-----] >>>>>> ==== Building luajit2 (release) ==== >>>>>> Creating ../obj/Release/luajit2 >>>>>> mkdir -p ../obj/Release/luajit2 >>>>>> Running pre-build commands >>>>>> [---cut----] >>>>>> In file included from ../src/luajit2/src/lj_obj.h:14:0, >>>>>> from ../src/luajit2/src/lj_dispatch.c:9: >>>>>> ../src/luajit2/src/lj_arch.h:321:2: error: #error "No support for >>>>>> little-endian PowerPC" >>>>>> #error "No support for little-endian PowerPC" >>>>>> ^ >>>>>> >>>>>> I propose to remove powerpc as a supported arch >>>>> >>>>> Fair proposal. lang/luajit has a patch to explicitely disable this >>>>> error message, plus another one for the generated asm. >>>>> >>>>> What I don't understand, though, is why luajit thinks that powerpc is >>>>> little-endian: our powerpc architecture is definitely big-endian. >>>>> >>>>> Could you please try to run ''make test'' in lang/luajit on powerpc? >>>> >>>> Hi! >>>> I'm currently trying to build tome4 on macppc. It will take some more time >>>> on my single core G5 :) >>>> But... audio/openal uses COMPILER=base-clang ports-clang ports-gcc and >>>> have >>>> no chance to be built on powerpc because llvm is marked broken on that >>>> arch. >>>> It builds with ports-gcc. >>> >>> Yep, this is not nice. >>> >>> The problem that kills luajit and tome4 is silly: our sys/_endian.h >>> header always defines _LITTLE_ENDIAN and _BIG_ENDIAN. To check >>> endianness, one should check the value of _BYTE_ORDER. >>> >>> (BYTE_ORDER if including <endian.h>) >>> >>> I don't know how upstream ended up with this additional check, but it >>> probably doesn't receive much attention, given that Linux defines >>> __LITTLE_ENDIAN, not _LITTLE_ENDIAN. So who's actually building >>> little-endian powerpc programs? I think it makes sense to just patch >>> out the offending test. >> >> I like the idea. > > Looks like you already have a patch, then. :)
No, U have not :) > >>> Then we can take a look at COMPILER ordering. >>> tome4 is probably not the only affected port... >> >> I'm sure there are plenty of them. > > (See ports/infrastructure/mk/arch-defines.mk rev. 1.46, ports-clang > should now be skipped.) > >> tome4 builds and starts (currently with a black screen) on my G5 with >> both patches taken from lang/luajit. > > Well, if it's unusable, maybe it's not worth building it. Well, still black screen on my mac. Not sure if this a hw issue or tome4. I guess we could mark it as i386 and amd64 only. >
