[lxc-devel] [patch] configure.ac - allow user to switch off/on documentation generation

[Michel Normand]

Le mercredi 04 novembre 2009 à 13:22 +0100, Daniel Lezcano a écrit :
> Andrian Nord wrote:
> > On Tue, Nov 03, 2009 at 07:48:39PM +0100, Michel Normand wrote:
> >   
> >> Andrian Nord a écrit :
> >>     
> >>> As documentation requires docbook2man to be installed, which is not,
> >>> otherwise, required for proper LXC work or compilation process, it
> >>> might be usefull to be able to switch it off.
> >>>
> >>> Sorry for attaching file, I currently unable to paste it into mail, as
> >>> my mail client (gmail webmail) will word-wrap lines, making patch
> >>> unusable
> >>>
> >>>
> >>>       
> >> For me, it is Ok to add a --enable/disable/-doc,
> >> but not make configure to fail if no option specified
> >> and no docbook2man package.
> >>
> >> For me it should be optionnal.
> >> I like the current behaviour where configure is running without option
> >> and is enabling/disabling by itself the doc building.
> >> Could you send a new patch with this idea ?
> >>
> >> ---
> >> Michel
> >>     
> >
> > Of course. You mean, that you what default behaviour to remain
> > auto-detection? That is:
> > --enable-doc: require docbook2man or fail, generate mans
> > --enable-doc=auto, or not specified (default): check for docbook2man,
> > generate mans if found, silently ignore if not found (I suppose
> > diagnostic message is redundant, as information already contains into
> > ./configure --help)
> > --disable-doc: never check for docbook2man and don't gen mans
> >
> > Here comes a patch what do this, as far as I see
> > (I'm sorry for violating post-rules in previous mail, now I'll do all
> > right, I hope. Should I attach patch anyway, as it might be usefull
> > for applying?)
> >
> > Signed-off-by: Andrian Nord <NightNord at gmail.com>
> >   
> Michel,
> 
> are you ok for this patch ?
> 

Daniel, yes it is OK, for me, the only concequence 
is that there will be no warning message if no option is specified
and docbook2man is not available.
We probably can live with that.

---
Michel