Official Apache-1.1 license text is not being matched correctly by LicenseCompareHelper.matchingStandardLicenseIdsWithinText()
まだ誰も着手していません。
評価
調査の方向性
LicenseCompareHelper.matchingStandardLicenseIdsWithinText() と src/Apache-1.1.xml の Clause5 代替テキストタグから始めてください。公式の Apache-1.1 ライセンステキストを使って失敗を再現し、そのタグがどのように変換されるか、またコメント文字と改行がどのように処理されるかを調べてください。公式テキストが、報告された Clause5 の差異なしに、期待される標準ライセンスと一致すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
When org.spdx.utility.compare.LicenseCompareHelper.matchingStandardLicenseIdsWithinText() is run on the official Apache-1.1 license text, it fails to find any matches, and I believe I've narrowed down the problem to the Clause5 alternative text tag in the template; if I remove the example header from the license text, and run org.spdx.utility.compare.LicenseCompareHelper.isTextStandardLicense().getDifferenceMessage() on it, I get:
Variable text rule combined-bullet-Clause5 did not match the compare text starting at line #31 column #1 "5" while processing rule var: combined-bullet-Clause5
When I manually converted that <alt> tag into a Java regex, and bullet 5 from the Apache 1.1 license text is manually cleansed of comment characters and newlines, I do get a match, so I'm pretty confident the problem is in the library rather than the template. Beyond that I'm not really sure what the root cause might be - whether it has to do with comment character handling, regexification of that particular <alt> tag, or something else entirely.
This was reproduced with Spdx-Java-Library v1.11 and SPDX license list v3.23.
- 主要言語
- Java
- スター
- 71
- フォーク
- 43
- 平均マージ
- 23時間 56分
- マージ済み PR(30日)
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
spdx/Spdx-Java-Library のほかの issue
-
question
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
spdx/Spdx-Java-Library#449 ·
-
question
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
spdx/Spdx-Java-Library#398 ·
-
wontfix
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
spdx/Spdx-Java-Library#393 · コメント 2 件 · リアクション 1 件 ·
-
matching
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
spdx/Spdx-Java-Library#392 · コメント 4 件 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
spdx/Spdx-Java-Library#390 · コメント 11 件 · リアクション 1 件 ·
spdx/Spdx-Java-Library の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
QuiltMC/update-quilt-meta#28 ·
-
area/kotlin
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
Large page[number] overflows the computed offset and returns the wrong page対応中かも @auraseb が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信