Skip to content

Conversation

@bfops
Copy link
Collaborator

@bfops bfops commented Jul 29, 2025

Description of Changes

We recently merged several repos together. This PR clarifies the license terms for several subdirectories, as well as the relationship between the licenses.

The licenses in our subdirectories have become symbolic links to licenses in our toplevel licenses directory. For any particular subdirectory's license file in the diff, you can click ... -> View file and then click on the text that says "Symbolic Link" on that page. This will take you to the license file that it links to.

I have also updated the tools/upgrade-version script to update the change date in the new licenses/BSL.txt file.

API and ABI breaking changes

None.

Expected complexity level and risk

1

Testing

None. Only changes to license files.

@bfops bfops requested a review from tlefky August 7, 2025 21:44
@bfops bfops marked this pull request as ready for review August 7, 2025 21:48
@bfops bfops requested a review from cloutiertyler as a code owner August 7, 2025 21:48
@bfops bfops added release-any To be landed in any release window no runtime change This change does not affect the final binaries labels Aug 11, 2025
Copy link
Contributor

@cloutiertyler cloutiertyler left a comment

Choose a reason for hiding this comment

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

LGTM, two comments:

  1. We might want to consider making the Rust SDK dependencies Apache 2.0, but we can do that as needed later.
  2. We want to make the Rust SDK Apache 2.0, but we can do that in the separate PR that changes the directory.

@bfops bfops added this pull request to the merge queue Aug 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 12, 2025
@bfops bfops added this pull request to the merge queue Aug 12, 2025
Merged via the queue into master with commit f6f0909 Aug 12, 2025
25 checks passed
@bfops bfops deleted the bfops/license-update branch August 19, 2025 21:07
github-merge-queue bot pushed a commit that referenced this pull request Aug 22, 2025
# Description of Changes

Apparently, I missed several license files in #3002. I'm not sure what
method I was using to find them, but apparently it was insufficient.

**This replaces all empty `LICENSE` files with an explicit (symlink to)
BSL license, and all apache licenses with symlinks to the root apache
license.** This PR does not intentionally change any license terms, so
if you see one that changed, **it's a mistake**.

# API and ABI breaking changes

None

# Expected complexity level and risk

1

# Testing

```bash
$ find . -name '*LICENSE*' -type f | grep -v '\.meta$'
./crates/sqltest/standards/LICENSE # this one is an external library that we are not allowed to re-license
./LICENSE.txt # this is the root license
```

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
bfops added a commit that referenced this pull request Sep 23, 2025
# Description of Changes

We recently merged several repos together. This PR clarifies the license
terms for several subdirectories, as well as the relationship between
the licenses.

The licenses in our subdirectories have become symbolic links to
licenses in our toplevel `licenses` directory. For any particular
subdirectory's license file in the diff, you can click `... -> View
file` and then click on the text that says "Symbolic Link" on that page.
This will take you to the license file that it links to.

I have also updated the `tools/upgrade-version` script to update the
change date in the new `licenses/BSL.txt` file.

# API and ABI breaking changes

None.

# Expected complexity level and risk

1

# Testing

None. Only changes to license files.

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no runtime change This change does not affect the final binaries release-any To be landed in any release window

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants