almalinux/kiwi-el8
14
0
Fork 0
Code Issues Pull Requests Releases Activity

Fixed debian bootstrap script calls

Browse Source 
Run scripts as commands with their native shebang and not
through bash. Not all debian package scripts uses bash, some
of them uses sh which can be a link to dash or other
interpreters. This Fixes #2660
This commit is contained in:
Marcus Schäfer 2024-10-17 20:53:40 +02:00
parent 0e24761929
commit cbdddefcff
No known key found for this signature in database
GPG Key ID: A16C1128698C8CAC
2 changed files with 40 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
kiwi/package_manager/apt.py
View File

@ -557,17 +557,19 @@ class PackageManagerApt(PackageManagerBase):
script_post = f'{package_metadata_dir}/postinst'
# 1. preinst
if os.path.exists(script_pre):
Command.run(['chmod', '755', script_pre])
Command.run(
[
'chroot', self.root_dir, 'bash',
'chroot', self.root_dir,
f'{script_pre.replace(self.root_dir, "")}', 'install'
], self.command_env
)
# 2. postinst
if os.path.exists(script_post):
Command.run(['chmod', '755', script_post])
Command.run(
[
'chroot', self.root_dir, 'bash',
'chroot', self.root_dir,
f'{script_post.replace(self.root_dir, "")}', 'configure'
], self.command_env
)

42
test/unit/package_manager/apt_test.py
View File

@ -182,13 +182,23 @@ class TestPackageManagerApt:
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/base-passwd/preinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/base-passwd/preinst', 'install'
], new_env
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/base-passwd/postinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/base-passwd/postinst', 'configure'
], new_env
),
@ -197,13 +207,23 @@ class TestPackageManagerApt:
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/base-passwd/preinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/base-passwd/preinst', 'install'
], new_env
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/base-passwd/postinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/base-passwd/postinst', 'configure'
], new_env
),
@ -212,13 +232,23 @@ class TestPackageManagerApt:
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/vim/preinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/vim/preinst', 'install'
], new_env
),
call(
[
'chroot', 'root-dir', 'bash',
'chmod', '755', 'tempdir/vim/postinst'
]
),
call(
[
'chroot', 'root-dir',
'tempdir/vim/postinst', 'configure'
], new_env
)