AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Proposal to remove pystring dependency

Aperta
#2,256 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
dependencies Modernise (C++17) Needs Discussion
Lingua principale
C++
Stelle
2.1k
Fork
503
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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:

1) 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.
2) Replace filesystem interaction by calls to std::filesystem (C++17)
3) Remove all pystring from CMake

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia individuando gli usi esistenti di pystring e separando la manipolazione delle stringhe dall’interazione con il filesystem. Esamina StringUtils e i relativi test di split(), quindi valuta le modifiche proposte a std::filesystem e CMake. Il lavoro è completo quando le tre fasi di sostituzione sono state completate e pystring è stato rimosso da CMake.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cmake, cpp
Ambito
build-system
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Tranquilla
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.