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

Python path case sensitivity issue on Windows (IDFGH-11666) #12779

Closed
3 tasks done
dyarkovoy opened this issue Dec 12, 2023 · 2 comments
Closed
3 tasks done

Python path case sensitivity issue on Windows (IDFGH-11666) #12779

dyarkovoy opened this issue Dec 12, 2023 · 2 comments
Assignees
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Type: Bug bugs in IDF

Comments

@dyarkovoy
Copy link
Contributor

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.

v5.3-dev-888-gcb66d6f6fa

Espressif SoC revision.

all

Operating System used.

Windows

How did you build your project?

Command line with idf.py

If you are using Windows, please specify command line type.

CMD

Development Kit.

all

Power Supply used.

USB

What is the expected behavior?

idf.py does not complain about wrong python path when building project

What is the actual behavior?

When the path of python.exe recorded in project's build config is case-different on a case-insensitive filesystem (Windows), this error occurs:

'c:\work\esp32\.espressif\python_env\idf5.3_py3.11_env\Scripts\python.exe' is currently active in the environment while the project was configured with 'C:\work\esp32\.espressif\python_env\idf5.3_py3.11_env\Scripts\python.exe'. Run 'idf.py fullclean' to start again.

the C: drive is identical to c: drive on Windows, but this is not taken into account

Steps to reproduce.

  1. Build any ESP-IDF project in VSCode with vscode-esp-idf-extension on Windows
  2. Try to build the same project from command line using idf.py build
    you should see the error as described above
    ...

Debug Logs.

No response

More Information.

I will submit PR correcting this behavior shortly

@dyarkovoy dyarkovoy added the Type: Bug bugs in IDF label Dec 12, 2023
dyarkovoy added a commit to dyarkovoy/esp-idf that referenced this issue Dec 12, 2023
@github-actions github-actions bot changed the title Python path case sensitivity issue on Windows Python path case sensitivity issue on Windows (IDFGH-11666) Dec 12, 2023
@espressif-bot espressif-bot added the Status: Opened Issue is new label Dec 12, 2023
@dobairoland
Copy link
Collaborator

@jakub-kocka PTAL

@dobairoland dobairoland linked a pull request Dec 13, 2023 that will close this issue
@jakub-kocka
Copy link
Collaborator

@dyarkovoy, thanks for submitting also the PR which solves this.
I haven't been able to reproduce this issue but with the fix everything works as expected.

As you said, according to documentation and my tests the string path is changed only on Windows platform.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new labels Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Type: Bug bugs in IDF
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants