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

export.bat IDF_PATH (IDFGH-6258) #7927

Closed
pius-braun opened this issue Nov 18, 2021 · 2 comments
Closed

export.bat IDF_PATH (IDFGH-6258) #7927

pius-braun opened this issue Nov 18, 2021 · 2 comments
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@pius-braun
Copy link

In export.bat, I had to move the statements:

:: Infer IDF_PATH from script location
set IDF_PATH=%~dp0
set IDF_PATH=%IDF_PATH:~0,-1%

from line 24 to line 17, i.e. above the statement "set PREFIX=python.exe %IDF_PATH%"
This avoids an error, when running "idf.py build".
The error message was "/tools/idf.py not found".

@espressif-bot espressif-bot added the Status: Opened Issue is new label Nov 18, 2021
@github-actions github-actions bot changed the title export.bat IDF_PATH export.bat IDF_PATH (IDFGH-6258) Nov 18, 2021
@dobairoland
Copy link
Collaborator

I believe that this is the same issue as #7605. It was fixed together with a larger fix in the master branch but it hasn't been backported. So thank you @pius-braun for raising the issue. I assume that you use v4.3.x or an earlier version. We will fix this ASAP.

espressif-bot pushed a commit that referenced this issue Nov 24, 2021
Also includes fix for DOSKEY definitions.
Closes #7605
Closes #7927
@dobairoland
Copy link
Collaborator

This has been fixed in v4.3 branch as well. The fix for v4.2 and v4.1 will come soon. The fixing commits will be linked here.

@espressif-bot espressif-bot added Resolution: Done Issue is done internally Status: Done Issue is done internally and removed Status: Opened Issue is new labels Nov 26, 2021
espressif-bot pushed a commit that referenced this issue Nov 29, 2021
Also includes fix for DOSKEY definitions.
Closes #7605
Closes #7927
espressif-bot pushed a commit that referenced this issue Nov 29, 2021
Also includes fix for DOSKEY definitions.
Closes #7605
Closes #7927
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

3 participants