Skip to content

DirectShowCam関係ファイルの削除#10

Merged
n-ando merged 3 commits intoOpenRTM:masterfrom
n-kawauchi:for_RTM121
Jun 14, 2019
Merged

DirectShowCam関係ファイルの削除#10
n-ando merged 3 commits intoOpenRTM:masterfrom
n-kawauchi:for_RTM121

Conversation

@n-kawauchi
Copy link
Copy Markdown
Contributor

@n-kawauchi n-kawauchi commented Jun 12, 2019

close #9

Identify the Bug

Link #9

  • Windows SDKがサポートされているWindows7環境のvc2008と2010のみを対象にしたRTCであるDirectShowCamは、ソースツリーから削除し、ビルドスクリプトも整理した

    • DirectShowCamのビルドでは、strmbase.libを生成するDirectShowのソースが必要で、これはWindowsSDKのインストールで取得できるが、提供されるのはWindows7まで
    • Windows8.1からはMedia Foundation SDKへ移行したので、これを使ったMFCamera RTCがDirectShowCamに代わり作成された。MFCamera RTCは、vc2012以降で提供している。
  • ビルドスクリプトでは、vc2008とvc2010でDirectShowCam用の特別処理を行っていたが、vc2008はMicrosoftのサポートが終了しているので、OpenRTM-aistでもサポートから外すため、スクリプト内のvc2008に関する処理全てを削除した

  • また、アクティブ化、非アクティブ化、リセット時のタイムアウトを設定 #5 にてOpenCVCameraのタイムアウト設定を行なったので、バージョンは1.2.1となり、OpenRTM-aist 1.2.1リリースに向けてImageProcessingのバージョン番号の 1.2.1 への更新修正も合わせて行った

    • この修正において、ImgeProcessing全体のバージョン番号を新たに一番トップのCMakeLists.txtへ追加した
    • 追加したバージョン番号定義は、Linux環境下でImageProcessingのdeb/rpmパッケージ作成時に利用される
    • Windows用マージモジュールのバージョン番号は、msm作成用スクリプトで管理しているので、追加したバージョン番号は現時点で使用していない

Description of the Change

Verification

  • Did you succesed the build?
  • No warnings for the build?
  • Have you passed the unit tests?
  • ビルド時の条件が変わる「vc10」と「vc10以外」でのビルド動作を確認

@n-kawauchi n-kawauchi requested a review from n-ando June 12, 2019 05:08
@n-ando
Copy link
Copy Markdown
Contributor

n-ando commented Jun 14, 2019

OpenCVCameraのコードって、1.2.0以降変更されてませんでしたっけ?なら、ImageProcessingも1.2.1でもいいのではないですか?

@n-kawauchi n-kawauchi changed the title [compat,->RELEASE_1_2_1] Removed unnecessary code in OpenRTM 1.2.1. DirectShowCam関係ファイルの削除 Jun 14, 2019
@n-kawauchi
Copy link
Copy Markdown
Contributor Author

了解しました。バージョン番号を1.2.1に修正します。

Copy link
Copy Markdown
Contributor Author

@n-kawauchi n-kawauchi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Linux環境でcpackによりimageprocessingのパッケージ(deb/rpm)を作成した際、バージョン番号が1.2.1となるように、一番トップのCMakeLists.txtへPROJECT_VERSIONを新たに定義した。

ImageProcessing下の全てのRTCが同じバージョン番号だった場合はこれまでの設定で問題なかったが、一部のRTCの変更でImageProcessingのバージョン番号を更新する場合は対応できないことが判明。詳細は、一番最初にcmakeするAffine RTCのcmake変数${PROJECT_VERSION}の値が最後まで更新されず残ってしまうことによる不具合。

@n-ando n-ando merged commit 3f37d19 into OpenRTM:master Jun 14, 2019
@n-kawauchi n-kawauchi deleted the for_RTM121 branch April 9, 2020 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows SDK(Win8.1以降廃止)が必要なDirectShowCam関係のファイルを削除

2 participants