X-Original-To: alpine-aports@lists.alpinelinux.org Received: from mx1.tetrasec.net (mx1.tetrasec.net [74.117.190.25]) by lists.alpinelinux.org (Postfix) with ESMTP id E39B95C57BB for ; Mon, 18 Dec 2017 15:51:03 +0000 (GMT) Received: from mx1.tetrasec.net (mail.local [127.0.0.1]) by mx1.tetrasec.net (Postfix) with ESMTP id A95659E0EFF; Mon, 18 Dec 2017 15:51:03 +0000 (GMT) Received: from ncopa-desktop.copa.dup.pw (15.63.200.37.customer.cdi.no [37.200.63.15]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by mx1.tetrasec.net (Postfix) with ESMTPSA id E6ECA9E0425; Mon, 18 Dec 2017 15:51:02 +0000 (GMT) Date: Mon, 18 Dec 2017 16:50:58 +0100 From: Natanael Copa To: Nils Andreas Svee Cc: alpine-aports@lists.alpinelinux.org Subject: Re: [alpine-aports] [PATCH 7/7] main/libffi: add linux-headers dependency on mips archs Message-ID: <20171218165058.02e719a3@ncopa-desktop.copa.dup.pw> In-Reply-To: <20171210001814.28762-7-me@lochnair.net> References: <20171210001814.28762-1-me@lochnair.net> <20171210001814.28762-7-me@lochnair.net> X-Mailer: Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-alpine-linux-musl) X-Mailinglist: alpine-aports Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Sun, 10 Dec 2017 01:18:14 +0100 Nils Andreas Svee wrote: > libffi depends on the asm/sgidefs.h kernel header, so if we're compiling on MIPS, > add linux-headers to makedepends_host > --- > main/libffi/APKBUILD | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/main/libffi/APKBUILD b/main/libffi/APKBUILD > index 36ce7085f8..9ba93bcd2e 100644 > --- a/main/libffi/APKBUILD > +++ b/main/libffi/APKBUILD > @@ -15,6 +15,10 @@ source="ftp://sourceware.org/pub/$pkgname/$pkgname-$pkgver.tar.gz > pax-dlmmap.patch > " > > +case "$CTARGET_ARCH" in > +mips*) makedepends_host="linux-headers" ;; > +esac > + > builddir="$srcdir"/$pkgname-$pkgver > > build () { I think we can add linux-headers for all architectures, with a comment that its specifically needed for mips, so we don't need the conditional. -nc --- Unsubscribe: alpine-aports+unsubscribe@lists.alpinelinux.org Help: alpine-aports+help@lists.alpinelinux.org ---