From: Adrian Freihofer <adrian.freiho...@siemens.com>
The keys variable was intended as an array of keys. But it looks like
this has not been used for more than 10 years now. Adding files
automatically to packages needs probably anyway very specific code
rather than a generic loop. Lets simplify this a bit.
Using python code should also not be slower for these usually small
files.
Signed-off-by: Adrian Freihofer <adrian.freiho...@siemens.com>
---
meta/classes-recipe/systemd.bbclass | 28 ++++++++++++----------------
1 file changed, 12 insertions(+), 16 deletions(-)
diff --git a/meta/classes-recipe/systemd.bbclass
b/meta/classes-recipe/systemd.bbclass
index 7324af8555d..be77da48125 100644
--- a/meta/classes-recipe/systemd.bbclass
+++ b/meta/classes-recipe/systemd.bbclass
@@ -124,29 +124,26 @@ python systemd_populate_packages() {
return appended
# Add systemd files to FILES:*-systemd, parse for Also= and follow
recursive
- def systemd_add_files_and_parse(pkg_systemd, path, service, keys):
+ def systemd_add_files_and_parse(pkg_systemd, path, service):
# avoid infinite recursion
if systemd_append_file(pkg_systemd, oe.path.join(path, service)):
fullpath = oe.path.join(d.getVar("D"), path, service)
if service.find('.service') != -1:
# for *.service add *@.service
service_base = service.replace('.service', '')
- systemd_add_files_and_parse(pkg_systemd, path, service_base +
'@.service', keys)
+ systemd_add_files_and_parse(pkg_systemd, path, service_base +
'@.service')
if service.find('.socket') != -1:
# for *.socket add *.service and *@.service
service_base = service.replace('.socket', '')
- systemd_add_files_and_parse(pkg_systemd, path, service_base +
'.service', keys)
- systemd_add_files_and_parse(pkg_systemd, path, service_base +
'@.service', keys)
- for key in keys.split():
- # recurse all dependencies found in keys
('Also';'Conflicts';..) and add to files
- cmd = "grep %s %s | sed 's,%s=,,g' | tr ',' '\\n'" % (key,
shlex.quote(fullpath), key)
- pipe = os.popen(cmd, 'r')
- line = pipe.readline()
- while line:
- line = line.replace('\n', '')
- systemd_add_files_and_parse(pkg_systemd, path, line, keys)
- line = pipe.readline()
- pipe.close()
+ systemd_add_files_and_parse(pkg_systemd, path, service_base +
'.service')
+ systemd_add_files_and_parse(pkg_systemd, path, service_base +
'@.service')
+ # Add all units which have an Also= referring a unit in this
package to this package as well.
+ with open(fullpath, 'r') as unit_f:
+ for line in unit_f:
+ if line.startswith('Also'):
+ also_unit = line.split('=', 1)[1].strip()
+ bb.warn("also: %s" % also_unit)
+ systemd_add_files_and_parse(pkg_systemd, path,
also_unit)
# Check service-files and call systemd_add_files_and_parse for each entry
def systemd_check_services():
@@ -155,7 +152,6 @@ python systemd_populate_packages() {
searchpaths.append(d.getVar("systemd_user_unitdir"))
systemd_packages = d.getVar('SYSTEMD_PACKAGES')
- keys = 'Also'
# scan for all in SYSTEMD_SERVICE[]
for pkg_systemd in systemd_packages.split():
for service in get_package_var(d, 'SYSTEMD_SERVICE',
pkg_systemd).split():
@@ -179,7 +175,7 @@ python systemd_populate_packages() {
break
if path_found != '':
- systemd_add_files_and_parse(pkg_systemd, path_found,
service, keys)
+ systemd_add_files_and_parse(pkg_systemd, path_found,
service)
else:
bb.fatal("Didn't find service unit '{0}', specified in
SYSTEMD_SERVICE:{1}. {2}".format(
service, pkg_systemd, "Also looked for service unit
'{0}'.".format(base) if base is not None else ""))
--
2.47.0
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#208614):
https://lists.openembedded.org/g/openembedded-core/message/208614
Mute This Topic: https://lists.openembedded.org/mt/110070404/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
