public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
From: Michael Frysinger <vapier@sourceware.org>
To: newlib-cvs@sourceware.org
Subject: [newlib-cygwin] newlib: fix mkdoc dependencies
Date: Thu, 10 Feb 2022 04:27:21 +0000 (GMT)	[thread overview]
Message-ID: <20220210042721.72BB73858D1E@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=5c21b3721475c0d848b567837b4459cffcf0ca13

commit 5c21b3721475c0d848b567837b4459cffcf0ca13
Author: Mike Frysinger <vapier@gentoo.org>
Date:   Wed Feb 9 23:26:19 2022 -0500

    newlib: fix mkdoc dependencies
    
    Make sure we depend on the right name of mkdoc all the time, and that
    the rules that need it (e.g. .def files) depend on it.
    
    Reported-by: Jon Turney <jon.turney@dronecode.org.uk>

Diff:
---
 newlib/Makefile.in       | 4 +++-
 newlib/doc/local.mk      | 6 +++---
 newlib/libc/Makefile.inc | 1 +
 newlib/libm/Makefile.inc | 1 +
 4 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/newlib/Makefile.in b/newlib/Makefile.in
index e76fa582a..fd6323260 100644
--- a/newlib/Makefile.in
+++ b/newlib/Makefile.in
@@ -93,7 +93,7 @@ host_triplet = @host@
 @HAVE_MULTISUBDIR_TRUE@am__append_2 = $(BUILD_MULTISUBDIR)/libc.a $(BUILD_MULTISUBDIR)/libm.a
 
 # We can't use noinst_PROGRAMS, because automake will add $(EXEEXT).
-@HAVE_DOC_TRUE@am__append_3 = doc/makedoc
+@HAVE_DOC_TRUE@am__append_3 = $(MKDOC)
 @HAVE_DOC_TRUE@am__append_4 = man-cache
 
 #
@@ -1759,6 +1759,7 @@ clean-local:
 
 @HAVE_DOC_TRUE@.c.xml:
 @HAVE_DOC_TRUE@	$(AM_V_GEN)$(DOCBOOK_CHEW) < $< > $*.xml || ( rm $*.xml && false )
+$(LIBC_CHEWOUT_FILES): $(MKDOC)
 
 libc/libc.dvi: libc/targetdep.tex $(LIBC_CHEWOUT_FILES)
 
@@ -1786,6 +1787,7 @@ libc_install-man: libc_man
 	$(MKDIR_P) $(DESTDIR)$(mandir)/man3
 	$(INSTALL_DATA) libc/*.3 $(DESTDIR)$(mandir)/man3/
 install-man: libc_install-man
+$(LIBM_CHEWOUT_FILES): $(MKDOC)
 
 libm/libm.dvi: libm/targetdep.tex $(LIBM_CHEWOUT_FILES)
 
diff --git a/newlib/doc/local.mk b/newlib/doc/local.mk
index 15c43f225..de1dd1f2b 100644
--- a/newlib/doc/local.mk
+++ b/newlib/doc/local.mk
@@ -1,8 +1,8 @@
-# We can't use noinst_PROGRAMS, because automake will add $(EXEEXT).
-noinst_DATA += doc/makedoc
-
 MKDOC = doc/makedoc$(EXEEXT_FOR_BUILD)
 
+# We can't use noinst_PROGRAMS, because automake will add $(EXEEXT).
+noinst_DATA += $(MKDOC)
+
 # We don't use CFLAGS with CC_FOR_BUILD because here CFLAGS will
 # actually be CFLAGS_FOR_TARGET, and in some cases that will include
 # -Os, which CC_FOR_BUILD may not recognize.
diff --git a/newlib/libc/Makefile.inc b/newlib/libc/Makefile.inc
index fff9e8805..f30ced867 100644
--- a/newlib/libc/Makefile.inc
+++ b/newlib/libc/Makefile.inc
@@ -5,6 +5,7 @@ LIBC_CHEWOUT_FILES =
 
 LIBC_CHAPTERS = %D%/sys.tex
 LIBC_DOCBOOK_OUT_FILES = $(LIBC_CHEWOUT_FILES:.def=.xml)
+$(LIBC_CHEWOUT_FILES): $(MKDOC)
 
 %D%/libc.dvi: %D%/targetdep.tex $(LIBC_CHEWOUT_FILES)
 
diff --git a/newlib/libm/Makefile.inc b/newlib/libm/Makefile.inc
index f2f192ea8..7fdea0ca5 100644
--- a/newlib/libm/Makefile.inc
+++ b/newlib/libm/Makefile.inc
@@ -5,6 +5,7 @@ LIBM_CHEWOUT_FILES =
 
 LIBM_CHAPTERS =
 LIBM_DOCBOOK_OUT_FILES = $(LIBM_CHEWOUT_FILES:.def=.xml)
+$(LIBM_CHEWOUT_FILES): $(MKDOC)
 
 %D%/libm.dvi: %D%/targetdep.tex $(LIBM_CHEWOUT_FILES)


                 reply	other threads:[~2022-02-10  4:27 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20220210042721.72BB73858D1E@sourceware.org \
    --to=vapier@sourceware.org \
    --cc=newlib-cvs@sourceware.org \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).