操作
リファクタ #173
完了build.gradle の古い localLibs exclude とローカル絶対パスを整理する
リファクタ #173:
build.gradle の古い localLibs exclude とローカル絶対パスを整理する
説明
Redmine #165 のコードベースレビューからの派生タスクです。
背景¶
build.gradle に古い設定や開発者ローカル環境依存の値があります。
対応内容¶
-
localLibsのfileTreeexclude にある、現在のlib/に存在しない jar 名を整理する。 - Maven 依存へ移行済みの名残であれば削除する。
-
verifyFfmpegDecodeの既定引数D:/rei-no-are/010/010_BGA_256_no_sound.mpgを、環境依存しない指定方法に変更する。 - 必要に応じてタスクの使い方をコメントまたはドキュメント化する。
確認観点¶
-
./gradlew.bat testまたは関連 Gradle タスクが従来通り動作すること。 - ローカルに検証用動画が無い環境でも Gradle 設定評価が壊れないこと。
いづ さん さんが25日前に更新
- 関連している リファクタ #165: コードベースの棚卸 を追加
エージェント ちゃん さんが11日前に更新
- ステータス を 未着手 から 実装中 に変更
- 進捗率 を 0 から 10 に変更
実装に着手しました。
- ブランチ:
feature/173 - 対応方針:
build.gradleのlocalLibsexclude を Maven 移行済みの名残として削除し、verifyFfmpegDecodeの環境依存な絶対パス既定値を撤廃して-PbeatorajaFfmpegSmokeFileを実行時必須化する。
エージェント ちゃん さんが11日前に更新
- ステータス を 実装中 から 完了 に変更
- 進捗率 を 10 から 100 に変更
対応が完了しました。
概要¶
build.gradle に残っていた古い設定・開発者ローカル環境依存の値を整理しました(#165 派生)。振る舞いは維持しつつ、移行済みの名残と絶対パス埋め込みを除去しています。
変更点¶
-
localLibs fileTreeの exclude リスト(commons-dbutils.jar/sqlite-jdbc.jar/jbms-parser.jar/jbmstable-parser.jar/ffmpeg-windows-*.jar/gdx*.jar)を削除。いずれも Maven 依存へ移行済みで現在のlib/には存在せず(実体は jflac-codec / luaj-jse / tarsosdsp の3つの vendored jar のみ)、fileTreeの解決結果は変更前後で同一。 -
verifyFfmpegDecodeの環境依存な既定引数D:/rei-no-are/010/010_BGA_256_no_sound.mpgを撤廃。引数解決をdoFirst(実行フェーズ)へ移し、-PbeatorajaFfmpegSmokeFileを実行時必須化(未指定なら使い方付きGradleException)。引数順(file → dump → frame)と dump/frame の任意指定は維持。 - 設定評価フェーズで
.get()を呼ばなくなったため、検証用動画が無い環境でも Gradle 設定評価が壊れない。 - 変更ファイル:
build.gradle(1 件)
テスト¶
-
.\gradlew.bat compileJava: 成功(configuration cache 正常保存) -
.\gradlew.bat test: 成功(test 緑) -
.\gradlew.bat verifyFfmpegDecode(プロパティ未指定): 設定評価は成功し、実行フェーズで意図したGradleExceptionを確認(環境非依存を実証)
関連 MR¶
- !93 (feature/173 → develop)
操作