Re: [PATCH 3/3] texinfo: fix info not work

[Hongxu Jia <hongxu.jia@windriver.com>]

On 09/24/2014 04:40 AM, Burton, Ross wrote:
> On 23 September 2014 21:37, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>> I think there are a few issues here. Firstly, this postinst can run at
>> build time which means there is a dependency on texinfo-native. I guess
>> if this fails, it will just run on target so its probably "harmless" but
>> not deterministic. Secondly, anything installing an info directory
>> really needs to update the dir files with its own postinst. For optimal
>> handling at rootfs time, that will need trigger/intercept support.
>>
>> So this probably can merge as it improves things however we need some
>> enhacement bugs opening related to this, we may want to force the
>> postinst to run on target for now so things are deterministic too.
> There's a texinfo.bbclass which all recipes installing info files are
> probably inheriting anyway, so moving the postinst there should solve
> the race on the directory.  That class says that texinfo-native is
> assume-provided but you'll want to check that usual configurations
> ship install-info alongside makeinfo on the distributions we support.

Hi Ross,

I am afraid moving the postinst to texinfo.bbclass will cause some
potential issues:

- It will be invoked many times if multiple packages installed.

- We could not use a generic way to name pkg_postinst.
   Such as texinfo recipe, we need pkg_postinst_info.

I will refer what Ubuntu/Debian did, add postinst/postrm to info
packages.

//Hongxu

> Ross