Skip to content

SPDX [ 40 ][ Src / Mod / Robot ]#25115

Merged
chennes merged 1 commit intoFreeCAD:mainfrom
PhoneDroid:SPDX-Src-Mod-Robot
Dec 25, 2025
Merged

SPDX [ 40 ][ Src / Mod / Robot ]#25115
chennes merged 1 commit intoFreeCAD:mainfrom
PhoneDroid:SPDX-Src-Mod-Robot

Conversation

@PhoneDroid
Copy link
Member

Added missing SPDX license identifiers.

@PhoneDroid PhoneDroid moved this to Open in 📜 Licensing Nov 6, 2025
@maxwxyz maxwxyz added the Type: Other Tasks, refactoring, quality improvement, actions, dependencies, packaging, translations or other... label Nov 6, 2025
@maxwxyz maxwxyz added this to the 1.1 milestone Nov 6, 2025
@PhoneDroid PhoneDroid marked this pull request as draft November 6, 2025 23:02
@PhoneDroid PhoneDroid marked this pull request as ready for review November 9, 2025 05:31
@maxwxyz maxwxyz moved this to Queue in Merge Queue Nov 9, 2025
@maxwxyz maxwxyz moved this from Queue to Merge Meeting in Merge Queue Nov 9, 2025
@@ -1,3 +1,5 @@
// SPDX-License-Identifier: LGPL-2.1-or-later
Copy link
Member

Choose a reason for hiding this comment

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

Considering the license history of the Robot workbench, I took a closer look at this one. I believe that it comes from here: https://github.com/orocos/orocos_kinematics_dynamics/blob/fb840b0edb3e9905c354276d6513dddf4fd4ac1b/orocos_kdl/src/frameacc.inl -- so I can confirm that LGPL-2.1 is correct.

@chennes
Copy link
Member

chennes commented Nov 12, 2025

It's not obvious to a casual inspection that the entire kdl_cp subdirectory is really a 3rd party library -- it's been totally separated from its provenance. For future work we should consider adding some kind of acknowledgement and backlink. Or maybe someone like @onekk would like to take on making it a subtree?

@onekk
Copy link

onekk commented Nov 12, 2025

From the little I know the kdl library was patched by jregel (I have read this info in one of the old posts, but I don't remember where). So it is not an exact copy of the upstream, that in the meantime has evolved a lot.

I don't code in C++ as my knowledge is very limited to the minimum to read some not too complicated sources.

as the kdl is the core of the inverse kinematics probably some quite deep knowledge of the matter is needed to think to update the sources to a more recent upstream code.

kind regards
Carlo D.

@maxwxyz maxwxyz added backport releases/FreeCAD-1-1 Applied to a PR that is on main to trigger the automatic creation of another PR onto 1.1 and removed backport releases/FreeCAD-1-1 Applied to a PR that is on main to trigger the automatic creation of another PR onto 1.1 labels Nov 14, 2025
@maxwxyz maxwxyz modified the milestones: 1.1, 1.2 Nov 18, 2025
@chennes chennes merged commit 4d0348d into FreeCAD:main Dec 25, 2025
16 checks passed
@github-project-automation github-project-automation bot moved this from Open to Merged in 📜 Licensing Dec 25, 2025
@github-project-automation github-project-automation bot moved this from Merge Meeting to Done in Merge Queue Dec 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Other Tasks, refactoring, quality improvement, actions, dependencies, packaging, translations or other...

Projects

Archived in project
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants