Skip to content

Conversation

@mayo031042
Copy link

@mayo031042 mayo031042 commented Dec 11, 2025

プロジェクトルートで以下のようにして実行してください

# CLImodeで起動(内部でsudoを呼んでいます)
./ppcli

CLImodeで起動しlogコマンドを実行した状態で
echo "Added to log" >>logs/gulp.log
のようにログファイルへ追記を実行すると挙動の確認ができます

cli modeではencode modeとdecode modeが存在し、switchコマンドで切り替えが可能ですが、それ以外の機能は未実装です

  • cli機能を追加
  • logコマンドを追加、継続的にログ出力可能
    • log 10 などで指定行数のみ出力可能

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このファイルですが、現在PacketProxyを利用している人が、kotlinの環境を整えないとコンパイルできなくなってしまうので、もとのJavaファイルに戻してもらえると助かります。
PacketProxy本体をJavaからkotlinに変更するときは、一斉に全てのファイルをkotlinに変更した上でメジャーバージョンを1つ繰り上げ、さらに周知をしたいと思います。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このファイルは復活させてもらえると

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このファイルも復活させてください

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このファイルはJavaで書き直してもらえると

@mayo031042 mayo031042 force-pushed the feature/gulp-interface branch from 40ef9af to 064ccd7 Compare December 17, 2025 06:28
@mayo031042 mayo031042 force-pushed the feature/gulp-interface branch from a30d87a to 4e8ba23 Compare December 17, 2025 12:24
@mayo031042
Copy link
Author

mayo031042 commented Dec 17, 2025

現状存在する以下のinstall/build方法において、ユーザが「実装にjavaの他にkotlinファイルを含んでいるか否か」を意識する必要はない認識です

  • ソースコードからビルドする
    • gradleがkotlinのビルドに必要なコンポーネントの取得を自動的に行う
  • Releaseページから取得する
    • ユーザの環境でkotlinファイルがコンパイルされることはない
  • homebrewから取得する
    • ユーザの環境でkotlinファイルがコンパイルされることはない

@mayo031042 mayo031042 force-pushed the feature/gulp-interface branch from 4e8ba23 to 66d11c0 Compare December 18, 2025 06:38
Copy link
Contributor

@funa-tk funa-tk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです!

@funa-tk funa-tk merged commit f9fef01 into DeNA:master Dec 18, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants