On Sat, Aug 25, 2018 at 12:17 PM Khem Raj <raj.k...@gmail.com> wrote: > > On Fri, Aug 17, 2018 at 6:00 PM Ankur Tyagi <ankur.tyag...@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.tyag...@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@lists.openembedded.org > > http://lists.openembedded.org/mailman/listinfo/openembedded-core -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core