artemis-dev / artemis

A RooT Extension with Modular processors for Instant Switching
2 stars 6 forks source link

ヘッダファイルのインストール先がおかしい #41

Closed shinsuke-ota closed 1 year ago

shinsuke-ota commented 1 year ago

b140bd6 を用いてコンパイル、インストールをスクラッチから行ったところ、ヘッダファイルのインストール先が $CMAKE_INSTALL_PREFIX/sources/... などとなってしまって、元のソースディレクトリの構造を保存したままになってしまっているようです。

rin-yokoyama commented 1 year ago

@shinsuke-ota 確か元のリポジトリに使われていないファイルが残っており、そのままだとsources/ artemis-share/ cat-src/間で競合があって一か所にインストールできなかったので、後にディレクトリ構造を含め整理を行うつもりで今の状態になっています。至急解決する必要があれば問題のあるヘッダーファイルのみ探して削除して一か所にインストールするようにしますがいかがですか? cmake を使う場合 find_package(artemis) すると自動でincludeが設定されるのであまり問題視していませんでした・・・

shinsuke-ota commented 1 year ago

@rin-yokoyama なるほど。そのあたりを整理しないとだめだったという。ひとまず至急ではないのですが、.C+ とかでマクロを aclic でコンパイルするときに artemis のクラスを使う場合には問題が出てくるかなと思います。 (cmake に統一するのであれば便利ですね。)

もしやってもらえるのであれば、削除するまえにひとまずどのファイルが競合するかを教えてもらえると嬉しいです。

rin-yokoyama commented 1 year ago

競合があるファイルは削除しました。 まだ多少artemis-shareに使われていないファイルは残っています。 @shinsuke-ota にreview requestしました。

rin-yokoyama commented 1 year ago

masterにpull request していたので developにやり直しました。#44 でお願いします。