On Wed, Dec 06 2017, "Kirill Bychkov" <ki...@linklevel.net> wrote: > On Wed, December 6, 2017 16:16, Stuart Henderson wrote: >> On 2017/12/06 15:32, Kirill Bychkov wrote: >>> On Wed, December 6, 2017 14:37, Stuart Henderson wrote: >>> > On 2017/12/06 11:49, Kirill Bychkov wrote: >>> >> On Wed, December 6, 2017 11:34, Jeremie Courreges-Anglas wrote: >>> >> > On Wed, Dec 06 2017, "Kirill Bychkov" <ki...@linklevel.net> wrote: >>> >> >> On Wed, December 6, 2017 03:23, Jeremie Courreges-Anglas wrote: >>> >> >>> On Sun, Dec 03 2017, "Kirill Bychkov" <ki...@linklevel.net> wrote: >>> >> >>>> Hi! >>> >> >>>> This patch enables build of libraw on other gcc4 arches, not only >>> arm. >>> >> >>>> Tested on macppc. >>> >> >>>> OK? >>> >> >>> >>> >> >>> This looks heavy-handed to me, why extend this to all non-clang >>> archs, >>> >> >>> afaik base-gcc has support for 4-bytes atomics on powerpc. How does >>> the >>> >> >>> build fail exactly? >>> >> >> >>> >> >> Without patch I see >>> >> >> ===> libraw-0.18.5 is only for aarch64 amd64 i386 arm, not powerpc >>> >> >> (macppc) . >>> >> >> MODGCC4_ARCHS = arm somehow overrides ONLY_FOR_ARCHES: >>> >> >> >>> >> >> make show=ONLY_FOR_ARCHS >>> >> >> aarch64 amd64 i386 arm >>> >> >> >>> >> >> With patch: >>> >> >> make show=ONLY_FOR_ARCHS >>> >> >> aarch64 amd64 i386 amd64 arm hppa i386 mips64 mips64el powerpc sparc64 >>> >> >> >>> >> >> Switching MODULES=gcc4 to COMPILER=gcc made libraw unavailable on most >>> >> >> arches. >>> >> >> See >>> >> >> http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/graphics/libraw/Makefile.diff?r1=1.24&r2=1.25 >>> >> > >>> >> > ok, thanks for confirming. >>> >> > >>> >> >> Better patch: >>> >> >> >>> >> >> Index: Makefile >>> >> >> =================================================================== >>> >> >> RCS file: /cvs/ports/graphics/libraw/Makefile,v >>> >> >> retrieving revision 1.29 >>> >> >> diff -u -p -u -r1.29 Makefile >>> >> >> --- Makefile 16 Nov 2017 23:20:39 -0000 1.29 >>> >> >> +++ Makefile 6 Dec 2017 08:13:27 -0000 >>> >> >> @@ -22,9 +22,10 @@ WANTLIB += c jasper jpeg lcms2 m pthread >>> >> >> >>> >> >> MASTER_SITES = https://www.libraw.org/data/ >>> >> >> >>> >> >> -COMPILER = base-clang ports-gcc >>> >> >> # for atomic builtins (__sync_fetch_and_add_4) >>> >> >> -MODGCC4_ARCHS = arm >>> >> >> +.if ${MACHINE_ARCH} == "arm" >>> >> >> +COMPILER = base-clang ports-gcc >>> >> >> +.endif >>> >> > >>> >> > I guess it's fine, but isn't the shortest fix to add "base-gcc" at the >>> >> > end of COMPILER? >>> >> >>> >> macppc is quite happy with base-gcc, so it should be before ports-gcc. >>> And >>> >> for arm ports-gcc is the only solution I suppose (have no hw to test). >>> >> So I see no other way to deal with arm other than this. >>> >> >>> >> >>> >> >>> > >>> > I think COMPILER should be removed here, seems it should probably >>> > use this instead (untested): >>> > >>> > # XXX remove when armv7 switches to clang in base? >>> > MODULES= gcc4 >>> > MODGCC4_ARCHS= arm >>> > MODGCC4_LANGS= c++ >>> > >>> > >>> Missed that point that we could simply return back to MODULES=gcc instead >>> of COMPILER. Works for me on macppc and amd64. >>> >> >> Is it worth trying that on hppa as well? >> >> OK? > > OK.
ok here too, thanks for taking care of this. -- jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE