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

バージョン情報の記載のあるheaderを作る #170

Closed
meltingrabbit opened this issue Jan 6, 2022 · 27 comments
Closed

バージョン情報の記載のあるheaderを作る #170

meltingrabbit opened this issue Jan 6, 2022 · 27 comments
Assignees
Labels

Comments

@meltingrabbit
Copy link
Collaborator

meltingrabbit commented Jan 6, 2022

概要

バージョン情報の記載のあるheaderを作る

詳細

#define C2A_CORE_VER_MAJOR 3
#define C2A_CORE_VER_MINOR 4
#define C2A_CORE_VER_PATCH 0

みたいな

close条件

できたら

備考

議論点は

  • どのフォルダに配置する?
  • いつどのように更新する?

関連

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

場所的には c2a_core_main.hでは,と思っています(何にも属していないのでルートにあるのが望ましい & ルート直下のファイルを増やしたくない)

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jan 6, 2022

バージョンしか書かれてないヘッダのほうが,toolなどで読むときとか,自動で値更新するとき(があるかわからんが)に便利そうって思ったりはした.(あと,適当な場所で使うときにincludeするときとかもわかりやすい.バージョン番号つかうのだけに c2a_core_main.h をincludeしたくないなと.)

@meltingrabbit
Copy link
Collaborator Author

ルート直下のファイルを増やしたくない

はそうなので,一旦それで,で,今後困ったらファイル分割するか~

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

toolで読む時,自動更新は別にどこにあってもそんなに変わらないかな,と思っています.
ただ,バージョン番号使うためだけに c2a_core_main.hをincludeしたくない,はかなりあると思っていて,個人的にはファイル分けたいですね.ルート直下にファイル増えることを許容できるか次第.

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jan 6, 2022

バージョン番号使うためだけに

実際,これはほぼなさそう? commit hashはtlmにいれたり,デバッグ表示にいれたりするので使うけど

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

そうなんですよね.そんなにプログラム中で使う気はしないというのはある.

@meltingrabbit
Copy link
Collaborator Author

じゃあ一旦 c2a_core_main.h で.

あとはどう更新するかか.main update prでもいいが,そうするとmain mergeではじめて上がった記載ファイルをどうdevまで持ってくるかが課題.

これ,devをmain merge後に毎回rebaseするのってどうなんだ?(まあいいのかな)

@meltingrabbit
Copy link
Collaborator Author

dev rebase って,PRでできるのか,というのも気になり

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

dev rebase って,PRでできるのか,というのも気になり

これ何を気にしてます?

@meltingrabbit
Copy link
Collaborator Author

手元でのforce pushにならない?って.(devとmainはいまforce push禁止してる

@meltingrabbit
Copy link
Collaborator Author

あー,ff mergeのpr出せばいいのか?

@meltingrabbit
Copy link
Collaborator Author

といってもいま,repository設定で--no-ffマージしか許されていない

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

そうだった... > force push/--no-ff

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

じゃあmain update PRでやればいいのでは

@meltingrabbit meltingrabbit self-assigned this Jan 6, 2022
@meltingrabbit
Copy link
Collaborator Author

じゃあmain update PRでやればいいのでは

あーーー,頭がついてなかった.

そっか,main update prってdevにcommitできるじゃん.

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

実はそう

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jan 6, 2022

あれ,devはpush禁止してるので,commitできない説(PRマージしかできない)

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

アッ

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Jan 6, 2022

  • devのバージョンあげますPR(すぐmain updateするよ |ω・)チラ
  • main update PR

の2段??...というリリースフロー?

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

dev push解禁しない限りそれしかないかあ

@meltingrabbit
Copy link
Collaborator Author

厳密にはtagよりも古いcommitにバージョン含まれちゃうが,まあ動作に影響のあるファイルには更新ないので,いいか.

@meltingrabbit
Copy link
Collaborator Author

dev push解禁はないなぁ

@meltingrabbit
Copy link
Collaborator Author

ではそれで行くかぁ.

#169 とあわせて,リリースフロー,ドキュメント化します.

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

dev push解禁はないなぁ

ですよねー.adminだけpush可みたいなかんじにできればまた別なんですが.

@meltingrabbit
Copy link
Collaborator Author

まあ,それもそれで事故こわいんで,一旦これで運用しよう,でクソだるかったらadmin解禁含めて考えよう.

@sksat
Copy link
Collaborator

sksat commented Jan 6, 2022

人間ではなくリリース担当Botを生やしてそいつにやらせる,などはないではない(が,運用コストを考えるとノーチャンな気がする)

@meltingrabbit
Copy link
Collaborator Author

https://github.com/ut-issl/c2a-core/blob/develop/Docs/General/release.md でやった.

更新については,やってみてだるかったらまた考える.

close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants