Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: akuster808 <akuster808@gmail.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-oe][krogoth][PATCH] syslog-ng: expand service file @variables@ earlier
Date: Wed, 7 Dec 2016 12:40:15 -0500	[thread overview]
Message-ID: <20161207174015.GC30576@denix.org> (raw)
In-Reply-To: <af58f301-6a21-6f9b-3c87-26fa8d44935b@gmail.com>

On Fri, Nov 18, 2016 at 07:25:48AM -0800, akuster808 wrote:
> 
> 
> On 11/16/2016 07:58 PM, Denys Dmytriyenko wrote:
> >On Mon, Nov 14, 2016 at 02:09:40PM -0500, Denys Dmytriyenko wrote:
> >>On Thu, Nov 03, 2016 at 08:20:53AM -0700, akuster808 wrote:
> >>>
> >>>On 10/27/2016 10:02 AM, Denys Dmytriyenko wrote:
> >>>>From: Ian Ray <ian.ray@ge.com>
> >>>>
> >>>>Move variable expansion of @SBINDIR@, @LOCALSTATEDIR@, and @BASEBINDIR@
> >>>>earlier during install, to before the systemd service file is installed
> >>>>by an automake rule.
> >>>>
> >>>>Signed-off-by: Ian Ray <ian.ray@ge.com>
> >>>>Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >>>merged to staging.
> >>Armin,
> >>
> >>Any updates on this? Also, I don't see it in krogoth-next - what staging you
> >>are referring to?
> >Ping again, this time copy Armin directly.
> oops, Had it sitting on my system forgot to push to contrib. Pull
> request sent.

Armin,

Thanks, this hasn't been merged yet, it appears... Any ETA?

-- 
Denys


> >>>>  meta-oe/recipes-support/syslog-ng/syslog-ng.inc | 9 ++++++---
> >>>>  1 file changed, 6 insertions(+), 3 deletions(-)
> >>>>
> >>>>diff --git a/meta-oe/recipes-support/syslog-ng/syslog-ng.inc b/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
> >>>>index 4e0f697..898e9c2 100644
> >>>>--- a/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
> >>>>+++ b/meta-oe/recipes-support/syslog-ng/syslog-ng.inc
> >>>>@@ -60,6 +60,12 @@ do_configure_prepend() {
> >>>>      eval "${@bb.utils.contains('DISTRO_FEATURES', 'largefile', '', 'sed -i -e "s/-D_LARGEFILE_SOURCE//" -e "s/-D_FILE_OFFSET_BITS=64//" ${S}/configure.in', d)}"
> >>>>  }
> >>>>+do_install_prepend() {
> >>>>+    sed -i -e 's,@SBINDIR@,${sbindir},g' ${S}/contrib/systemd/*.service
> >>>>+    sed -i -e 's,@LOCALSTATEDIR@,${localstatedir},g' ${S}/contrib/systemd/*.service
> >>>>+    sed -i -e 's,@BASEBINDIR@,${base_bindir},g' ${S}/contrib/systemd/*.service
> >>>>+}
> >>>>+
> >>>>  do_install_append() {
> >>>>      install -d ${D}/${sysconfdir}/${BPN}
> >>>>      install ${WORKDIR}/syslog-ng.conf ${D}${sysconfdir}/${BPN}/${BPN}.conf
> >>>>@@ -70,9 +76,6 @@ do_install_append() {
> >>>>      install -d ${D}/${localstatedir}/lib/${BPN}
> >>>>      # Remove /var/run as it is created on startup
> >>>>      rm -rf ${D}${localstatedir}/run
> >>>>-    sed -i -e 's,@SBINDIR@,${sbindir},g' ${S}/contrib/systemd/*.service
> >>>>-    sed -i -e 's,@LOCALSTATEDIR@,${localstatedir},g' ${S}/contrib/systemd/*.service
> >>>>-    sed -i -e 's,@BASEBINDIR@,${base_bindir},g' ${S}/contrib/systemd/*.service
> >>>>  }
> >>>>  FILES_${PN} += "${datadir}/include/scl/ ${datadir}/xsd ${datadir}/tools"
> >>>>
> >>>-- 
> >>>_______________________________________________
> >>>Openembedded-devel mailing list
> >>>Openembedded-devel@lists.openembedded.org
> >>>http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >>-- 
> >>_______________________________________________
> >>Openembedded-devel mailing list
> >>Openembedded-devel@lists.openembedded.org
> >>http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 


  reply	other threads:[~2016-12-07 17:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-27 17:02 [meta-oe][krogoth][PATCH] syslog-ng: expand service file @variables@ earlier Denys Dmytriyenko
2016-10-27 19:55 ` Khem Raj
2016-10-27 20:04   ` Denys Dmytriyenko
2016-10-28  1:27     ` Khem Raj
2016-11-03 15:20 ` akuster808
2016-11-14 19:09   ` Denys Dmytriyenko
2016-11-17  3:58     ` Denys Dmytriyenko
2016-11-18 15:25       ` akuster808
2016-12-07 17:40         ` Denys Dmytriyenko [this message]
2016-12-13  3:55           ` akuster808
2016-12-14  2:25             ` Denys Dmytriyenko

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=20161207174015.GC30576@denix.org \
    --to=denis@denix.org \
    --cc=akuster808@gmail.com \
    --cc=openembedded-devel@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