Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update building-setup-windows-cygwin.rst #5474

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chenyuanzhen
Copy link

cygwin waf use /usr/bin/python3 not /usr/bin/python

cygwin waf use /usr/bin/python3 not /usr/bin/python
@Hwurzburg Hwurzburg requested a review from TunaLobster October 10, 2023 11:23
@TunaLobster
Copy link
Contributor

Not bad idea. I haven't had issues with this though. These commands are different/non-existent in the environment setup script. Was there a reason to deviate from what was in the install script? https://github.com/ArduPilot/ardupilot/blob/master/Tools/environment_install/install-prereqs-windows.ps1

@khancyr
Copy link
Contributor

khancyr commented Oct 31, 2023

we should probably nuke the whole instruction and ask to use the script, that would be one less diff to maintain.

otherwise the change look good as we are now correctly supporting python3 .

@Hwurzburg
Copy link
Contributor

@TunaLobster @khancyr need someone to approve before I can merge this

@khancyr khancyr self-assigned this Oct 31, 2023
@chenyuanzhen
Copy link
Author

The system I was using at the time was windows11, and I did use Tools/environment_install/install-prereqs-windows.ps1 for installation. There were no errors during the installation. But when executing the ln -s /usr/bin/python3.7 /usr/bin/pythoncommand, the terminal prompts that the creation failed and /use/bin/python already exists. However, this may be because python2 was already installed on the computer before running install-prereqs-windows.ps1 . As a result, when running the ./waf command, python2 is used, and ultimately fails because all dependencies are installed in python3.
Finally i use ln -sf /usr/bin/python3.7 /usr/bin/python3 command, waf use correct version of python, and all things run ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants