Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/3] man/info: fix not support xz/bz2 compression
Date: Tue, 23 Sep 2014 10:16:21 -0500	[thread overview]
Message-ID: <54218EC5.6000708@windriver.com> (raw)
In-Reply-To: <cover.1411473883.git.hongxu.jia@windriver.com>

On 9/23/14, 9:19 AM, Hongxu Jia wrote:
> Test Case:
>
> vim local.conf
> ..
> IMAGE_INSTALL_append = " man man-doc texinfo texinfo-doc info info-doc"
> INHERIT += "compress_doc"
> DOC_COMPRESS = "xz" or DOC_COMPRESS = "bz2"
> ..

Patch 1/3 the busybox change.  I'm not sure we really want to make that change 
as it will increase the size of busybox.

(following is me thinking aloud and hopefully someone will have a suggest on how 
best to unify this..)

I'm wondering if instead of forcing the change for everyone via the busybox 
change, if it makes sense to add a image validation routine or something that 
verifies if the DOC_COMPRESS is set, that the corresponding tool was installed 
into the target environment (along w/ man, info or other appropriate items.. and 
the pkgs-docs was set as well.. if any of those are missing then a warning 
should probably be presented.)

I'm sure there is probably a better way to set this via a distribution 
configuration switch or something as well.. maybe something like?

DISTRO += "man info"

That could then force the man and info systems to add a dependency on bz2 (or xz 
or whatever else was set in DOC_COMPRESS)..  and also add man, info and the 
associated man and info pages to the image itself.

(As I see it, DOC_COMPRESS is really a distribution wide setting, so we should 
treat it as such...)


--Mark

> bitbake core-image-minimal
>
> runqemu qemux86
>
> On target,
> ...
> root@qemux86:~# man man
>
> root@qemux86:~# info info
> ...
>
> //Hongxu
>
> The following changes since commit afdbe3112b940987f3f3cddcb32c91a5e17a297d:
>
>    bitbake: bitbake-worker: Fix bitbake -n (2014-09-22 15:54:40 +0100)
>
> are available in the git repository at:
>
>    git://git.pokylinux.org/poky-contrib hongxu/fix-doc
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-doc
>
> Hongxu Jia (3):
>    busybox: enable CONFIG_BZIP2/CONFIG_XZ/CONFIG_UNXZ by default
>    man: fix not support xz/bz2 compression
>    texinfo: fix info not work
>
>   meta/recipes-core/busybox/busybox/defconfig  | 6 +++---
>   meta/recipes-extended/man/man/man.conf       | 3 ++-
>   meta/recipes-extended/texinfo/texinfo_5.2.bb | 8 +++++++-
>   3 files changed, 12 insertions(+), 5 deletions(-)
>



      parent reply	other threads:[~2014-09-23 15:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-23 14:19 [PATCH 0/3] man/info: fix not support xz/bz2 compression Hongxu Jia
2014-09-23 14:19 ` [PATCH 1/3] busybox: enable CONFIG_BZIP2/CONFIG_XZ/CONFIG_UNXZ by default Hongxu Jia
2014-09-23 14:47   ` Burton, Ross
2014-09-24 10:46     ` Hongxu Jia
2014-09-23 14:19 ` [PATCH 2/3] man: fix not support xz/bz2 compression Hongxu Jia
2014-09-23 14:19 ` [PATCH 3/3] texinfo: fix info not work Hongxu Jia
2014-09-23 14:52   ` Burton, Ross
2014-09-23 20:37   ` Richard Purdie
2014-09-23 20:40     ` Burton, Ross
2014-09-24  9:16       ` Hongxu Jia
2014-09-24 10:50         ` Burton, Ross
2014-09-24  8:58     ` Hongxu Jia
2014-09-23 15:16 ` Mark Hatle [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54218EC5.6000708@windriver.com \
    --to=mark.hatle@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox