From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from sandeen.net ([63.231.237.45]:47448 "EHLO sandeen.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758663AbcLUVdN (ORCPT ); Wed, 21 Dec 2016 16:33:13 -0500 Subject: Re: [PATCH v3] Fix building xfsprogs on 32-bit platforms References: <20161221190413.GA136777@gmail.com> From: Eric Sandeen Message-ID: <1a75e88c-014c-b245-2271-f7d201da1285@sandeen.net> Date: Wed, 21 Dec 2016 15:33:11 -0600 MIME-Version: 1.0 In-Reply-To: <20161221190413.GA136777@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Eric Biggers , linux-xfs@vger.kernel.org Cc: Dave Chinner , Eric Sandeen , Eric Biggers , Felix Janda On 12/21/16 1:04 PM, Eric Biggers wrote: >> xfslibs now requires that its users enable transparent largefile >> support. This broke building xfsprogs on 32-bit Linux (with glibc) >> because _FILE_OFFSET_BITS=64 was not getting defined. Although the >> autoconf macro AC_SYS_LARGEFILE was intended to define it, this didn't >> work because AC_SYS_LARGEFILE will only define _FILE_OFFSET_BITS in a >> config header, which doesn't work for xfsprogs because not all .c files >> include platform_defs.h as their first include. Also, >> platform_defs.h.in is not generated by autoheader and didn't contain a >> template for _FILE_OFFSET_BITS. >> >> Therefore, to fix the problem remove the useless autoconf macros and >> instead add -D_FILE_OFFSET_BITS=64 to CFLAGS in builddefs.in. Use >> CFLAGS rather than PCFLAGS because this definition could be needed by >> platforms other than "linux", and it doesn't hurt to always define it. >> >> Signed-off-by: Eric Biggers > > Hi, > > Is this patch planned to be applied? Building 32-bit Linux binaries of xfsprogs > is still broken on the master branch, and there's nothing in for-next. TBH I haven't really tested this on a 32-bit system yet, I'm sorry. Right now, I plan to get it into the tree right after we cut the next release this week. -Eric > Eric > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >