From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28892 invoked by alias); 28 Jan 2011 21:12:49 -0000 Received: (qmail 28869 invoked by uid 22791); 28 Jan 2011 21:12:48 -0000 X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from g5t0006.atlanta.hp.com (HELO g5t0006.atlanta.hp.com) (15.192.0.43) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 28 Jan 2011 21:12:44 +0000 Received: from g5t0029.atlanta.hp.com (g5t0029.atlanta.hp.com [16.228.8.141]) by g5t0006.atlanta.hp.com (Postfix) with ESMTP id D5E3DC383; Fri, 28 Jan 2011 21:12:42 +0000 (UTC) Received: from [16.89.92.85] (hpsje.cup.hp.com [16.89.92.85]) by g5t0029.atlanta.hp.com (Postfix) with ESMTP id C1A4D2030D; Fri, 28 Jan 2011 21:12:41 +0000 (UTC) Subject: Re: PATCH: import move-if-change from gnulib (was: PATCH: PR binutils/12283: bfd/doc doesn't support parallel build) From: Steve Ellcey Reply-To: sje@cup.hp.com To: Ralf Wildenhues Cc: gcc-patches@gcc.gnu.org, Gerald Pfeifer , binutils@sourceware.org, "H.J. Lu" In-Reply-To: <20110128201452.GM7234@gmx.de> References: <201101272357.p0RNvPZ13996@lucas.cup.hp.com> <20110128201452.GM7234@gmx.de> Content-Type: text/plain; charset="UTF-8" Date: Fri, 28 Jan 2011 21:12:00 -0000 Message-ID: <1296249160.12233.50.camel@hpsje.cup.hp.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org X-SW-Source: 2011-01/txt/msg00377.txt.bz2 On Fri, 2011-01-28 at 21:14 +0100, Ralf Wildenhues wrote: > > I have meanwhile added an improved and fixed version of my proposed > patch to the (more complete) gnulib copy of move-if-change. See here: > http://thread.gmane.org/gmane.comp.lib.gnulib.bugs/24860/focus=24865 > > I hereby propose to just import the gnulib copy into GCC and src, > http://git.savannah.gnu.org/cgit/gnulib.git/tree/build-aux/move-if-change > and document the change in status as below in patches to src/MAINTAINERS > and GCC www. (The Autoconf copy of move-if-change has been updated from > the gnulib one anyway.) > > Any reasons against doing so? > > Thanks, > Ralf I have no objection to this change and I think it will fix things on most platforms but I have found one platform where I still have a problem. On hppa*-*-hpux*, compiling the same program twice in a row does not result in the exact same executable. So even with this change the cmp will fail and the build will still abort like it does now. During a GCC bootstrap build we work around these issues by using: cmp --ignore-initial=16 to compare stage2 and stage3 bits. And we also have to do the GCC compilation with -frandom-seed=fixed-seed to avoid other differences in object files. But for some reason doing both of these still didn't get this to work for me on the hppa platform. I was still getting failures in the cmp check, even with the --ignore-initial flag and the use of -frandom-seed when compiling chew. I wonder if it would be reasonable to add a configure flag to bfd that would enforce the chew dependency (--enable-parallel-doc or something like that). It would be off by default but if specified then we would enforce the chew dependency and parallel builds would just work normally.