Skip to content

[codex] Add Apache license compliance metadata#2285

Merged
kkraus14 merged 2 commits into
NVIDIA:mainfrom
kkraus14:codex/apache-license-compliance
Jul 2, 2026
Merged

[codex] Add Apache license compliance metadata#2285
kkraus14 merged 2 commits into
NVIDIA:mainfrom
kkraus14:codex/apache-license-compliance

Conversation

@kkraus14

@kkraus14 kkraus14 commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • add a repository-level Apache 2.0 license copy and NVIDIA copyright preambles to the Apache-licensed component license files
  • document the DCO sign-off workflow, including git commit -s and the resulting Signed-off-by trailer
  • add DLPack and Deprecated attributions to a cuda.core NOTICE file and include that file in source distributions
  • clarify the repository's mixed-license scope and link to the third-party notices

Why

The repository distributes Apache-licensed components alongside proprietary components and includes vendored third-party source in cuda.core. The existing files did not centralize all associated license metadata, attribution notices, or the contributor sign-off procedure.

Impact

This changes documentation and packaging metadata only; runtime behavior is unchanged. Built cuda-core packages now carry both LICENSE and NOTICE under their distribution license metadata.

Validation

  • git diff --check
  • repository pre-commit hooks; all applicable hooks pass
  • verified the Pixi cuda-core package build contains both LICENSE and NOTICE under dist-info/licenses

The lychee hook was skipped because the current repository configuration references a non-release rev and exits before checking links.

@kkraus14 kkraus14 added this to the cuda.core v1.1.0 milestone Jul 1, 2026
@kkraus14 kkraus14 added documentation Improvements or additions to documentation packaging Anything related to wheels or Conda packages cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module labels Jul 1, 2026
@copy-pr-bot

copy-pr-bot Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@kkraus14 kkraus14 marked this pull request as ready for review July 1, 2026 14:33
@kkraus14 kkraus14 self-assigned this Jul 1, 2026
@github-actions

This comment has been minimized.

Comment thread LICENSE-Apache2.txt

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.

Q: Don't we have this license for both cuda-core and cuda-pathfinder?
https://github.com/NVIDIA/cuda-python/blob/main/cuda_core/LICENSE
Maybe we should just move it to the top-level and make every project point to the same license?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This will be cleaned up in the future regardless, so I'd be more keen to just keep this for now and clean it up when we make the future changes.

Comment thread LICENSE-Apache2.txt Outdated
Comment thread LICENSE.md
This repository contains components under different licenses. The
component-specific license files listed below govern each component. A
repository-level copy of the Apache 2.0 license is available in
[`LICENSE-Apache2.txt`](./LICENSE-Apache2.txt) for Apache-licensed content.

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.

ditto

Comment thread cuda_core/NOTICE
Comment on lines +15 to +18
Deprecated
Copyright (c) 2017 Laurent LAPORTE
Licensed under the MIT License.
Source: https://github.com/tantale/deprecated

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.

btw I've been meaning to ask... @mdboom @kkraus14 have we filed an OSRB for this? If not, we should do so. For OSS-licensed projects it's quick.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

We need to do this still.

kkraus14 added 2 commits July 1, 2026 22:18
Signed-off-by: Keith Kraus <keith.j.kraus@gmail.com>
Signed-off-by: Keith Kraus <keith.j.kraus@gmail.com>
@kkraus14 kkraus14 force-pushed the codex/apache-license-compliance branch from 0d8bd1a to 998cfc6 Compare July 2, 2026 02:19
@kkraus14 kkraus14 merged commit d9831c4 into NVIDIA:main Jul 2, 2026
200 of 202 checks passed
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

@kkraus14

kkraus14 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator Author

Sorry, this was an accidental merge. @leofang if you'd like me to revert, just let me know.

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

Labels

cuda.core Everything related to the cuda.core module cuda.pathfinder Everything related to the cuda.pathfinder module documentation Improvements or additions to documentation packaging Anything related to wheels or Conda packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants