[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/13] build: Generate man pages after compiling guile object
From: |
Mathieu Lirzin |
Subject: |
Re: [PATCH 06/13] build: Generate man pages after compiling guile objects. |
Date: |
Fri, 29 Jan 2016 17:17:13 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Bavier <address@hidden> writes:
> On Thu, 28 Jan 2016 18:00:35 +0100
> Mathieu Lirzin <address@hidden> wrote:
>
>> OTOH I must admit that for maintainability reasons I would prefer
>> avoiding complexity in Makefiles as much as possible. This means
>> using Automake's abstract concepts as much as possible, having a
>> simple dependency graph on top of that, and put things that don't fit
>> well in this scheme in individual "build-aux/" scripts. For that
>> reason I still prefer the embed help2man solution. but otherwise it
>> looks nice. :)
>
> I sympathize with the concerns for maintainability. In this case, it
> seems to me that a few lines of make/shell would be preferable to 20k
> lines of perl. Perhaps the simpler attached patch is better?
A few lines of Make/Shell seems more maintainable than a full perl
program. But in this case the maintenance is handled upstream, so it
only consists in updating the script periodically (which is still not
ideal).
Your patch is definitely better now since the hack is localized to a
specific target. Let's use this solution! I have attached an updated
version which simplifies how silent rules are handled and integrates
changes from my initial patch.
0001-build-Generate-man-pages-after-compiling-Guile-objec.patch
Description: Text Data
Are you OK with it?
Thanks,
--
Mathieu Lirzin
ps: I was speaking of 20KB (790 lines of perl) not 20K lines of code.
:)
- [PATCH 03/13] lint: Rewrite 'check-patch-file-names'., (continued)
- [PATCH 03/13] lint: Rewrite 'check-patch-file-names'., Mathieu Lirzin, 2016/01/24
- [PATCH 06/13] build: Generate man pages after compiling guile objects., Mathieu Lirzin, 2016/01/24
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Ludovic Courtès, 2016/01/26
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Mathieu Lirzin, 2016/01/26
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Eric Bavier, 2016/01/28
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Mathieu Lirzin, 2016/01/28
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Eric Bavier, 2016/01/28
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Mathieu Lirzin, 2016/01/28
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Eric Bavier, 2016/01/29
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects.,
Mathieu Lirzin <=
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Eric Bavier, 2016/01/29
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Mathieu Lirzin, 2016/01/29
- Re: [PATCH 06/13] build: Generate man pages after compiling guile objects., Ludovic Courtès, 2016/01/31
[PATCH 08/13] build: Add 'DOT' silent rule., Mathieu Lirzin, 2016/01/24
[PATCH 11/13] build: Delete emacs/guix-autoloads.el when 'make clean'., Mathieu Lirzin, 2016/01/24
[PATCH 10/13] build: Add 'HELP2MAN' silent rule., Mathieu Lirzin, 2016/01/24