On 07/27/15 at 08:50am, Dave Young wrote:
On 07/25/15 at 04:11pm, Minfei Huang wrote:
> On 07/23/15 at 04:47pm, Dave Young wrote:
> > On 07/22/15 at 06:20pm, Minfei Huang wrote:
> > > Since get_persistent_dev is called in dracut-module-setup.sh and
> > > mkdumprd, we move this lib from mkdumprd to kdump-lib.sh. Thus
> > > dracut-module-setup.sh can use this lib, not the function in dracut.sh.
> > >
> > > Copy the function get_persistent_dev from dracut to make the function
> > > consistent.
> >
> > Cleanup means drop get_presistent_dev function and use dracut one instead
> > of moving it to another file...
> >
>
> Hi, Dave.
>
> Try the way which you points out. It seems difficult to source the
> dracut-functions.sh into mkdumprd, because there are several variants
> which are defined in dracut.sh, such as initdir. In the other hand,
> dracut-functions.sh will source the dracut-logger.sh and other file.
Thanks Dave.
I am a bit confused with this patch. IMOP, if there is not heavy
dependency with other module, self_containing is better.
Also as Vivek reported in bugzilla, moving the common libraries to
kdump-lib.sh is suggested.
Thanks
Minfei
What's the error you see? I think it does not matter, if there's something
which should not be in dracut-functions.sh you can send patch to dracut
list and cleanup. OTOH, the functions mkdumprd uses do not care about
initdir.
Thanks
Dave