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

一括インストールスクリプトのOS判定修正 #1148

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

n-kawauchi
Copy link

close #1147

Identify the Bug

Link to #1147

Description of the Change

  • Ubuntu, Raspbian 両スクリプト共通修正
    • コードネーム検索は、従来の /etc/apt/sources.list に加え、/etc/apt/sources.list.d ディレクトリも対象にした
    • 今回の修正で、スクリプトのバージョン番号を 2.0.2.01 へ更新
  • Ubuntu用スクリプトの修正
    • Fluentbitのdebパッケージ名が変更され fluent-bit となっているので対応
  • Raspbian用スクリプトの修正
    • Fluentbitに関する処理をコメントアウトの形で残しておいた
    • Ubuntuと同様の処理を組み込んだが、2022/08時点での確認でFluentbitのソースビルドが通らない、arm64版のdebパッケージが見たらない等の理由でコメントアウトしたもの
    • Ubuntu用の修正でFluentbitのdebパッケージ名が変更になったこともあり、古くて使えない処理をコメントアウトの形で残していても紛らわしいため削除した

Verification

  • Ubuntu24.04環境へ -c オプションで開発ツールをインストールできることを確認した
  • Did you succeed the build?
  • No warnings for the build?
  • Have you passed the unit tests?

@n-kawauchi n-kawauchi requested a review from n-ando June 25, 2024 03:06
@n-kawauchi n-kawauchi self-assigned this Jun 25, 2024
@n-ando n-ando merged commit 197f28b into OpenRTM:master Jul 1, 2024
4 checks passed
@n-kawauchi n-kawauchi deleted the updated_install_scripts branch July 2, 2024 01:02
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.

一括インストールスクリプトのOS判定修正
2 participants