kusumotolab / kGenProg

A High-performance, High-extensibility and High-portability APR System
MIT License
48 stars 13 forks source link

バージョン情報の追加 #694

Closed shinsuke-mat closed 4 years ago

shinsuke-mat commented 4 years ago

やったこと

kgp起動時のConfigの一つにversionを追加

以下の3つの実行コンテキストを読み取って出力を変更するように. 1.5.1++ はローカル編集により 1.5.1 よりも少し新しいという意味.

### ローカルで `java` コマンドで実行
結果: version = 1.5.1+
理想: version = 1.5.1+

### ローカルで生成した `jar` を使って実行
結果: version = 1.5.1+
理想: version = 1.5.1+

### CIで生成した `jar` を使って実行
結果: version = 1.5.1
理想: version = 1.5.1

出力例

$ (cd example/CloseToZero01/; java -jar $kgp)
2019-12-24 17:37:57 [main] [INFO]  KGenProgMain -
==================== kGenProg Configuration ====================
...
historyRecord = false
currentDirectory = D:\dev\eclipse-workspace\kGenProg\example\CloseToZero01
version = 1.5.4+
================================================================

--version オプションを追加

$ java -jar $kgp --version
kGenProg version: 1.5.4

併せてcurrentDirectory 属性の解決方法を改善

Mainでの処理を削除して,Configurationの一つとして扱うように変更.

その他

build.gradleのコメント追加 Configurationクラスのコメント削除( // region 等) ci周りの設定を少し改善(バージョンの取り回しをシンプルに)

codecov-io commented 4 years ago

Codecov Report

Merging #694 into master will decrease coverage by 0.14%. The diff coverage is 54.28%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #694      +/-   ##
============================================
- Coverage     78.63%   78.48%   -0.15%     
- Complexity      876      880       +4     
============================================
  Files           143      144       +1     
  Lines          3422     3454      +32     
  Branches        251      254       +3     
============================================
+ Hits           2691     2711      +20     
- Misses          629      638       +9     
- Partials        102      105       +3
Impacted Files Coverage Δ Complexity Δ
...ain/java/jp/kusumotolab/kgenprog/KGenProgMain.java 93.33% <ø> (-0.15%) 26 <0> (ø)
src/main/java/jp/kusumotolab/kgenprog/Version.java 53.57% <53.57%> (ø) 5 <5> (?)
...in/java/jp/kusumotolab/kgenprog/Configuration.java 84.76% <57.14%> (+1.01%) 25 <0> (ø) :arrow_down:
.../java/jp/kusumotolab/kgenprog/output/Exporter.java 76.47% <0%> (-3.93%) 16% <0%> (-1%)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7ae6e91...eb3f884. Read the comment docs.

shinsuke-mat commented 4 years ago

「ソースの中にver情報を埋め込まない」という方針をとった以上, java実行(=非jar実行)時にverを解決できるのがそもそも変ともいえます.

あちらを立てるかこちらを立てるか問題