Comments (6)
@200km 先日slackにてご質問させていただいた件です。
再考した結果,やはりCMakeLists.txtはトップに置きたいと考えましたのでご意見をお伺いしたいです。
変更するという結論が出ていないためPRはまだ出していませんが,議論の参考ためパス(とそれに合わせて中身)を変更したものを feature/fix_cmakelists_path および feature/remove_s2e-aocs-core_directory に push しました。前者は CMakeLists.txt のみ移動,後者は他のディレクトリも移動させ,s2e-aocs-core をなくしたものになっています。
from s2e-aobc.
例えばこのサイトには以下のような記述があります。
- CMakeLists.txt is placed at the root of the source tree of any application, library it will work for.
- If there are multiple modules, and each module can be compiled and built separately, CMakeLists.txt can be inserted into the sub folder.
s2e-aocs-coreが単独でビルドできるのであれば現状のようにs2e-aocs-core/CMakeLists.txtを置きさらにルート下にもCMakeLists.txtを置くかたちが良いかと思いますが,s2e-aocs-coreがs2e-coreに依存しているため提案した形式が良いと考えています。
from s2e-aobc.
まず現在のディレクトリ構成で技術的な問題があるか
という点を整理したいです。問題があれば、それを修正するのが良いと思っています。
技術的な問題はなくこの方が良さそう
という感覚や好みの問題なら、答えを出すのは難しいのでメンバー内で相談して最終的には多数決的に決めるのが良いと思います。
from s2e-aobc.
今のディレクトリ構成になっている一つの理由は、s2e-documentのtutorial等で推奨しているディレクトリ構成が次のようになっているからです。
- s2e-user
- s2e-userのCMakeListはここにある
- s2e-core
- ExtLibraries
その上で、submoduleを使うかどうかはユーザーごとに好きにして良いのですが、ディレクトリ構成を合わせておいたほうがTutorialのCMakeListやdataファイルなどをコピーしやすいので、現在のような形になっています。
s2e-documentのtutorialを変えて、submoduleを使った方法などを推奨にするというのも有りだとは思います。
このあたりも含めて議論したいですね。
from s2e-aobc.
ありがとうございます。
現在のディレクトリ構成で技術的な問題があるか
といえばないとは思います。
経緯については理解しました。
from s2e-aobc.
AOCS内ミーティングで下記のように決まった
- s2e-aobcのディレクトリ構成は変更する方向
- CMakeListとsrc, dataを抜き出す方向
- srcの中にs2e-coreを入れるかどうかは要議論
- s2e-core, documentのチュートリアルはとりあえず今のままにする(submoduleを使わない人もいると思うので)
- チュートリアルとs2e-aobcの構成が異なるので、今後アップデートがあってもそのままコピーなどできなくなるが、そこはs2e-aobc側で運用を気をつければ良いだけなので許容する。
from s2e-aobc.
Related Issues (20)
- Port Configの整理 HOT 2
- 軽微な表示バグ
- MTQ温度依存特性の実装
- プロジェクト固有情報の分離 HOT 4
- C2Aを含めたビルド方法がREADMEと食い違っている HOT 4
- submoduleのs2e-coreをSSH認証にする HOT 9
- clang-formatの置き場所検討
- doxygenコメントをつける
- s2e-coreの.clang-formatと合わせるための仕組みを作る
- Rename files which contain `ISSL6U` keyword HOT 7
- Fix typo in data/initialize_files/satellite_structure.ini
- Remove merged remote branch automatically HOT 2
- Apply coding conventions provided by s2e-core
- Dependency Dashboard
- GPS-Rで時刻情報が桁落ちしている
- Coding ruleをS2E-COREと合わせる
- v5.0.0 Major update前の微修正
- ユーザー側で使うコンポーネントを選択できるようにする
- cube wheel 関連コードの修正
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from s2e-aobc.