Re: [ANNOUNCE] Guilt v0.31

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Michael J Gruber
Date: Tuesday, September 9, 2008 - 8:40 am

Junio C Hamano venit, vidit, dixit 09.09.2008 17:13:

It makes it more systematic.

Right now:

Documentation/Makefile:
all: builds html and man
install: installs only man (depends on man)
install-{info,html}: installs $foo (depends on $foo)
quick-install: installs man from man branch

Makefile:
doc: builds html and man (-C D all)
install-doc: installs only man (-C D install)
install-{info,html}: installs $foo (-C D install-$foo)
quick-install-doc: installs man from man branch (-C D install-$foo)

So there is a distinct asymmetry between build targets and install
targets. Wouldn't you expect "make install" to install what "make"
builds? That is true, but "make install-doc" doesn't install what "make
doc" builds.

Put it differently:
doc = man + html for build targets
doc = man for install targets

What's worse: "make doc" fails if you don't have xmlto, even though
"make install-doc" may succeed; but you'll never try it when the build
fails. Until a few hours ago I thought I couldn't build man pages at
work (asciidoc, but no xmlto), and I had never heard about
quick-install-doc until I looked at the Makefiles in detail.


Sure I can. "Namen sind Schall und Rauch". Names don't matter.
quick-install-html would follow the above logic. I just think that the
doc logic is flawed.

So, my suggestion is that doc = man + html, whether it appears in a
build target or an install target.


I keep hearing this argument, and I'm even more surprised to hear it
right after a major release. If not now then when's a good time for
cleaning up confusing inconsistencies?

Michael
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[ANNOUNCE] Guilt v0.31, Josef 'Jeff' Sipek, (Mon Sep 8, 6:52 am)
Re: [ANNOUNCE] Guilt v0.31, Bert Wesarg, (Mon Sep 8, 11:33 am)
Re: [ANNOUNCE] Guilt v0.31, Josef 'Jeff' Sipek, (Mon Sep 8, 11:43 am)
Re: [ANNOUNCE] Guilt v0.31, Bert Wesarg, (Mon Sep 8, 11:50 am)
Re: [ANNOUNCE] Guilt v0.31, Junio C Hamano, (Mon Sep 8, 12:17 pm)
Re: [ANNOUNCE] Guilt v0.31, Michael J Gruber, (Tue Sep 9, 3:10 am)
Re: [ANNOUNCE] Guilt v0.31, Michael J Gruber, (Tue Sep 9, 6:02 am)
Re: [ANNOUNCE] Guilt v0.31, Junio C Hamano, (Tue Sep 9, 8:13 am)
Re: [ANNOUNCE] Guilt v0.31, Michael J Gruber, (Tue Sep 9, 8:40 am)
Re: [ANNOUNCE] Guilt v0.31, Junio C Hamano, (Tue Sep 9, 9:02 am)
Re: [ANNOUNCE] Guilt v0.31, Michael J Gruber, (Tue Sep 9, 9:29 am)