summaryrefslogtreecommitdiff
path: root/external/icu/no-python.patch
AgeCommit message (Collapse)Author
2024-04-23external/icu: Drop hidden dependency on PythonStephan Bergmann
...which was always satisfied directly from the system, regardless of our --enable-python=... setting. (And which was a problem for my Git Bash and WSL based Windows build attempt, where it happened to find a dysfunctional Python wrapper at C:/Users/steph/AppData/Local/Microsoft/WindowsApps/python3 which caused a [...] > Not rebuilding data/rules.mk, assuming prebuilt data in data/in > Spawning Python to generate test/testdata/rules.mk... > Python was not found; run without arguments to install from the Microsoft Store, or disable this shortcut from Settings > Manage App Execution Aliases. > configure: error: Python failed to run; see above error. error when building ExternalProject_icu.) There are three uses of that PYTHON setting across workdir/UnpackedTarget/icu: 1 In source/configure.ac to generate source/data/rules.mk if source/data/locales/root.txt would exist---but which doesn't, so we don't actually need PYTHON there. 2 In source/configure.ac to generate source/test/testdata/rules.mk, but which our (non-check) build apparently doesn't need anyway. 3 In source/data/Makefile.in for target check-local, which is a sub-target of check (which we don't build). Change-Id: I7455cc91fc67f36582bf54851c07030830cd3b8d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/166500 Reviewed-by: Stephan Bergmann <stephan.bergmann@allotropia.de> Tested-by: Jenkins