Fix licensing issues in source release found during 0.63.0 RC1 vote (missing NOTICE file, flamegraph.svg with CDDL-licensed JavaScript)

オープン 初心者向け
#2,495 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜2日
初心者へのやさしさ
72/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
rust
領域
release

調査の方向性

リポジトリのルートから 0.63.0-rc1 のソースアーカイブを使って開始し、NOTICE、LICENSE.TXT、sqlparser_bench/img/flamegraph.svg を調査します。調査結果を確認するため、issue の ls および grep チェックを実行します。ソースリリースに必要な NOTICE ファイルが含まれ、生成された flamegraph.svg が除外され、CDDL ライセンスの JavaScript が残されていなければ完了です。

索引モデルが issue の本文から書いたものです。

説明

Describe the bug

During the vote on the Apache DataFusion sqlparser-rs 0.63.0 RC1 release, @Xuanwo identified two licensing-material issues in the source archive and voted -1 (non-binding): https://lists.apache.org/thread/3lnhgs0bwzl148yrdrcl5g005h9pgtmx

  1. Missing NOTICE file: the source archive contains LICENSE.TXT but no NOTICE file, which ASF releases are expected to include.
  2. sqlparser_bench/img/flamegraph.svg embeds third-party licensed code: the generated flamegraph embeds JavaScript from Inferno, whose upstream license is CDDL 1.0, but the archive contains no corresponding license materials.

To Reproduce

Download and unpack the 0.63.0 RC1 source archive (or check the repository at v0.63.0-rc1):

$ ls NOTICE*
ls: NOTICE*: No such file or directory

$ grep -c -i inferno sqlparser_bench/img/flamegraph.svg
2

Expected behavior

The source release complies with ASF licensing policy:

  1. Add a NOTICE file to the repository so it is included in source archives.
  2. Exclude the generated sqlparser_bench/img/flamegraph.svg from the source release

Additional context

主要言語
Rust
スター
3.5k
フォーク
774
平均マージ
3日 10時間
マージ済み PR（30日）
15

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

apache/datafusion-sqlparser-rs のほかの issue

apache/datafusion-sqlparser-rs の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。