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

Error reverting changes in the checkouts directory #27

Closed
fredpi opened this issue Apr 20, 2019 · 1 comment · Fixed by #40
Closed

Error reverting changes in the checkouts directory #27

fredpi opened this issue Apr 20, 2019 · 1 comment · Fixed by #40
Labels
bug Something isn't working

Comments

@fredpi
Copy link
Contributor

fredpi commented Apr 20, 2019

In my current configuration (with only 1 package), I somehow managed to break the ✨ Reverting any changes in the checkouts directory ... step. The following problem occurs: An error occurred: Command '/bin/bash -c "git -C '/[MyProjectPath]/.accio/checkouts/PMJSON' reset HEAD --hard --quiet 2> /dev/null"' returned with error code 128.

When I copy this repo to another folder and run the failing command (git -C '[NewPath]/PMJSON' reset HEAD --hard) manually, there's still an error: fatal: Could not parse object 'HEAD'.

Maybe this problem is related to the fact that after initializing accio and then switching to a branch where it wasn't installed yet, I performed a Reset All of the original repo via Sourcetree to reset the .accio folder. Maybe this had an effect on the recursive subrepos (those within .accio/checkouts), so that after switching to the accio branch again, running accio install failed. Still weird, that the checkouts folder survived the git reset... 🤔

@Jeehut Jeehut added the bug Something isn't working label Apr 20, 2019
@Jeehut
Copy link
Contributor

Jeehut commented Apr 24, 2019

I saw this issue once, too in an app project. So it's definitely happening often enough, that I will have a detailed look once I come across it again. Unfortunately I didn't have much time when it happened to I worked around it by running accio clean once and accio update after that. Then everything worked without issues.

I will also checkout your example ZIP file and try to reproduce this once I have time. Thank you for reporting this and the detailed description.

@Jeehut Jeehut closed this as completed in #40 May 9, 2019
Jeehut added a commit that referenced this issue May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants