Declare BSD-3-Clause in strided-perm license for the HPTT-derived module#144
Merged
Conversation
The strided-perm/src/hptt/ module follows the structure of HPTT's C++ implementation (file comments map to specific transpose.cpp line ranges), so treat it as derived on the safe side: - strided-perm Cargo.toml license set to "(MIT OR Apache-2.0) AND BSD-3-Clause" - THIRD-PARTY-LICENSES: replace the overstated "independent reimplementation; no C++ source code was copied" claim with an accurate description and state the license split The BSD-3-Clause text and the ARRAY 2017 citation were already present in THIRD-PARTY-LICENSES; no change there. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
shinaoka
enabled auto-merge (squash)
July 21, 2026 21:44
Add HPTT to the root README acknowledgments and license note, and add an Acknowledgments and License section to the strided-perm README stating the derivation and the (MIT OR Apache-2.0) AND BSD-3-Clause split. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The transpose engine in
strided-perm/src/hptt/follows the structure of HPTT's C++ implementation closely (module comments map to specifictranspose.cppline ranges), while HPTT's SIMD kernels, autotuning plan search, and parallelism heuristics were not ported. To stay on the safe side of the derived-work question:strided-permlicensefield changed from the workspaceMIT OR Apache-2.0to the SPDX expression(MIT OR Apache-2.0) AND BSD-3-Clause(validated withcargo metadata)THIRD-PARTY-LICENSES: the HPTT entry claimed "This is an independent Rust reimplementation; no C++ source code was copied", which overstated independence given the line-range correspondence comments. Replaced with an accurate description that also explains the license split.The BSD-3-Clause text ("Copyright 2018 Paul Springer") and the ARRAY 2017 paper citation were already in
THIRD-PARTY-LICENSES, so the substantive BSD compliance was in place; this PR aligns the claims and the package metadata with it. Upstream HPTT was verified to be fully BSD-3-Clause with no LGPL remnants (the LGPLv3 period in its history was cleaned up by the final upstream commit in 2022).Text and metadata only; no code changes.
Test plan
cargo fmt --checkcargo test --workspace(all green)cargo metadataparses the new SPDX expression🤖 Generated with Claude Code