diff mbox series

python3-docutils: fix interpreter lines

Message ID GV1PR07MB91203857CFAC6393A256F929A8D22@GV1PR07MB9120.eurprd07.prod.outlook.com
State Accepted, archived
Commit f0037d2b2a4b72f279f3fe60c82d4b4044cbc6d5
Headers show
Series python3-docutils: fix interpreter lines | expand

Commit Message

Konrad Weihmann June 30, 2024, 9:25 a.m. UTC
when compiling a non native variant, latest update
introduced package-qa errors like

ERROR: QA Issue: .../docutils/utils/smartquotes.py contained in package
nativesdk-python3-docutils requires /usr/bin/python3, but no providers
found in RDEPENDS:nativesdk-python3-docutils? [file-rdeps]

Fix that by patching the interpreter lines after install

Signed-off-by: Konrad Weihmann <kweihmann@outlook.com>
---
 meta/recipes-devtools/python/python3-docutils_0.21.2.bb | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Ross Burton July 2, 2024, 1:42 p.m. UTC | #1
On 30 Jun 2024, at 10:25, Konrad Weihmann via lists.openembedded.org <kweihmann=outlook.com@lists.openembedded.org> wrote:
> 
> when compiling a non native variant, latest update
> introduced package-qa errors like
> 
> ERROR: QA Issue: .../docutils/utils/smartquotes.py contained in package
> nativesdk-python3-docutils requires /usr/bin/python3, but no providers
> found in RDEPENDS:nativesdk-python3-docutils? [file-rdeps]
> 
> Fix that by patching the interpreter lines after install

This is a genuine problem with the upstream source, I filed https://sourceforge.net/p/docutils/bugs/491/.  I’d prefer to see a patch instead of a sed, as the patch can be removed when it is no longer needed, whereas the sed will remain there forever.

Ross
diff mbox series

Patch

diff --git a/meta/recipes-devtools/python/python3-docutils_0.21.2.bb b/meta/recipes-devtools/python/python3-docutils_0.21.2.bb
index ac41ef6858..d9f8160913 100644
--- a/meta/recipes-devtools/python/python3-docutils_0.21.2.bb
+++ b/meta/recipes-devtools/python/python3-docutils_0.21.2.bb
@@ -11,4 +11,12 @@  inherit pypi python_setuptools_build_meta
 RDEPENDS:${PN} += " \
                    python3-pprint \
 "
+
+do_install:append() {
+	# Make sure we use /usr/bin/env python3
+	for PYTHSCRIPT in `grep -rIl '^#!.*python' ${D}`; do
+		sed -i -e '1s|^#!.*|#!/usr/bin/env python3|' $PYTHSCRIPT
+	done
+}
+
 BBCLASSEXTEND = "native nativesdk"