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

CMake対応 #15

Closed
yumetodo opened this issue May 15, 2019 · 13 comments · Fixed by #18
Closed

CMake対応 #15

yumetodo opened this issue May 15, 2019 · 13 comments · Fixed by #18
Labels
enhancement New feature or request
Milestone

Comments

@yumetodo
Copy link
Contributor

yumetodo commented May 15, 2019

./MSVC以下がmasterから消え去ったらPull Requestを投げつける

@yumetodo yumetodo changed the title CM CMake対応 May 15, 2019
@AsPJT
Copy link
Owner

AsPJT commented May 15, 2019

./MSVC以下がmasterから消え去りました。

@AsPJT AsPJT added the enhancement New feature or request label May 15, 2019
@yumetodo
Copy link
Contributor Author

よっしゃ、やるわ。金曜日か土曜日くらいには作業終わると良いなぁ

@yumetodo
Copy link
Contributor Author

@Kasugaccho
Sampleの中をdirectory分けしたいのです。つまり

  • Reference
    • Sample
      • *.cpp

から

  • sample
    • Shape
      • *.cpp
    • Storage
      • *.cpp
    • etc

としたいです。

理由

  • shape_のようなprefixをつけるくらいならdirectoryを分けるべき
  • sampleがtop levelになくてReference以下にあるのは直感的ではない

@AsPJT
Copy link
Owner

AsPJT commented May 18, 2019

確かにそうですね。
ということで Sample フォルダはトップに、
プレフィックスはフォルダとして生まれ変わりました。

@yumetodo
Copy link
Contributor Author

@Kasugaccho includeSampleで命名規則がガバガバでは?

@AsPJT
Copy link
Owner

AsPJT commented May 18, 2019

ちゃんと命名規則があるのでガバガバではないと思いますが、どの辺がガバガバでしょうか。

@yumetodo
Copy link
Contributor Author

いまいちどういう命名規則かわからないのです。

  • include
    • DTL
      • Shape
      • Storage
  • sample
    • Shape
    • Storage

てっきりこういうふうになるのだと思ってました。

@AsPJT
Copy link
Owner

AsPJT commented May 18, 2019

DTLフォルダを作成しました。

@yumetodo
Copy link
Contributor Author

ちょっと立て込んでいたので作業できてませんでした。

ところで

  • include
    • DTL
      • Shape
      • Storage
  • Sample
    • DTL
      • shape
      • storage

となったようですが、実に面白い命名規則ですね!ということでこのまま作業を進めます、CMake対応には直接関係ないので。

@AsPJT
Copy link
Owner

AsPJT commented May 18, 2019

  • include
    • DTL
      • Shape
      • Storage
  • Sample
    • DTL
      • Shape
      • Storage

へ変更しました。
(includeフォルダ以外のフォルダ名が小文字になっているのはバグです)

@yumetodo
Copy link
Contributor Author

(includeフォルダ以外のフォルダ名が小文字になっているのはバグです)

??

@AsPJT
Copy link
Owner

AsPJT commented May 18, 2019

includeフォルダ以外のフォルダは、
フォルダ名を先頭は大文字でそろえているので
ミスでした。

@yumetodo
Copy link
Contributor Author

あ、治りましたね。本格的に作業に取り掛かります。

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

Successfully merging a pull request may close this issue.

2 participants