[XEN PATCH v2 5/5] README: update to remove old note about the build system's python expectation

Javi Merino posted 5 patches 12 months ago
[XEN PATCH v2 5/5] README: update to remove old note about the build system's python expectation
Posted by Javi Merino 12 months ago
The configure script tests for the availability of python3, python and
python2 in that order and sets PYTHON to the first one found in path.
You don't need to have a symlink to python.

Signed-off-by: Javi Merino <javi.merino@cloud.com>
---
 README | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/README b/README
index f8cc426f78..6212a3a9c5 100644
--- a/README
+++ b/README
@@ -184,13 +184,6 @@ Various tools, such as pygrub, have the following runtime dependencies:
           URL:    https://www.python.org/
           Debian: python
 
-Note that the build system expects `python` to be available. If your system
-only has `python2` or `python3` but not `python` (as in Linux From Scratch),
-you will need to create a symlink for it, or specify PYTHON= when invoking
-make, like (note the position of PYTHON= matters):
-
-    # make PYTHON=/usr/bin/python3
-
 Intel(R) Trusted Execution Technology Support
 =============================================
 
-- 
2.41.0
Re: [XEN PATCH v2 5/5] README: update to remove old note about the build system's python expectation
Posted by Andrew Cooper 12 months ago
On 11/09/2023 5:51 pm, Javi Merino wrote:
> The configure script tests for the availability of python3, python and
> python2 in that order and sets PYTHON to the first one found in path.
> You don't need to have a symlink to python.

I think this was fixed by 5852ca48526316918cd82fba1033a6a5379fbc4c
"build: fix tools/configure in case only python3 exists"

> Signed-off-by: Javi Merino <javi.merino@cloud.com>

Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

I'm happy to fix the commit message on commit.
Re: [XEN PATCH v2 5/5] README: update to remove old note about the build system's python expectation
Posted by Javi Merino 11 months, 3 weeks ago
On Tue, Sep 12, 2023 at 01:25:06PM +0100, Andrew Cooper wrote:
> On 11/09/2023 5:51 pm, Javi Merino wrote:
> > The configure script tests for the availability of python3, python and
> > python2 in that order and sets PYTHON to the first one found in path.
> > You don't need to have a symlink to python.
> 
> I think this was fixed by 5852ca48526316918cd82fba1033a6a5379fbc4c
> "build: fix tools/configure in case only python3 exists"

Indeed! I should have put it in the commit message.

> > Signed-off-by: Javi Merino <javi.merino@cloud.com>
> 
> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
> 
> I'm happy to fix the commit message on commit.

Thanks!