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.fish: set IDF_PATH without changing current working directory (IDFGH-8140) #9637

Merged
merged 1 commit into from Sep 5, 2022

Conversation

imcaizheng
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Aug 25, 2022

CLA assistant check
All committers have signed the CLA.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Aug 25, 2022
@github-actions github-actions bot changed the title export.fish: set IDF_PATH without changing current working directory export.fish: set IDF_PATH without changing current working directory (IDFGH-8140) Aug 25, 2022
@dobairoland
Copy link
Collaborator

Thank you for the contribution. We'll take a look at this soon.

@imcaizheng
Copy link
Contributor Author

I just found that my initial commit didn't take care of the case when the source command is executed under the root directory of esp-idf. The last commit I uploaded should fix it.

export.fish Outdated Show resolved Hide resolved
@dobairoland
Copy link
Collaborator

sha=84ec01c3301c33ec9a61ff03ab0703c653d39180

@dobairoland dobairoland added the PR-Sync-Merge Pull request sync as merge commit label Aug 31, 2022
@espressif-bot espressif-bot added Status: In Progress Work is in progress Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Opened Issue is new Status: In Progress Work is in progress Resolution: NA Issue resolution is unavailable labels Aug 31, 2022
@espressif-bot espressif-bot merged commit 84ec01c into espressif:master Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Sync-Merge Pull request sync as merge commit Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sourcing export.fish changes current working directory unexpectedly on fish shell (IDFGH-8146)
5 participants