fix travis build issue
まだ誰も着手していません。
- 主要言語
- C++
- スター
- 74
- フォーク
- 9
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
This morning, I found that all the c++ build job related to "TOOLSET=asan" returns an error, even we try to rebuild the old jobs which already passed:


Debug w/ @springmeyer, we thought this is a new Travis error due to a new leak in node.js.
To fix this issue we need to add echo "leak:node::Start" >> ${SUPPRESSION_FILE} in scripts/sanitize.sh
diff --git a/scripts/sanitize.sh b/scripts/sanitize.sh
index 4168cca..28c9344 100755
--- a/scripts/sanitize.sh
+++ b/scripts/sanitize.sh
@@ -24,6 +24,7 @@ SUPPRESSION_FILE="/tmp/leak_suppressions.txt"
echo "leak:__strdup" > ${SUPPRESSION_FILE}
echo "leak:v8::internal" >> ${SUPPRESSION_FILE}
echo "leak:node::CreateEnvironment" >> ${SUPPRESSION_FILE}
+echo "leak:node::Start" >> ${SUPPRESSION_FILE}
echo "leak:node::Init" >> ${SUPPRESSION_FILE}
export ASAN_SYMBOLIZER_PATH=$(pwd)/mason_packages/.link/bin/llvm-symbolizer
export MSAN_SYMBOLIZER_PATH=$(pwd)/mason_packages/.link/bin/llvm-symbolizer
OR see this PR https://github.com/mapbox/node-cpp-skel/pull/156/files.
This will fix the issue.
cc w/ @mapbox/maps-api @flippmoke @mapsam @millzpaugh
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
scripts/sanitize.sh と TOOLSET=asan を使用する Travis ジョブから始めます。Sanitizer の suppression エントリを参照されている node-cpp-skel pull request と比較し、その後、影響を受ける C++ ASAN ビルドを再実行するか調査します。ASAN ジョブが報告された Node.js のリーク失敗なしに完了すれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp, node.js, shell
- 領域
- build-system, ci-cd
- issue の種類
- バグ
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 35/100