Skip to content

chore: Allow Apache-2.0 WITH LLVM-exception license - #661

Merged
sbernauer merged 1 commit into
mainfrom
chore/apache2-llvm-license
Nov 3, 2026
Merged

sbernauer merged 1 commit into
mainfrom
chore/apache2-llvm-license

Conversation

@sbernauer

@sbernauer sbernauer commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Discussed in https://stackable-workspace.slack.com/archives/C02FZ581UCD/p1793698053521549

The license is defined in https://github.com/rust-lang/ar_archive_writer/blob/d8ef2b27e4e077ba420f98592edbe729e1c8f287/Cargo.toml#L6

We pull that now in because of pest-parser/pest_stack_guard#1

cargo update caused

error[rejected]: failed to satisfy license requirements
   ┌─ /home/sbernauer/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/ar_archive_writer-0.5.3/Cargo.toml:29:12
   │
29 │ license = "Apache-2.0 WITH LLVM-exception"
   │            ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   │            │
   │            rejected: license is not explicitly allowed
   │
   ├ Apache-2.0 - Apache License 2.0:
   ├   - OSI approved
   ├   - FSF Free/Libre
   ├ ar_archive_writer v0.5.3
     └── (build) psm v0.1.32
         ├── pest_stack_guard v1.0.1
         │   └── pest v2.9.5
         │       ├── dockerfile-parser v0.9.0
         │       │   └── stackable-operator v0.121.0
         │       │       └── stackable-trino-operator v0.0.0-dev
         │       ├── jsonpath-rust v1.0.12
         │       │   └── kube-client v5.0.0
         │       │       ├── kube v5.0.0
         │       │       │   ├── stackable-certs v0.4.1
         │       │       │   │   └── stackable-webhook v0.10.0
         │       │       │   │       └── stackable-operator v0.121.0 (*)
         │       │       │   ├── stackable-operator v0.121.0 (*)
         │       │       │   ├── stackable-shared v0.2.0
         │       │       │   │   ├── stackable-certs v0.4.1 (*)
         │       │       │   │   ├── stackable-operator v0.121.0 (*)
         │       │       │   │   └── stackable-webhook v0.10.0 (*)
         │       │       │   ├── stackable-versioned v0.13.0
         │       │       │   │   └── stackable-operator v0.121.0 (*)
         │       │       │   ├── stackable-versioned-macros v0.13.0
         │       │       │   │   └── stackable-versioned v0.13.0 (*)
         │       │       │   └── stackable-webhook v0.10.0 (*)
         │       │       └── kube-runtime v5.0.0
         │       │           └── kube v5.0.0 (*)
         │       ├── pest_derive v2.9.5
         │       │   ├── dockerfile-parser v0.9.0 (*)
         │       │   └── jsonpath-rust v1.0.12 (*)
         │       ├── pest_generator v2.9.5
         │       │   └── pest_derive v2.9.5 (*)
         │       └── pest_meta v2.9.5
         │           └── pest_generator v2.9.5 (*)
         └── stacker v0.1.25
             └── pest_stack_guard v1.0.1 (*)

@sbernauer
sbernauer requested a review from lfrancke November 3, 2026 09:35

@NickLarsenNZ NickLarsenNZ left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Saw Lars' approval in slack

@sbernauer sbernauer moved this to Development: In Review in Stackable Engineering Nov 3, 2026
@sbernauer

Copy link
Copy Markdown
Member Author

Thanks @NickLarsenNZ. I'll wait for a bit (we need to wait for the next rollout anyway) to give time for an objection 😅

@NickLarsenNZ

Copy link
Copy Markdown
Member

I believe the only objection would come from Lars (and he has approved it) 😅

@sbernauer
sbernauer added this pull request to the merge queue Nov 3, 2026
@sbernauer sbernauer moved this from Development: In Review to Development: Done in Stackable Engineering Nov 3, 2026
Merged via the queue into main with commit de006c1 Nov 3, 2026
2 checks passed
@sbernauer
sbernauer deleted the chore/apache2-llvm-license branch November 3, 2026 12:06
sbernauer added a commit to stackabletech/trino-operator that referenced this pull request Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

2 participants