Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

衝突するconfigureオプションをグループ化する #81

Closed
ma8ma opened this issue Jun 8, 2019 · 1 comment
Closed

衝突するconfigureオプションをグループ化する #81

ma8ma opened this issue Jun 8, 2019 · 1 comment
Labels
feature 機能の追加と削除

Comments

@ma8ma
Copy link
Collaborator

ma8ma commented Jun 8, 2019

衝突する(どれか一つしか指定できない)configureオプションをグループ化することを提案します。
グループ化は--with-foo=[baz|qux]のように指定のオプション値(bazqux)を選択させます。

長所と短所

  • 👍 衝突関係がはっきりする
  • 👍 オプションの数が減り分かりやすくなる
  • 👎 オプション値を指定するときにシェル補完を当てにできない
  • 👎 オプション値ごとにヘルプメッセージが表示できない

グループ化の手順案

新オプションの導入と旧オプションの廃止予定を進めます。

  1. グループ化された新オプションをconfigureスクリプトに実装する。
  2. configureスクリプトやドキュメントを修正して旧オプションが廃止予定になったことを通知する。代替手段として新オプションを指定する方法を示す。
  3. 次期バージョンのリリース後に旧オプションを削除する。

グループ化するconfigureオプション案

現状 新オプション 指定しない場合(デフォルト)
--with-gthread
--with-stdthread
--with-thread=[posix|glib|std] posix
--with-oniguruma
--with-pcre
--with-regex=[posix|oniguruma|pcre] posix
--with-openssl --with-tls=[gnutls|openssl] gnutls
エラーとする値

グループに入ってない値(e.g. --with-thread=foobar)や値なしオプション(e.g. --with-thread)はエラーとします。

@ma8ma ma8ma added the feature 機能の追加と削除 label Jun 8, 2019
@ma8ma ma8ma added this to To do in State of JDim-v0.1.0+ via automation Jun 8, 2019
@yama-natuki
Copy link
Collaborator

いいと思います。

@ma8ma ma8ma moved this from To do to In progress in State of JDim-v0.1.0+ Jun 9, 2019
State of JDim-v0.1.0+ automation moved this from In progress to Done Jun 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 機能の追加と削除
Projects
No open projects
Development

No branches or pull requests

2 participants