google / google/codesearch

Fix temporary file deletion not working on Windows

Offen
#82 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
4k
Forks
403
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

See [this issue](https://github.com/junkblocker/codesearch/issues/13).

The Windows version of codesearch is broken because it doesn't properly close any of its file handles before it tries to remove the files.

My fork has [this change](https://macoy.me/code/macoy/codesearch/commit/c5f1d5ebd4e48099be0970402d5218c9fd7ac1e8) which fixes this. My fork has diverged (I added a `tempDir` argument so that I could move the temp files where I wanted them), but someone should be able to port over my changes without too much issue.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit dem Lesen von Issue 13 und vergleiche den Fork-Commit c5f1d5ebd4e48099be0970402d5218c9fd7ac1e mit der aktuellen Go-Implementierung. Verfolge den Lebenszyklus der temporären Dateien und ermittle, wo Datei-Handles vor dem Löschen offen bleiben. Als erledigt gilt die Aufgabe, wenn die Windows-Version ihre Handles vor dem Entfernen temporärer Dateien schließt, ohne die unabhängigen tempDir-Änderungen des Forks zu übernehmen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
cli
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.