Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Incorrect formatting for git fetch #13968

Closed
kali opened this issue May 27, 2024 · 2 comments · Fixed by #13969
Closed

Incorrect formatting for git fetch #13968

kali opened this issue May 27, 2024 · 2 comments · Fixed by #13969
Labels
A-git Area: anything dealing with git C-bug Category: bug regression-from-stable-to-nightly Regression in nightly that previously worked in stable.

Comments

@kali
Copy link

kali commented May 27, 2024

Problem

Problem appeared 2 days ago running CI against rust nightly for tract.

See: https://github.com/sonos/tract/actions/runs/9251978441/job/25448634660

Same CI is still working on both stable and beta.

Steps

git clone https://github.com/sonos/tract
cd tract
CARGO_NET_GIT_FETCH_WITH_CLI=true cargo +nightly -vvv  test -p test-f16
    Updating crates.io index
    Updating git repository `https://github.com/kali/tflitec-rs.git`
     Running `git fetch --verbose --force --update-head-ok 'https://github.com/kali/tflitec-rs.git' '+b37d65a000000000000000000000000000000000:refs/commit/b37d65a000000000000000000000000000000000'`
POST git-upload-pack (102 bytes)
POST git-upload-pack (167 bytes)
fatal: remote error: upload-pack: not our ref b37d65a000000000000000000000000000000000
error: failed to get `tflitec` as a dependency of package `test-f16 v0.1.0 (/tract/test-rt/test-f16)`

Caused by:
  failed to load source for dependency `tflitec`

Caused by:
  Unable to update https://github.com/kali/tflitec-rs.git?rev=b37d65a

Caused by:
  failed to fetch into: /root/.cargo/git/db/tflitec-rs-2d4221cf6acd5dab

Caused by:
  process didn't exit successfully: `git fetch --verbose --force --update-head-ok 'https://github.com/kali/tflitec-rs.git' '+b37d65a000000000000000000000000000000000:refs/commit/b37d65a000000000000000000000000000000000'` (exit status: 128)

I had to run this on a clean docker container though. On my regular system some cache was masking the issue.

Possible Solution(s)

No response

Notes

Probably related to #13946

See comment #13946 (review) maybe.

Issue appears with or without CARGO_NET_GIT_FETCH_WITH_CLI=true , but it makes it very visible as cargo will show git command line call.

Version

No response

@kali kali added C-bug Category: bug S-triage Status: This issue is waiting on initial triage. labels May 27, 2024
@weihanglo weihanglo added A-git Area: anything dealing with git regression-from-stable-to-nightly Regression in nightly that previously worked in stable. and removed S-triage Status: This issue is waiting on initial triage. labels May 27, 2024
@weihanglo
Copy link
Member

My apologies. Forgot that libgit2 will zero-pad revisions. A fix is up: #13969.

taiki-e added a commit to taiki-e/atomic-maybe-uninit that referenced this issue May 28, 2024
rust-lang/cargo#13968

```
fatal: remote error: upload-pack: not our ref afa43cc000000000000000000000000000000000
```
@weihanglo
Copy link
Member

The fix is now available on nightly-2024-05-29 toolchain.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-git Area: anything dealing with git C-bug Category: bug regression-from-stable-to-nightly Regression in nightly that previously worked in stable.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants