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(-)
>
prev 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