AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Proposal to remove pystring dependency

Abierto
#2,256 2 comentarios 0 reacciones 0 asignados Ver en GitHub
dependencies Modernise (C++17) Needs Discussion
Lenguaje dominante
C++
Estrellas
2.1k
Forks
503
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza localizando los usos existentes de pystring y separando la manipulación de cadenas de la interacción con el sistema de archivos. Revisa StringUtils y sus pruebas de split(), y después evalúa los cambios propuestos en std::filesystem y CMake. Se considera terminado cuando se hayan completado las tres etapas de reemplazo y pystring se haya eliminado de CMake.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
cmake, cpp
Área
build-system
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Tranquilo
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.