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

devtoolsをホストする #33

Merged
merged 10 commits into from
Dec 21, 2023
Merged

devtoolsをホストする #33

merged 10 commits into from
Dec 21, 2023

Conversation

kobkaz
Copy link
Contributor

@kobkaz kobkaz commented Dec 6, 2023

#30

gaiaを起動してlocalhost:8900をブラウザで開くとlocalhot:8900/devtools/にリダイレクトされ、devtoolsが開きます

一番目のコミットはrust側の変更と、devtools( arkedge/c2a-devtools@610baac 時点)のコピー
二番目のコミットはdevtoolsを/devtools/動かす為の変更

tmtc-c2a/src/devtools_server.rs Outdated Show resolved Hide resolved
tmtc-c2a/build.rs Outdated Show resolved Hide resolved
@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 6, 2023

axumを0.7にしようかと思ったけれどtonic側の対応が終わっていない

@KOBA789
Copy link
Member

KOBA789 commented Dec 6, 2023

axumを0.7にしようかと思ったけれどtonic側の対応が終わっていない

なるほど〜〜

@KOBA789
Copy link
Member

KOBA789 commented Dec 6, 2023

概ねいい感じなので、手元で動かして細かいところ見ていきますー

@KOBA789 KOBA789 mentioned this pull request Dec 6, 2023
@KOBA789
Copy link
Member

KOBA789 commented Dec 7, 2023

tmtc-c2a/devtools_frontend/ 以下のファイルを編集した際に cargo buid で build.rs が発火しない=yarn buildが走らないという問題があり、build.rs に詳しい @sksat にちょっと見てほしい

@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 8, 2023

build.rsはデフォルトでは配下のファイルが一つでも変わっていたらrerunするが、cargo:rerun-ifで明示的に依存ファイルが指定されている場合は指定されたファイル以外を無視する
今回はtonicが.protoを明示的に指定しているので明示的な指定のないフロントエンドは無視されてしまった

ということかな

@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 8, 2023

フロントエンド変更時にbuild.rsが発火するようにしました
これに伴いフロントエンドのビルド先をrustのOUT_DIRの下に移動しています
viteへのoutDir指定をyarnを介して行う方法がよくわからずnpxを直接叩いてしまっているので詳しい人に見てほしい

@KOBA789
Copy link
Member

KOBA789 commented Dec 8, 2023

あー、tonic-build が rerun-if 吐いてるのか。なるほど。
あとでみます

tmtc-c2a/build.rs Outdated Show resolved Hide resolved
tmtc-c2a/build.rs Outdated Show resolved Hide resolved
Co-authored-by: KOBA789 <koba789@arkedgespace.com>
@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 20, 2023

ビルドは大丈夫そうです

@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 20, 2023

eslint等設定ファイルをコピーし忘れていたので追加

Copy link
Member

@KOBA789 KOBA789 left a comment

Choose a reason for hiding this comment

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

よ〜〜し READY TO MERGE 🚀

@kobkaz kobkaz merged commit 4a585ff into main Dec 21, 2023
3 checks passed
@kobkaz kobkaz deleted the host_devtools branch December 21, 2023 01:04
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.

None yet

2 participants