all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Paul Eggert <eggert@cs.ucla.edu>
To: "Mattias Engdegård" <mattiase@acm.org>
Cc: phst@google.com, Eli Zaretskii <eliz@gnu.org>,
	Lars Ingebrigtsen <larsi@gnus.org>,
	emacs-devel@gnu.org
Subject: Re: master bf7041a: Centralize subprocess creation in a single function.
Date: Fri, 25 Dec 2020 19:19:13 -0800	[thread overview]
Message-ID: <b5676633-2db7-9ae4-fd3a-de8f481e91f5@cs.ucla.edu> (raw)
In-Reply-To: <0662240F-DA88-4F47-B348-0EDBD9589E43@acm.org>

[-- Attachment #1: Type: text/plain, Size: 375 bytes --]

On 12/25/20 2:19 AM, Mattias Engdegård wrote:

>    CC       malloc/scratch_buffer_grow.o
> error: unable to open output file 'malloc/scratch_buffer_grow.o': 'No such file
>        or directory'
> 1 error generated.
> make[1]: *** [malloc/scratch_buffer_grow.o] Error 1
> make: *** [lib] Error 2

I installed the attached to try to fix this; please give it a try.

[-- Attachment #2: 0001-Fix-issues-with-lib-malloc-out-of-dir-builds.patch --]
[-- Type: text/x-patch, Size: 2322 bytes --]

From 597eef7b947a64ae8067058478f26a15c0e24a07 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Fri, 25 Dec 2020 19:16:27 -0800
Subject: [PATCH] Fix issues with lib/malloc out-of-dir builds
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Problem reported by Mattias Engdegård in:
https://lists.gnu.org/r/emacs-devel/2020-12/msg01626.html
* configure.ac: If the scratch_buffer module is enabled, create
lib/malloc and (if doing dependencies) lib/deps/malloc.
* lib/Makefile.in (DEPFLAGS) [AUTO_DEPEND]: No longer any need to
squash lib dependencies into a single directory.
(clean): Also remove */*.o and $(DEPDIR)/*/*.d, to clean out
malloc/*.o and deps/malloc/*.d.  Just remove *.d files in
dependencies so that rm does not complain about not being able to
remove deps/malloc.
(extraclean): Also remove malloc, if it is empty.
---
 configure.ac    | 6 ++++++
 lib/Makefile.in | 6 +++---
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index 888b415148..bf768441fe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -5878,6 +5878,12 @@ m4_define
      AS_MKDIR_P([$dir/deps])
    done
 fi
+if $gl_gnulib_enabled_scratch_buffer; then
+  AS_MKDIR_P([lib/malloc])
+  if test $AUTO_DEPEND = yes; then
+    AS_MKDIR_P([lib/deps/malloc])
+  fi
+fi
 
 AC_OUTPUT
 
diff --git a/lib/Makefile.in b/lib/Makefile.in
index a280b7a331..e6b1ae6a9a 100644
--- a/lib/Makefile.in
+++ b/lib/Makefile.in
@@ -67,7 +67,7 @@ SYSTEM_TYPE =
 
 DEPDIR = deps
 ifeq ($(AUTO_DEPEND),yes)
-  DEPFLAGS = -MMD -MF $(DEPDIR)/$(*F).d -MP
+  DEPFLAGS = -MMD -MF $(DEPDIR)/$*.d -MP
   -include $(ALLOBJS:%.o=$(DEPDIR)/%.d)
 else
   DEPFLAGS =
@@ -118,7 +118,7 @@ TAGS:
 .PHONY: $(ETAGS) tags
 
 clean:
-	rm -f ./*.[ao] ./*-t \#* $(DEPDIR)/*
+	rm -f ./*.[ao] ./*/*.o ./*-t \#* $(DEPDIR)/*.d $(DEPDIR)/*/*.d
 mostlyclean: clean
 	rm -f $(filter-out %-t,$(MOSTLYCLEANFILES))
 distclean bootstrap-clean: mostlyclean
@@ -127,7 +127,7 @@ distclean bootstrap-clean:
 maintainer-clean: distclean
 	rm -f TAGS gnulib.mk
 extraclean: distclean
-	-rmdir sys 2>/dev/null
+	-rmdir malloc sys 2>/dev/null
 
 .PHONY: mostlyclean clean distclean bootstrap-clean maintainer-clean
 
-- 
2.27.0


  reply	other threads:[~2020-12-26  3:19 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20201224145337.5156.69415@vcs0.savannah.gnu.org>
     [not found] ` <20201224145339.8699721351@vcs0.savannah.gnu.org>
2020-12-24 16:08   ` master bf7041a: Centralize subprocess creation in a single function Lars Ingebrigtsen
2020-12-24 16:09     ` Lars Ingebrigtsen
2020-12-24 16:21       ` Eli Zaretskii
2020-12-24 16:24         ` Eli Zaretskii
2020-12-24 16:37           ` Lars Ingebrigtsen
2020-12-24 16:41             ` Lars Ingebrigtsen
2020-12-24 17:01               ` Lars Ingebrigtsen
2020-12-24 17:11                 ` Lars Ingebrigtsen
2020-12-24 17:20                   ` Eli Zaretskii
2020-12-24 17:31                     ` Lars Ingebrigtsen
2020-12-24 17:43                       ` Lars Ingebrigtsen
2020-12-25 10:19                         ` Mattias Engdegård
2020-12-26  3:19                           ` Paul Eggert [this message]
2020-12-26 10:29                             ` Mattias Engdegård
2020-12-25  8:19                       ` Paul Eggert
2020-12-25 10:42                         ` Paul Eggert
2020-12-25 11:10                           ` Philipp Stephani
2020-12-25 19:58                           ` Lars Ingebrigtsen
2020-12-25 11:14             ` Philipp Stephani
2020-12-24 18:05     ` Philipp Stephani

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b5676633-2db7-9ae4-fd3a-de8f481e91f5@cs.ucla.edu \
    --to=eggert@cs.ucla.edu \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=larsi@gnus.org \
    --cc=mattiase@acm.org \
    --cc=phst@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.