Control `pathSuffix` option via configuration file(s)
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- 機能追加
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- babel, node.js, react-native, typescript
調査の方向性
ソースファイルやテストは指定されていません。まず、pod install または Gradle ビルド中の NODE_API_PATH_SUFFIX を含め、pathSuffix が Babel プラグインとホストパッケージの CLI にどのように渡されるかを追跡しながら、pull request 104 が及ぼす可能性のある影響を確認してください。アプリ全体またはライブラリ単位の設定によって、別個のランタイム設定なしにオプションを一貫して制御できれば完了です。
索引モデルが issue の本文から書いたものです。
説明
We're performing a transformation of the Node-API addon dynamic library paths on disk to their dynamic library names as they're linked into the host package.
Here's the documentation for the pathSuffix option of the NamingStrategy:
Controls how the path of the addon inside a package is transformed into a library name.
The transformation is needed to disambiguate and avoid conflicts between addons with the same name (but in different sub-paths or packages).As an example, if the package name is
my-pkgand the path of the addon within the package isbuild/Release/my-addon.node:
"omit": Only the package name is used and the library name will bemy-pkg."strip": Path gets stripped to its basename and the library name will bemy-pkg--my-addon."keep": The full path is kept and the library name will bemy-pkg--build-Release-my-addon.
In the current state, this pathSuffix option is passed to the babel plugin as well as to the host package's CLI (either through a CLI runtime option or more likely the NODE_API_PATH_SUFFIX environment variable while running pod install or the Gradle build) and they need to match for the app to find the dynamic library files at runtime.
It would be more flexible and less brittle if ☝ was instead determined by
- a single app-wide configuration setting, either in configuration file (or a key in the App's package.json) and / or
- by individual per-library configuration files (or a key in their package.json files)
[!NOTE]
This will likely all change once https://github.com/callstackincubator/react-native-node-api/pull/104 merge.
- 主要言語
- TypeScript
- スター
- 188
- フォーク
- 10
- 平均マージ
- 2日 17時間
- マージ済み PR(30日)
- 3
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
callstackincubator/react-native-node-api のほかの issue
-
Android 🤖
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
Sanity check of Hermes source directory via environment variable, when building Android projects オープンAndroid 🤖 good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
callstackincubator/react-native-node-api#163 · コメント 1 件 ·
-
enhancement good first issue gyp-to-cmake
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
callstackincubator/react-native-node-api#161 · コメント 3 件 · リアクション 1 件 ·
-
enhancement Host 🏡
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
Automatable good first issue
難易度 3/5 1〜2日 初心者へのやさしさ 15/100
callstackincubator/react-native-node-api#424 · コメント 1 件 ·
callstackincubator/react-native-node-api の issue をすべて見る
似ている issue
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
Mend: dependency security vulnerability untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100