All of lore.kernel.org
 help / color / mirror / Atom feed
From: Florin Sarbu <florin.sarbu@windriver.com>
To: <openembedded-devel@lists.openembedded.org>
Subject: Re: [meta-oe][meta-systemd][] rpcbind: Add ${PN}-systemd to PACKAGES
Date: Wed, 6 Mar 2013 16:22:52 +0200	[thread overview]
Message-ID: <5137513C.4020007@windriver.com> (raw)
In-Reply-To: <5136F879.3050808@windriver.com>

So no plans to remove the systemd support from meta-openembedded now 
that poky has a systemd.bbclass merged in? My feeling is that this adds 
unneeded overhead now.

On 03/06/2013 10:04 AM, Florin Sarbu wrote:
> Actually no, forgot to mention that. My bad. I am using the systemd 
> class from poky. Now that poky has systemd.bbclass, what is the point 
> of the meta-openembedded systemd class? I thought it would go away and 
> people would rely on the poky features for systemd enabling.
>
> On 03/06/2013 10:00 AM, Martin Jansa wrote:
>> On Wed, Mar 06, 2013 at 09:38:13AM +0200, Florin Sarbu wrote:
>>> Hi,
>>> this is still needed (for me at least but it looks like should be the
>>> case for everybody too). I have today's master on both poky and
>>> meta-openembedded and get:
>>>
>>> ERROR: rpcbind-systemd does not appear in package list, please add it
>>> ERROR: Function failed:
>>>
>>> SYSTEMD_SERVICE_rpcbind-systemd value rpcbind.service does not exist
>> Are you sure you're using systemd.bbclass from meta-systemd?
>>
>> $ bitbake -e rpcbind | grep ^PACKAGES
>> PACKAGESPLITFUNCS="populate_packages_updatercd 
>> package_do_split_locales                 populate_packages"
>> PACKAGES="rpcbind-systemd rpcbind-dbg rpcbind-staticdev rpcbind-dev 
>> rpcbind-doc rpcbind-locale  rpcbind"
>> PACKAGES_DYNAMIC="^rpcbind-locale-.*"
>>
>> # $PACKAGES [3 operations]
>> #   set conf/bitbake.conf:270
>> #     "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale 
>> ${PACKAGE_BEFORE_PN} ${PN}"
>> #   set /OE/shr-core/openembedded-core/meta/conf/documentation.conf:91
>> #     [doc] "Names of packages to be generated. @group packaging"
>> #   set systemd.bbclass:33 [systemd_create_package]
>> #     "rpcbind-systemd rpcbind-dbg rpcbind-staticdev rpcbind-dev 
>> rpcbind-doc rpcbind-locale  rpcbind"
>> # computed:
>> #   "rpcbind-systemd rpcbind-dbg rpcbind-staticdev rpcbind-dev 
>> rpcbind-doc rpcbind-locale  rpcbind"
>> PACKAGES="rpcbind-systemd rpcbind-dbg rpcbind-staticdev rpcbind-dev 
>> rpcbind-doc rpcbind-locale  rpcbind"
>>
>>>
>>> Have Khem's patches you were talking about been merged? Can't see them
>>> anywhere though.
>>>
>>> Thank you,
>>> Florin
>>>
>>> On 02/14/2013 11:44 AM, Martin Jansa wrote:
>>>> On Thu, Feb 14, 2013 at 11:32:05AM +0200, Florin Sarbu wrote:
>>>>> The ${PN}-systemd should be added to PACKAGES
>>>>> so that the package gets created.
>>>>>
>>>>> Added do_install_append to have the systemd.service
>>>>> installed in the proper location.
>>>> see khem's patch fixing this in many recipes including rpcbind.
>>>>
>>>>> Signed-off-by: Florin Sarbu <florin.sarbu@windriver.com>
>>>>> ---
>>>>> .../meta-oe/recipes-extended/rpcbind/rpcbind_0.2.0.bbappend |    8 
>>>>> +++++++-
>>>>>    1 file changed, 7 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git 
>>>>> a/meta-systemd/meta-oe/recipes-extended/rpcbind/rpcbind_0.2.0.bbappend 
>>>>> b/meta-systemd/meta-oe/recipes-extended/rpcbind/rpcbind_0.2.0.bbappend 
>>>>>
>>>>> index 0fa7f97..8197550 100644
>>>>> --- 
>>>>> a/meta-systemd/meta-oe/recipes-extended/rpcbind/rpcbind_0.2.0.bbappend 
>>>>>
>>>>> +++ 
>>>>> b/meta-systemd/meta-oe/recipes-extended/rpcbind/rpcbind_0.2.0.bbappend 
>>>>>
>>>>> @@ -1,6 +1,8 @@
>>>>>    inherit systemd
>>>>>    -PRINC := "${@int(PRINC) + 1}"
>>>>> +PACKAGES =+ "${PN}-systemd"
>>>>> +
>>>>> +PRINC := "${@int(PRINC) + 2}"
>>>>>       # look for files in the layer first
>>>>>    FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>>>>> @@ -10,3 +12,7 @@ SRC_URI += "file://rpcbind.service"
>>>>>    SYSTEMD_PACKAGES = "${PN}-systemd"
>>>>>    SYSTEMD_SERVICE = "rpcbind.service"
>>>>>    +do_install_append() {
>>>>> +        install -d ${D}${systemd_unitdir}/system/
>>>>> +        install -m 0644 ${WORKDIR}/rpcbind.service 
>>>>> ${D}${systemd_unitdir}/system
>>>>> +}
>>>>> -- 
>>>>> 1.7.10.4
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Openembedded-devel mailing list
>>>>> Openembedded-devel@lists.openembedded.org
>>>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel 
>>>>>
>>>>
>>>> _______________________________________________
>>>> Openembedded-devel mailing list
>>>> Openembedded-devel@lists.openembedded.org
>>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>> _______________________________________________
>>> Openembedded-devel mailing list
>>> Openembedded-devel@lists.openembedded.org
>>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel




  parent reply	other threads:[~2013-03-06 14:40 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-14  9:32 [PATCH] rpcbind: Add ${PN}-systemd to PACKAGES Florin Sarbu
2013-02-14  9:32 ` [meta-oe][meta-systemd][] " Florin Sarbu
2013-02-14  9:44   ` Martin Jansa
2013-03-06  7:38     ` Florin Sarbu
2013-03-06  8:00       ` Martin Jansa
2013-03-06  8:04         ` Florin Sarbu
2013-03-06 12:17           ` Martin Jansa
2013-03-06 14:22           ` Florin Sarbu [this message]
2013-03-06 16:27             ` Khem Raj

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=5137513C.4020007@windriver.com \
    --to=florin.sarbu@windriver.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.