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

Incomplete Git Repository Detection (IDFGH-12856) #13345

Closed
3 tasks done
jonahbron opened this issue Mar 10, 2024 · 2 comments
Closed
3 tasks done

Incomplete Git Repository Detection (IDFGH-12856) #13345

jonahbron opened this issue Mar 10, 2024 · 2 comments
Assignees
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Type: Bug bugs in IDF

Comments

@jonahbron
Copy link

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.2.1

Espressif SoC revision.

n/a

Operating System used.

Linux

How did you build your project?

Command line with Make

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

None

Development Kit.

n/a

Power Supply used.

USB

What is the expected behavior?

n/a

What is the actual behavior?

n/a

Steps to reproduce.

  1. Build this Nix flake: https://github.com/mirrexagon/nixpkgs-esp-dev

Debug Logs.

No response

More Information.

The way in which esp.py detects whether it's in a git repository for version detection appears to pick up non-esp-idf repositories that it might happen to exist inside of.

mirrexagon/nixpkgs-esp-dev#5

This causes secondary issues for Nix where the derivation isolation collides with Git's attempts at ensuring the ownership of the Git repo is what is expected.

mirrexagon/nixpkgs-esp-dev#39

@jonahbron jonahbron added the Type: Bug bugs in IDF label Mar 10, 2024
@dobairoland dobairoland changed the title Incomplete Git Repository Detection Incomplete Git Repository Detection May 20, 2024
@dobairoland dobairoland changed the title Incomplete Git Repository Detection Incomplete Git Repository Detection May 20, 2024
@espressif-bot espressif-bot added the Status: Opened Issue is new label May 20, 2024
@github-actions github-actions bot changed the title Incomplete Git Repository Detection Incomplete Git Repository Detection (IDFGH-12856) May 20, 2024
@dobairoland
Copy link
Collaborator

We plan to remove the use of git for detecting ESP-IDF version. That will solve this issue as well.

to pick up non-esp-idf repositories that it might happen to exist inside of.

But I don't see why this should be necessary. ESP-IDF directory should be left as is and everything else can be added into the project's directory (which can be anywhere else on the disk).

@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development Status: In Progress Work is in progress and removed Status: Opened Issue is new Status: Selected for Development Issue is selected for development labels May 27, 2024
@espressif-bot espressif-bot added Status: Reviewing Issue is being reviewed and removed Status: In Progress Work is in progress labels Jun 6, 2024
@dobairoland
Copy link
Collaborator

Change of plans. We have to keep the use of git but will try to ignore all git related failures.

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

No branches or pull requests

4 participants