[oe] [meta-oe][for-denzil][PATCH] systemd.bbclass: use system variables instead of hardcoded paths

[Liu Ting-B28495]

> -----Original Message-----
> From: openembedded-devel-bounces at lists.openembedded.org
> [mailto:openembedded-devel-bounces at lists.openembedded.org] On Behalf Of
> Otavio Salvador
> Sent: Thursday, September 13, 2012 9:04 PM
> To: openembedded-devel at lists.openembedded.org
> Cc: McClintock Matthew-B29882
> Subject: Re: [oe] [meta-oe][for-denzil][PATCH] systemd.bbclass: use
> system variables instead of hardcoded paths
> 
> On Thu, Sep 13, 2012 at 4:33 AM,  <b28495 at freescale.com> wrote:
> > diff --git a/meta-oe/classes/systemd.bbclass
> > b/meta-oe/classes/systemd.bbclass index a2c8ddf..9690c18 100644
> > --- a/meta-oe/classes/systemd.bbclass
> > +++ b/meta-oe/classes/systemd.bbclass
> > @@ -154,7 +154,10 @@ python populate_packages_prepend () {
> >
> >         # check service-files and call systemd_add_files_and_parse for
> each entry
> >         def systemd_check_services():
> > -               searchpaths = '/etc/systemd/system/
> /lib/systemd/system/ /usr/lib/systemd/system/'
> > +               searchpaths = []
> > +               searchpaths.append(d.getVar('sysconfdir', True))
> > +               searchpaths.append(d.getVar('libdir', True))
> > +               searchpaths.append(d.getVar('base_libdir', True))
> >                 systemd_packages = d.getVar('SYSTEMD_PACKAGES', 1)
> >                 has_exactly_one_service = len(systemd_packages.split())
> == 1
> >                 if has_exactly_one_service:
> > @@ -168,9 +171,9 @@ python populate_packages_prepend () {
> >                 for pkg_systemd in systemd_packages.split():
> >                         for service in get_package_var(d,
> 'SYSTEMD_SERVICE', pkg_systemd).split():
> >                                 path_found = ''
> > -                               for path in searchpaths.split():
> > -                                       if os.path.exists('${D}' + path
> + service):
> > -                                               path_found = path
> > +                               for path in searchpaths:
> > +                                       if os.path.exists('${D}' + path
> + '/systemd/system/' + service):
> > +                                               path_found = path +
> '/systemd/system/'
> >                                 if path_found != '':
> >
> systemd_add_files_and_parse(pkg_systemd, path_found, service, keys)
> >                                 else:
> 
> Please check if a similar change is need on master (inside meta-systemd).
> 

[Liu Ting-B28495] the issue was fixed in master.
commit 980b338fb5100ff4c779335a86f3d9450ea8c54a
systemd: Upgrade to 187 tag

a new path has been sent out for denzil.

> --
> Otavio Salvador                             O.S. Systems
> E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
> Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel