| Submitter | Paul Eggleton |
|---|---|
| Date | Aug. 20, 2012, 8:47 a.m. |
| Message ID | <1345452476-5231-1-git-send-email-paul.eggleton@linux.intel.com> |
| Download | mbox | patch |
| Permalink | /patch/34917/ |
| State | Accepted |
| Commit | aa1c4519f8e549b43a7d8bc51c5342d3409b464b |
| Headers | show |
Comments
On Mon, 2012-08-20 at 09:47 +0100, Paul Eggleton wrote: > texi2html is not actually required to build world of OE-Core anymore, > so we don't need to mandate it. The only difference without it (verified > with buildhistory) is that groff doesn't produce HTML documentation; the > rest of its docs are still produced and packaged and no other packages > are affected. Can we fix groff to make it not even try to use texi2html even if it's installed? It seems a bit undesirable for the amount of documentation you get to vary depending on the host environment. p.
On Monday 20 August 2012 09:48:20 Phil Blundell wrote: > On Mon, 2012-08-20 at 09:47 +0100, Paul Eggleton wrote: > > texi2html is not actually required to build world of OE-Core anymore, > > so we don't need to mandate it. The only difference without it (verified > > with buildhistory) is that groff doesn't produce HTML documentation; the > > rest of its docs are still produced and packaged and no other packages > > are affected. > > Can we fix groff to make it not even try to use texi2html even if it's > installed? It seems a bit undesirable for the amount of documentation > you get to vary depending on the host environment. Actually, upon closer inspection, texi2html has nothing to do with this. The host needs to have pnmcut, pnmcrop, pnmtopng, psselect, pnmtops and gs installed or neither the HTML nor PDF backends for groff will be enabled. This leaves us with two alternatives: 1) Make groff depend on ghostscript, and that would at least enable the PDF backend, but for HTML it still leaves netpbm (for pnm*) and psutils (for psselect), neither of which we have in OE-Core. 2) Alternatively I can add --without-gs to the groff configure options which will always disable both backends. Cheers, Paul
Patch
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass index 575ce15..ff7c73b 100644 --- a/meta/classes/sanity.bbclass +++ b/meta/classes/sanity.bbclass @@ -2,7 +2,7 @@ # Sanity check the users setup for common misconfigurations # -SANITY_REQUIRED_UTILITIES ?= "patch diffstat texi2html makeinfo git bzip2 tar gzip gawk chrpath wget cpio" +SANITY_REQUIRED_UTILITIES ?= "patch diffstat makeinfo git bzip2 tar gzip gawk chrpath wget cpio" def raise_sanity_error(msg, d): if d.getVar("SANITY_USE_EVENTS", True) == "1":
texi2html is not actually required to build world of OE-Core anymore, so we don't need to mandate it. The only difference without it (verified with buildhistory) is that groff doesn't produce HTML documentation; the rest of its docs are still produced and packaged and no other packages are affected. Part of the work towards [YOCTO #2423]. Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com> --- meta/classes/sanity.bbclass | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)