-
Notifications
You must be signed in to change notification settings - Fork 196
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
Linux 自動ビルド: ビルド用OSをUbuntu 18.04に切り替え #116
Merged
Hiroshiba
merged 62 commits into
VOICEVOX:master
from
aoirint:patch-linux-docker-build-ubuntu18.04
Sep 23, 2021
Merged
Linux 自動ビルド: ビルド用OSをUbuntu 18.04に切り替え #116
Hiroshiba
merged 62 commits into
VOICEVOX:master
from
aoirint:patch-linux-docker-build-ubuntu18.04
Sep 23, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…base-image-ubuntu18.04
…base-image-ubuntu18.04
…r-base-image-ubuntu18.04
…r-base-image-ubuntu18.04
aoirint
changed the title
Linux 自動ビルド: OSをUbuntu 18.04に切り替え
Linux 自動ビルド: ビルド用OSをUbuntu 18.04に切り替え
Sep 22, 2021
こちらもコンフリクト解消のマージコミットをします。 |
Hiroshiba
approved these changes
Sep 23, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! プルリクエストの切り分けがしっかりしていてとても見やすかったです。
こちらのプルリクエストでVOICEVOXエンジンの自動ビルドは一段落かと思うのですが、VOICEVOXソフトウェアの方の自動ビルドにご興味はありませんか・・・?
このエンジンのビルドの方が圧倒的に難しくて、エンジンのビルド結果を取得する機構さえ整えれば、ソフトウェアのビルドはかなりゴールは近いのかなと感じています。
ローカルでは簡単にビルドできたので、試してみようと思います。 |
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ref: #95, #98
#114 と #98 のコミットを含んでいます。先にそちらのレビューをお願いします。
Linux用実行バイナリの自動ビルドに使うOSをUbuntu 20.04から、Ubuntu 18.04に切り替えます。
互換性の拡大が見込めます。
ubuntu:bionic
nvidia/cuda:11.4.1-cudnn8-runtime-ubuntu18.04
glibc 2.27環境のldconfigとLibTorchの互換性
Ubuntu 18.04 / glibc 2.27環境で、ldconfigでLibTorchを認識させようとすると、壊れた名前のシンボリックリンクがLibTorchのディレクトリに作成され、共有ライブラリリスト(
ldconfig -p
)が文字化けする問題があります。この問題は、glibc 2.27では未実装の、glibc 2.31で実装された機能に関係しているようです。
文字化けした出力(Ubuntu 18.04, glibc 2.27)
正常な状態(Ubuntu 20.04, glibc 2.31)
この問題を回避するため、このPRにおいて、build-arg
GLIBC_231_WORKAROUND
を使って、Ubuntu 18.04のDockerイメージでは、ld.so.conf.dからLibTorchのディレクトリパスを削除し、ldconfigがLibTorchを認識しないようにする変更を加えています。上の問題が起きたままNuitkaビルドを実行すると、Nuitkaの内部実装で、共有ライブラリの検索に
ldconfig -p
の出力を使っているため、文字列処理に失敗し、AssertionError や UnicodeDecodeError でビルドが失敗します。ビルド済みの実行用バイナリをUbuntu 18.04で実行する上では、この問題は発生しません。