From: Olaf Hering <olaf@aepfle.de>
To: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH 4/4] docs/html/: Arrange for automatic build of hypercall docs
Date: Thu, 1 Dec 2011 15:12:24 +0100 [thread overview]
Message-ID: <20111201141224.GA12300@aepfle.de> (raw)
In-Reply-To: <1322578893-9998-5-git-send-email-ian.jackson@eu.citrix.com>
On Tue, Nov 29, Ian Jackson wrote:
> - Use index.html rather than a stamp file.
> - Automatically generate dependencies.
> - Wire into the docs build system
>
> +-include html/hypercall/.deps
This creates a new .deps file which is also installed during make
install. Unfortunately the rpm post-build checks reject such a file and
the package build fails. My quick hack to remove it during make install
looks like this:
--- a/docs/Makefile
+++ b/docs/Makefile
@@ -102,7 +102,7 @@ install: all
$(INSTALL_DIR) $(DESTDIR)$(MANDIR)
cp -dR man1 $(DESTDIR)$(MANDIR)
cp -dR man5 $(DESTDIR)$(MANDIR)
- [ ! -d html ] || cp -dR html $(DESTDIR)$(DOCDIR)
+ if [ -d html ] ; then cp -dR html $(DESTDIR)$(DOCDIR) ; find $(DESTDIR)$(DOCDIR) -name .deps -print0 | xargs -0 --no-run-if-empty rm -f ; fi
pdf/%.pdf: ps/%.ps
$(INSTALL_DIR) $(@D)
There is probably a better way to get rid of the file.
Olaf
prev parent reply other threads:[~2011-12-01 14:12 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-29 15:01 [PATCH v2 0/4] docs/html/: Hypercall docs from headers Ian Jackson
2011-11-29 15:01 ` [PATCH 1/4] docs/html/: Initial cut of header documentation massager Ian Jackson
2011-11-29 15:19 ` Ian Campbell
2011-11-29 15:01 ` [PATCH 2/4] docs/html/: Annotations for two hypercalls Ian Jackson
2011-11-29 15:20 ` Ian Campbell
2011-11-29 15:29 ` Ian Jackson
2011-11-29 15:33 ` Ian Campbell
2011-11-29 15:01 ` [PATCH 3/4] docs/html/: Generate an "index.html" for hypercall docs Ian Jackson
2011-11-29 15:25 ` Ian Campbell
2011-11-29 15:01 ` [PATCH 4/4] docs/html/: Arrange for automatic build of " Ian Jackson
2011-11-29 15:27 ` Ian Campbell
2011-11-29 15:49 ` Ian Jackson
2011-12-01 14:12 ` Olaf Hering [this message]
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=20111201141224.GA12300@aepfle.de \
--to=olaf@aepfle.de \
--cc=ian.jackson@eu.citrix.com \
--cc=xen-devel@lists.xensource.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.
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.