AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Proposal to remove pystring dependency

Ouverte
#2,256 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
dependencies Modernise (C++17) Needs Discussion
Langage dominant
C++
Étoiles
2.1k
Forks
503
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par localiser les utilisations existantes de pystring et séparer la manipulation des chaînes de l’interaction avec le système de fichiers. Examinez StringUtils et ses tests de split(), puis évaluez les modifications proposées pour std::filesystem et CMake. Le travail est terminé lorsque les trois étapes de remplacement sont terminées et que pystring a été supprimé de CMake.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cmake, cpp
Domaine
build-system
Type d'issue
Refactorisation
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
Calme
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.