From mboxrd@z Thu Jan 1 00:00:00 1970 From: Danny Milosavljevic Subject: Re: [PATCH] gnu: Add optipng. Date: Mon, 26 Sep 2016 21:34:25 +0200 Message-ID: <20160926213425.2d7fcfe8@scratchpost.org> References: <87lgyglzmv.fsf@openmailbox.org> <20160925202832.17166-1-arunisaac@systemreboot.net> <20160925202832.17166-2-arunisaac@systemreboot.net> <87zimvlmj4.fsf@openmailbox.org> <20160926175958.GA5857@jasmine> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:49879) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bobmJ-0002MA-6C for guix-devel@gnu.org; Mon, 26 Sep 2016 15:41:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bobmG-0001A4-0n for guix-devel@gnu.org; Mon, 26 Sep 2016 15:41:27 -0400 Received: from dd1012.kasserver.com ([85.13.128.8]:41729) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bobmF-00019x-QP for guix-devel@gnu.org; Mon, 26 Sep 2016 15:41:23 -0400 In-Reply-To: <20160926175958.GA5857@jasmine> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: Leo Famulari Cc: guix-devel@gnu.org Hi Leo, On Mon, 26 Sep 2016 13:59:58 -0400 Leo Famulari wrote: > gcc -s -o optipng optipng.o optim.o bitset.o ratio.o osys.o wildargs.o ../opngreduc/libopngreduc.a ../pngxtern/libpngxtern.a ../libpng/libpng.a ../zlib/libz.a ../gifread/libgifread.a ../pnmio/libpnmio.a ../minitiff/libminitiff.a -lm > ../libpng/libpng.a(pngrutil.o): In function `png_read_filter_row': > pngrutil.c:(.text+0x13bc): undefined reference to `png_init_filter_functions_neon' > collect2: error: ld returned 1 exit status > make[1]: *** [Makefile:99: optipng] Error 1 > make[1]: Leaving directory '/tmp/nix-build-optipng-0.7.6.drv-0/optipng-0.7.6/src/optipng' > make: *** [Makefile:2: all] Error 2 Looks like a problem internal to libpng. There's a configure option '--enable-arm-neon=api' in libpng. If we supported only ARM with NEON SIMD extension then we could build libpng with it. Do we? Debian says on their https://wiki.debian.org/ArmHardFloatPort#NEON that one shouldn't require NEON since Marvell Dove and NVidia Tegra2 (?!?!?!) don't implement it. There's also a configure option '--enable-arm-neon=check' which would check whether NEON works at runtime - but according to configure, it's not well-supported. What does this mean?