Skip to contentSkip to conversation

meson: fix error when restorecon unavailable

PATCH New Fabrice Fontaine 2 participants 2 messages Mar 13, 2018 – Mar 31, 2018 Arm/Amlogic Meson SoC support

lore
Share & tools
lore

Fabrice Fontaine

author
#0 lore raw

PATCH v2 1/1 meson: fix error when restorecon unavailable

host-meson is used by some packages such as libmpdclient. If selinuxenabled is installed on host but restorecon is unavailable (for an "unknwown" reason), install will crash.

Fixes:
- http://autobuild.buildroot.net/results/d5dcdfdfab3503fdc387f99e68267972a38c417d

Signed-off-by Fabrice Fontaine <[email protected]>

---
Changes v1->v2:
- Update patch following upstream review: https://github.com/mesonbuild/meson/pull/3236#pullrequestreview-103570704

 ...NotFoundError-when-restorecon-unavailable.patch | 31 ++++++++++++++++++++++
 1 file changed, 31 insertions(+)
 create mode 100644 package/meson/0001-Fix-FileNotFoundError-when-restorecon-unavailable.patch
package/meson/0001-Fix-FileNotFoundError-when-restorecon-unavailable.patch new file +31 -0
new file mode 100644
index 0000000000..3f1ca68dfc
@@ -0,0 +1,31 @@
+From 8edc477b7d7b160004eb700a1b7523237333d9ec Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <[email protected]>
+Date: Tue, 13 Mar 2018 18:52:36 +0100
+Subject: [PATCH] Fix FileNotFoundError when restorecon unavailable
+
+Fixes:
+ - http://autobuild.buildroot.net/results/d5dcdfdfab3503fdc387f99e68267972a38c417d
+
+Signed-off-by: Fabrice Fontaine <[email protected]>
+---
+ mesonbuild/scripts/meson_install.py | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/mesonbuild/scripts/meson_install.py b/mesonbuild/scripts/meson_install.py
+index f895f17d..1414ace2 100644
+--- a/mesonbuild/scripts/meson_install.py
++++ b/mesonbuild/scripts/meson_install.py
+@@ -97,6 +97,10 @@ def restore_selinux_contexts():
+         # is ignored quietly.
+         return
+ 
++    if not shutil.which('restorecon'):
++        # If we don't have restorecon, failure is ignored quietly.
++        return
++
+     with subprocess.Popen(['restorecon', '-F', '-f-', '-0'],
+                           stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE) as proc:
+         out, err = proc.communicate(input=b'\0'.join(os.fsencode(f) for f in selinux_updates) + b'\0')
+-- 
+2.14.1
+
Signature · 1 line
2.14.1
Applied
#1 lore raw

PATCH v2 1/1 meson: fix error when restorecon unavailable

Hello,

Quoted 11 lines · Fabrice Fontaine

host-meson is used by some packages such as libmpdclient. If selinuxenabled is installed on host but restorecon is unavailable (for an "unknwown" reason), install will crash.

Fixes:
- http://autobuild.buildroot.net/results/d5dcdfdfab3503fdc387f99e68267972a38c417d

Signed-off-by Fabrice Fontaine <[email protected]>

---
Changes v1->v2:
- Update patch following upstream review: https://github.com/mesonbuild/meson/pull/3236#pullrequestreview-103570704

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com