Skip to content

Commit

Permalink
Update gitoxide to version with lenient tag parsing
Browse files Browse the repository at this point in the history
Summary:
ignore-conflict-markers

A while ago, I made the parsing of tags in gitoxide tolerate a missing
timestamp.

We have now seen this situation in two repos:
* aosp/platform/external/brctl
* aosp/platform/external/iproute2

Update all gitoxide crates to their latest version (equivalent to upstream main)
to propagate that fix to us.

I had to list all gitoxide crates explicitely to avoid issues with conflicting requirements
for any crates in the dependency chain.

The commit I'm pointing at in my fork of gitoxide is litterally `main` plus a version update
to propagate the fix throughout the gix ecosystem.

NOTE: I had to slightly edit the integration test to make the failure
representative of the incorrect parsing, but see the updated
integration test for proof that the latest version can parse a tag that's
missing a timestamp.

Reviewed By: RajivTS, singhsrb

Differential Revision: D63771328

fbshipit-source-id: fdbc9d762bacf48c7b17615181371cd92a4a921f
  • Loading branch information
Pierre Chevalier authored and facebook-github-bot committed Oct 3, 2024
1 parent eee9fb2 commit 2afb6ba
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion reverie-process/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ libc = "0.2.139"
nix = "0.25"
serde = { version = "1.0.185", features = ["derive", "rc"] }
syscalls = { version = "0.6.7", features = ["serde"] }
thiserror = "1.0.49"
thiserror = "1.0.64"
tokio = { version = "1.37.0", features = ["full", "test-util", "tracing"] }

[dev-dependencies]
Expand Down
2 changes: 1 addition & 1 deletion reverie-ptrace/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ raw-cpuid = "10.6.0"
reverie = { version = "0.1.0", path = "../reverie" }
safeptrace = { version = "0.1.0", path = "../safeptrace", features = ["memory", "notifier"] }
serde = { version = "1.0.185", features = ["derive", "rc"] }
thiserror = "1.0.49"
thiserror = "1.0.64"
tokio = { version = "1.37.0", features = ["full", "test-util", "tracing"] }
tokio-stream = { version = "0.1.14", features = ["fs", "io-util", "net", "signal", "sync", "time"] }
tracing = { version = "0.1.40", features = ["attributes", "valuable"] }
Expand Down
2 changes: 1 addition & 1 deletion reverie/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -27,5 +27,5 @@ reverie-process = { version = "0.1.0", path = "../reverie-process" }
reverie-syscalls = { version = "0.1.0", path = "../reverie-syscalls" }
serde = { version = "1.0.185", features = ["derive", "rc"] }
syscalls = { version = "0.6.7", features = ["serde"] }
thiserror = "1.0.49"
thiserror = "1.0.64"
typed-arena = "2"
2 changes: 1 addition & 1 deletion safeptrace/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ parking_lot = { version = "0.12.1", features = ["send_guard"] }
reverie-memory = { version = "0.1.0", path = "../reverie-memory", optional = true }
reverie-process = { version = "0.1.0", path = "../reverie-process" }
syscalls = { version = "0.6.7", features = ["serde"] }
thiserror = "1.0.49"
thiserror = "1.0.64"

[dev-dependencies]
quickcheck = "1.0"
Expand Down

0 comments on commit 2afb6ba

Please sign in to comment.