Skip to content

Add third-party license notices to native packages - #8594

Draft
cknitt wants to merge 2 commits into
masterfrom
codex/third-party-licenses
Draft

Add third-party license notices to native packages#8594
cknitt wants to merge 2 commits into
masterfrom
codex/third-party-licenses

Conversation

@cknitt

@cknitt cknitt commented Aug 29, 2026

Copy link
Copy Markdown
Member

This adds generated, target-specific THIRD_PARTY_LICENSES files to the native platform packages.

The notices cover:

  • Vendored Flow and Sedlex sources
  • Reanalyze code adapted from LexiFi/dead_code_analyzer
  • The JavaScript pretty-printer adapted from js_of_ocaml
  • Other incorporated OCaml sources and linked OCaml libraries
  • Rust dependencies generated from Cargo.lock with cargo-about
  • The Rust standard library and relevant platform toolchains
  • musl on Linux and MinGW-w64 on Windows

Repeated Rust standard-library notices are deduplicated while preserving the complete license and attribution information.

The root rescript package does not ship the notice bundle because it contains no native binaries. @rescript/runtime and @rescript/belt are also unaffected and remain MIT-licensed.

Generation and package-content checks run in CI, and notices are regenerated when publishing.

During the source audit, we found that the Rewatch queue implementation had been copied from rawnly/queue-rs, which has no declared license or license file. It has therefore been replaced with a small, independently written implementation providing only the operations Rewatch uses.

Signed-off-by: Christoph Knittel <ck@cca.io>
@cknitt
cknitt force-pushed the codex/third-party-licenses branch from 00a08aa to 3e4cda7 Compare August 29, 2026 05:04
Signed-off-by: Christoph Knittel <ck@cca.io>
@cknitt
cknitt force-pushed the codex/third-party-licenses branch from 3e4cda7 to 21709c8 Compare August 29, 2026 05:09
@pkg-pr-new

pkg-pr-new Bot commented Aug 29, 2026

Copy link
Copy Markdown

Open in StackBlitz

rescript

npm i https://pkg.pr.new/rescript-lang/rescript@8594

@rescript/belt

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/belt@8594

@rescript/darwin-arm64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/darwin-arm64@8594

@rescript/darwin-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/darwin-x64@8594

@rescript/linux-arm64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/linux-arm64@8594

@rescript/linux-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/linux-x64@8594

@rescript/runtime

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/runtime@8594

@rescript/win32-x64

npm i https://pkg.pr.new/rescript-lang/rescript/@rescript/win32-x64@8594

commit: 21709c8

@github-actions

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant