GNU bug report logs - #6398
[PATCH] build: don't let a large sparse temporary file cause "make dist" to fail

Previous Next

Package: coreutils;

Reported by: Jim Meyering <jim <at> meyering.net>

Date: Thu, 10 Jun 2010 21:40:03 UTC

Severity: normal

Tags: patch

Done: Jim Meyering <jim <at> meyering.net>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 6398 in the body.
You can then email your comments to 6398 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to owner <at> debbugs.gnu.org, bug-coreutils <at> gnu.org:
bug#6398; Package coreutils. (Thu, 10 Jun 2010 21:40:03 GMT) Full text and rfc822 format available.

Acknowledgement sent to Jim Meyering <jim <at> meyering.net>:
New bug report received and forwarded. Copy sent to bug-coreutils <at> gnu.org. (Thu, 10 Jun 2010 21:40:03 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: bug-coreutils <at> gnu.org
Subject: [PATCH] build: don't let a large sparse temporary file cause "make
	dist" to fail
Date: Thu, 10 Jun 2010 23:38:57 +0200
I had this problem:

  http://thread.gmane.org/gmane.comp.gnu.utils.bugs/16929

Here's a proposed work-around, while we wait for an upstream
change that fixes Makefile.in.in or at least lets us configure
the offending rule.

From 1f85bf38919791753c9a931ad05badd0e0274a5c Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering <at> redhat.com>
Date: Thu, 10 Jun 2010 23:33:17 +0200
Subject: [PATCH] build: don't let a large sparse temporary file cause "make dist" to fail

* bootstrap.conf (bootstrap_epilogue): Replace the offending grep
command from po/Makefile.in.in's $(DOMAIN).pot-update rule.
---
 bootstrap.conf |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/bootstrap.conf b/bootstrap.conf
index 576d308..6e85c9a 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -337,4 +337,15 @@ bootstrap_epilogue()
   m=gnulib-tests/gnulib.mk
   sed 's,\.\./\.\.,..,g' $m > $m-t
   mv -f $m-t $m
+
+  # Since this is a "GNU" package, replace this line
+  #   if LC_ALL=C grep 'GNU @PACKAGE@' $(top_srcdir)/* 2>/dev/null \
+  #      | grep -v 'libtool:' >/dev/null; then
+  # with this:
+  #   if true; then
+  # Why?  That pipeline searches all files in $(top_srcdir), and if you
+  # happen to have large files (or apparently large sparse files), the
+  # first grep may well run out of memory.
+  perl -pi -e 's/if LC_ALL=C grep .GNU .PACKAGE.*; then/if true; then/' \
+    po/Makefile.in.in
 }
--
1.7.1.501.g23b46




Reply sent to Jim Meyering <jim <at> meyering.net>:
You have taken responsibility. (Fri, 11 Jun 2010 05:47:02 GMT) Full text and rfc822 format available.

Notification sent to Jim Meyering <jim <at> meyering.net>:
bug acknowledged by developer. (Fri, 11 Jun 2010 05:47:02 GMT) Full text and rfc822 format available.

Message #10 received at 6398-done <at> debbugs.gnu.org (full text, mbox):

From: Jim Meyering <jim <at> meyering.net>
To: 6398-done <at> debbugs.gnu.org
Subject: Re: bug#6398: [PATCH] build: don't let a large sparse temporary file
	cause "make dist" to fail
Date: Fri, 11 Jun 2010 07:46:23 +0200
Jim Meyering wrote:
> Subject: [PATCH] build: don't let a large sparse temporary file cause "make dist" to fail
>
> * bootstrap.conf (bootstrap_epilogue): Replace the offending grep
> command from po/Makefile.in.in's $(DOMAIN).pot-update rule.

Marked as done.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Fri, 09 Jul 2010 11:24:03 GMT) Full text and rfc822 format available.

This bug report was last modified 14 years and 345 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.