all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Mathieu Lirzin <mthl@gnu.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 0/4] Build man pages in $(srcdir).
Date: Sat, 19 Mar 2016 23:50:07 +0100	[thread overview]
Message-ID: <877fgyjd5s.fsf@gnu.org> (raw)
In-Reply-To: <87mvpudufm.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Sat, 19 Mar 2016 22:33:33 +0100")

ludo@gnu.org (Ludovic Courtès) writes:

> Mathieu Lirzin <mthl@gnu.org> skribis:
>
>> Anyway, inspired by what is done by Automake for info manuals I have tried to
>> build man pages in $(srcdir), which is fixing the issue too.  The bonus is
>> that now all the documentation is consistently built in $(srcdir) and that we
>> avoid adding another "case $? ..." trick.
>
> This sounds reasonable (in fact I wonder why we were not doing it
> already given that things are supposed to live in $(srcdir)).

Why were you expecting such thing?

> Did you confirm that ‘make distcheck’ passes after that, and that
> ‘help2man’ is not invoked when building from a tarball?

I can't confirm for ‘make distcheck’ right now, because of:

--8<---------------cut here---------------start------------->8---
substitute: updating list of substitutes from 'http://hydra.gnu.org'...  33.3%
4 packages are not substitutable:
  /gnu/store/5apc4w376ls7fhydg08plc20agry7pn4-emacs-24.5
  /gnu/store/zfxj52lh6h6q7jmb1w2s34h004zsbaw3-emacs-24.5
  /gnu/store/10qcwd0gxypm977ksbvhjrpa4m1xk334-emacs-24.5
  /gnu/store/r8xphs9ka0lwmwhzxv6mfs9j8nm91843-emacs-24.5

Makefile:4913 : la recette pour la cible « assert-binaries-available » a échouée
--8<---------------cut here---------------end--------------->8---

However I have manually checked that with 'make dist' + (regular build or
VPATH build) 'help2man' is not invoked.

> Also, do these changes preserve the benefits of
> 36629097523b0abd89d1b931293150cb9c1f242d (“build: Generate man pages
> after compiling Guile objects.”)?

Yes it is.  That would be unacceptable if not.  ;)

-- 
Mathieu Lirzin

  reply	other threads:[~2016-03-19 22:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-19 16:18 [PATCH 0/4] Build man pages in $(srcdir) Mathieu Lirzin
2016-03-19 16:18 ` [PATCH 1/4] Revert "build: Do not remake doc/guix.1." Mathieu Lirzin
2016-03-19 21:29   ` Ludovic Courtès
2016-03-19 16:18 ` [PATCH 2/4] build: Make 'guix' man page depend on scripts/guix.in Mathieu Lirzin
2016-03-19 21:31   ` Ludovic Courtès
2016-03-19 16:18 ` [PATCH 3/4] build: Build man pages in $(srcdir) Mathieu Lirzin
2016-03-19 21:31   ` Ludovic Courtès
2016-03-19 16:18 ` [PATCH 4/4] build: Rewrite comments for man pages Mathieu Lirzin
2016-03-19 21:32   ` Ludovic Courtès
2016-03-19 21:28 ` [PATCH 0/4] Build man pages in $(srcdir) Ludovic Courtès
2016-03-19 21:33 ` Ludovic Courtès
2016-03-19 22:50   ` Mathieu Lirzin [this message]
2016-03-20 21:09     ` Ludovic Courtès
2016-03-20 23:03       ` Mathieu Lirzin

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=877fgyjd5s.fsf@gnu.org \
    --to=mthl@gnu.org \
    --cc=guix-devel@gnu.org \
    --cc=ludo@gnu.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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.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.