diff mbox series

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

Message ID 2b98d4409f0220c5b24517c5b230344228717eb8.1694450145.git.javi.merino@cloud.com (mailing list archive)
State New, archived
Headers show
Series python: Use setuptools instead of the deprecated distutils | expand

Commit Message

Javi Merino Sept. 11, 2023, 4:51 p.m. UTC
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(-)

Comments

Andrew Cooper Sept. 12, 2023, 12:25 p.m. UTC | #1
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.
Javi Merino Sept. 18, 2023, 8:25 p.m. UTC | #2
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!
diff mbox series

Patch

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
 =============================================