[OE-core] [oe-core][rocko][sumo][master][PATCH 2/2] base-files: respect PERSISTENT_LOG_DIR
Ankur Tyagi
ankur.tyagi85 at gmail.com
Thu Aug 30 08:42:22 UTC 2018
On Thu, Aug 30, 2018 at 8:38 PM Khem Raj <raj.khem at gmail.com> wrote:
>
> On Thu, Aug 30, 2018 at 1:34 AM Ankur Tyagi <ankur.tyagi85 at gmail.com> wrote:
> >
> > Hi Khem,
> >
> > Is this patch being accepted ?
> >
>
> I am not convinced if we need this patch as we can do this using volatile binds.
> or base-files changes if needed.
>
How volatile binds will help with VOLATILE_LOG_DIR set as "no" ?
This patch is modifying base-files so that users can simply set a
variable in conf file instead of modifying base-files.
> > regards
> > Ankur
> >
> > On Tue, Aug 28, 2018 at 7:45 AM Ankur Tyagi <ankur.tyagi85 at gmail.com> wrote:
> > >
> > > On Sat, Aug 25, 2018 at 12:17 PM Khem Raj <raj.khem at gmail.com> wrote:
> > > >
> > > > On Fri, Aug 17, 2018 at 6:00 PM Ankur Tyagi <ankur.tyagi85 at gmail.com> wrote:
> > > > >
> > > > > Respect PERSISTENT_LOG_DIR variable. In this way, if the user overrides
> > > > > this variable to be e.g "/home/root/log", /var/log on the final image would
> > > > > be a link pointing to /home/root/log on persistent storage.
> > > > >
> > > >
> > > > I would suggest to stick with VOLATILE_BINDS
> > >
> > > But this won't help with VOLATILE_LOG_DIR set as "no".
> > > I think it is a common requirement to have persistent logging mounted
> > > in a separate partition.
> > > This patch makes it easy (hopefully) for the user to achieve the above
> > > without any additional recipe.
> > >
> > > >
> > > > > Signed-off-by: Ankur Tyagi <ankur.tyagi85 at gmail.com>
> > > > > ---
> > > > > meta/recipes-core/base-files/base-files_3.0.14.bb | 6 +++++-
> > > > > 1 file changed, 5 insertions(+), 1 deletion(-)
> > > > >
> > > > > diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
> > > > > index 1c0863b..597fdaa 100644
> > > > > --- a/meta/recipes-core/base-files/base-files_3.0.14.bb
> > > > > +++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
> > > > > @@ -42,7 +42,7 @@ dirs755 = "/boot /dev ${base_bindir} ${base_sbindir} ${base_libdir} \
> > > > > ${localstatedir}/backups ${localstatedir}/lib \
> > > > > /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
> > > > > ${localstatedir}/volatile \
> > > > > - ${localstatedir}/${@'volatile/' if oe.types.boolean('${VOLATILE_LOG_DIR}') else ''}log \
> > > > > + ${@'${localstatedir}/volatile/log' if oe.types.boolean('${VOLATILE_LOG_DIR}') else '${PERSISTENT_LOG_DIR}' if d.getVar('PERSISTENT_LOG_DIR') else '${localstatedir}/log' } \
> > > > > /home ${prefix}/src ${localstatedir}/local \
> > > > > /media"
> > > > >
> > > > > @@ -106,6 +106,10 @@ do_install () {
> > > > > ln -sf volatile/$d ${D}${localstatedir}/$d
> > > > > done
> > > > >
> > > > > + if [ ${@ oe.types.boolean('${VOLATILE_LOG_DIR}') } = False -a ! -z ${PERSISTENT_LOG_DIR} ]; then
> > > > > + ln -sf ${PERSISTENT_LOG_DIR} ${D}/${localstatedir}/log
> > > > > + fi
> > > > > +
> > > > > ln -snf ../run ${D}${localstatedir}/run
> > > > > ln -snf ../run/lock ${D}${localstatedir}/lock
> > > > >
> > > > > --
> > > > > 2.7.4
> > > > >
> > > > > --
> > > > > _______________________________________________
> > > > > Openembedded-core mailing list
> > > > > Openembedded-core at lists.openembedded.org
> > > > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
More information about the Openembedded-core
mailing list