apache / apache/datafusion-sqlparser-rs
Fix licensing issues in source release found during 0.63.0 RC1 vote (missing NOTICE file, flamegraph.svg with CDDL-licensed JavaScript)
- Lenguaje dominante
- Rust
- Estrellas
- 3.5k
- Forks
- 772
- Merge medio
- 4 d 9 h
- PR fusionados (30 d)
- 17
Descripción
**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](https://github.com/jonhoo/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`):
```shell
$ 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**
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Línea de trabajo
Comienza en la raíz del repositorio con el archivo de código fuente 0.63.0-rc1 e inspecciona NOTICE, LICENSE.TXT y sqlparser_bench/img/flamegraph.svg; ejecuta las comprobaciones ls y grep del issue para confirmar los hallazgos. Se considera terminado cuando la versión fuente incluye el archivo NOTICE requerido y excluye flamegraph.svg generado sin conservar el JavaScript con licencia CDDL.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- rust
- Área
- release
- Tipo de issue
- Error
- Dificultad
- 2/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Activo
- Claridad
- Bien especificado
- Aptitud para principiantes
- 72/100