AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Proposal to remove pystring dependency

オープン
#2,256 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
dependencies Modernise (C++17) Needs Discussion
主要言語
C++
スター
2.1k
フォーク
503
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず、既存の pystring の使用箇所を特定し、文字列操作とファイルシステムとのやり取りを分離します。StringUtils とその split() のテストを確認し、続いて提案されている std::filesystem と CMake の変更を評価します。3 段階の置き換えが完了し、CMake から pystring が削除されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cmake, cpp
領域
build-system
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
静か
明瞭さ
説明が足りない
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。