Proposal to remove pystring dependency
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Refactoring
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- cmake, cpp
- Bereich
- build-system
Rechercherichtung
Beginne damit, die bestehenden Verwendungen von pystring zu lokalisieren und die String-Manipulation von der Interaktion mit dem Dateisystem zu trennen. Überprüfe StringUtils und die Tests für split(), und bewerte anschließend die vorgeschlagenen Änderungen an std::filesystem und CMake. Die Aufgabe ist abgeschlossen, wenn die drei Ersetzungsphasen vollständig sind und pystring aus CMake entfernt wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Our current usage of pystring falls into 2 camps
String manipulation e.g. split, startswith, endswith, splitlines, lstrip etc
Filesystem interaction pystring::os::*
I would propose that this be carried out in 3 stages:
- Replace string functionality with code based on our own StringUtils (add functionality where needed). We already have some tests for replacing the split() behaviour for example.
- Replace filesystem interaction by calls to std::filesystem (C++17)
- Remove all pystring from CMake
- Vorherrschende Sprache
- C++
- Sterne
- 2.1k
- Forks
- 505
- Ø Merge
- 12 Std. 24 Min.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
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 AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 Kommentare ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
Documentation
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 Kommentar ·
Alle Issues in AcademySoftwareFoundation/OpenColorIO
Ähnliche Issues
-
Component - Build
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug-unconfirmed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
wazuh/wazuh#40870 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
tenstorrent/tt-metal#63198 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Extended search mode does not find escape sequences (\t, \n, \r...)Evtl. vergeben @CyrilB1531 hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 87/100
dail8859/NotepadNext#1181 ·
Maintainer antworten meist innerhalb von 1 Tag