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

c2a-core repository に吸収合体したツールなどの置き場所を考えたい #104

Open
Tracked by #2
meltingrabbit opened this issue Sep 27, 2023 · 14 comments
Assignees
Labels

Comments

@meltingrabbit
Copy link
Member

詳細

  • Import c2a-enum-loader #99 ではひとまず(他よりマシという意味で) root においてる
  • /Tools ? とかにまとめたいきもちもあり

close条件

検討して整理したら

@sksat
Copy link
Member

sksat commented Oct 5, 2023

tools というよりは utils といった雰囲気

@meltingrabbit
Copy link
Member Author

なるほどね.
なんか,

src配下にあるのはどうなの?みたいな気持ちもあるよね(src配下はFSWを起きたい気持ちがある. src/{src_core / user} 構造が改善されればなんとかなる?)

@meltingrabbit
Copy link
Member Author

と言う意味では,/sils-runtime も同様に移動させたいな

@sksat
Copy link
Member

sksat commented Oct 12, 2023

src 配下は絶対にナシなのでディレクトリを作るかトップディレクトリに置くかで,ディレクトリを作るのは割と慎重にやりたいので一旦トップディレクトリ,というかんじです

@sksat
Copy link
Member

sksat commented Oct 12, 2023

sils-runtime は tools と意図違うので分けたい(なので,「同様に移動」はしたくない)

@meltingrabbit
Copy link
Member Author

今のrootのを,旧来のc2a-coreのファイルをsrc,それ以外を別フォルダ,にひとまず突っ込むのってなし?
今user←→coreのincludeは絶対パスなので,誰も困らないと思ってます.

@sksat
Copy link
Member

sksat commented Oct 12, 2023

それは絶対にナシです.特にメリットが無い(何度でも言いますが core と user で同じディレクトリによるツリー構造にしておくとうれしいような気がする幻想を捨ててください)上に,C2A user での src/{src_user, src_core} と中途半端に衝突して confusing です.

@sksat
Copy link
Member

sksat commented Oct 12, 2023

もしも長期的(v4 以降の breaking change を挟んでも)に src/{src_user, src_core} の構造を維持しないといけない事情があるのであれば c2a-core/src_core であればナシではないかな,とは思いますが,それよりは src/{src_user, src_core} の構造をどうにかする方が優先度が高いので結局微妙かな.

@meltingrabbit
Copy link
Member Author

ん,伝わってなさそう.今 src/{src_user, src_core} の構造は言及してないです(coreのrepositoryの中の話をしているつもり)

@sksat
Copy link
Member

sksat commented Oct 12, 2023

伝わってます.その上で困ります.

@meltingrabbit
Copy link
Member Author

meltingrabbit commented Oct 12, 2023

本repositoryを

  • FSWになる(おもに C / Rustの)コード郡.組み込み環境でビルドされるファイルたち( /src? ?)
  • ツール,util類(吸収合併したpythonのたぐいやSILSのたぐい) ( util ?)

に分けたくないですか?という話のつもりで,

  • core と user で同じディレクトリによるツリー構造にしておく
  • src/{src_user, src_core} の構造を維持しないといけない

などは全く言ってないです(そうした方がいいともいってない)

@meltingrabbit
Copy link
Member Author

↑直接話した.
ut-issl/c2a-core#6 (comment) の話が v4.0.0 ではなんともならないので
/src/c2a-core の FSW のさまざまなディレクトリ という構造は厳しい(user への影響がたくさんある),とのこと.

@sksat
Copy link
Member

sksat commented Oct 12, 2023

今までの事情とかをあんまり考えずに自然な構成を考えると,こんなかんじになる気がする

  • c2a-core
    • tlm_cmd
      • code-generator
    • library
      • sils-runtime
    • utils
      • enum-loader

@sksat sksat added the tools label Oct 12, 2023
@sksat sksat mentioned this issue Oct 12, 2023
22 tasks
@sksat sksat added this to the v4.0.0 milestone Oct 12, 2023
@sksat
Copy link
Member

sksat commented Oct 13, 2023

適切な置き場の前にリファクタが必要なやつなので, v4.0.0 ではやらない

@sksat sksat removed this from the v4.0.0 milestone Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

2 participants