This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [PATCH] Add install-html stub for libstdc++-v3
- From: Carlos O'Donell <carlos at codesourcery dot com>
- To: Benjamin Kosnik <bkoz at redhat dot com>
- Cc: libstdc++ at gcc dot gnu dot org, gcc-patches at gnu dot org
- Date: Thu, 25 May 2006 14:03:44 -0400
- Subject: Re: [PATCH] Add install-html stub for libstdc++-v3
- References: <20060524223600.GK26980@lios> <20060525105357.bc214055.bkoz@redhat.com>
On Thu, May 25, 2006 at 10:53:57AM -0500, Benjamin Kosnik wrote:
>
> > +# Install distribution html documentation, doxygen documentation
> > +# also needs install.
> > +
> > +.PHONY: install-html
> > +
> > +install-html:
>
> Hmm. Well, this does nothing.
> What are you trying to do?
The "html" and "install-html" targets have been added at the top-level.
Both of these targets are recursive. Each library should provide the
"html" and "install-html" targets.
I am implementing the "html" and "install-html" targets where
appropriate. I was not familiar enough with the library to write the
required "install-html" rule, so instead a stub was added.
> I see an "install-html" rule in libiberty. If you're going to do
> something similar, I'd rather it do something, like what the comment
> says. (Ie, install the docs, generate and install the doxygen bits.)
>
> Sound reasonable?
Could we change the comment to read:
# Installation of distribution html documentation not yet supported.
# TODO: Write custom install-html rule.
Cheers,
Carlos.
--
Carlos O'Donell
CodeSourcery
carlos@codesourcery.com
(650) 331-3385 x716