Checkout with modified but not dirty submodules

Offen

#1,150 erstellt am 09.7.2015 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 38/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: csharp, git
Bereich: devtools

Rechercherichtung

Keine Quelldatei oder kein Test ist angegeben. Beginne damit, das Checkout-Verhalten mit einem Submodul zu reproduzieren, dessen Indexzeiger abweicht, und einem, dessen Inhalt dirty ist, und verfolge anschließend die Fehlerbehandlung von libgit2sharp beim Checkout. Erledigt ist die Aufgabe, wenn Änderungen, die nur den Zeiger betreffen, kein force erfordern, während dirty Submodule weiterhin geschützt bleiben und ihre zugrunde liegenden Fehler erhalten bleiben.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Version : vNext & master
Native : 1.0.72 - git2-4d6362b

There are two types of modified submodules, one that shows as modified due to a index pointer(SHA) mismatch, and one due to the submodule's content actually being changed/modified. Git knows the different and does not require a 'checkout -force'. or a 'git submodule update' on each submodule before doing a checkout, but libgit2sharp/libgit2 does require the checkout to be forced.

Is this the intended/designed result with modified/index and modified/dirty submodules? Or a bug?

It is a thrown exception from libgit2, so I coded around it with a exception bound helper extension to determine if the errors are related to submodules and to go determine if just the index changed and thus it is safe to flip the force option on or if the submodules are dirty and what are the 'real' exceptions within each submodule and throw those. It just seems like this should be one in one the libraries like it is within git (IMHO)...

This will not be a forced git checkout:

diff --git a/external/cecil b/external/cecil
index 33d50b8..fc76b93 160000
--- a/external/cecil
+++ b/external/cecil
@@ -1 +1 @@
-Subproject commit 33d50b874fd527118bc361d83de3d494e8bb55e1
+Subproject commit fc76b93e3e0064b2d751796878f9cbe88df1d5f4

This would be a forced git checkout:

diff --git a/external/rx b/external/rx
index b29a4b0..17e8477 160000
--- a/external/rx
+++ b/external/rx
@@ -1 +1 @@
-Subproject commit b29a4b0fda609e0af33ff54ed13652b6ccf0e05e
-Subproject commit b29a4b0fda609e0af33ff54ed13652b6ccf0e05e-dirty

Vorherrschende Sprache: C#
Sterne: 3.5k
Forks: 925
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus libgit2/libgit2sharp

Potential AccessViolation at process exit on .NET Framework - SafeHandle finalizers now race `git_libgit2_shutdown` Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100

libgit2/libgit2sharp#2193 · 2 Kommentare · Erstellt am 01.9.2026
AccessViolationException accessing Commit.Parents concurrently on Linux .NET 10 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100

libgit2/libgit2sharp#2192 · 1 Kommentar · Erstellt am 01.9.2026
Website is down Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 20/100

libgit2/libgit2sharp#2191 · 2 Reaktionen · Erstellt am 20.8.2026
Indexing a tree by path yields different result with trailing slash Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

libgit2/libgit2sharp#2189 · 1 Reaktion · Erstellt am 04.8.2026
Running from Native AOT app fails to build signature Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

libgit2/libgit2sharp#2187 · 2 Kommentare · Erstellt am 22.7.2026

Alle Issues in libgit2/libgit2sharp