Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[scarthgap] License issues checking by SPDX License. #1216

Open
mark-bchoi opened this issue Aug 27, 2024 · 5 comments
Open

[scarthgap] License issues checking by SPDX License. #1216

mark-bchoi opened this issue Aug 27, 2024 · 5 comments

Comments

@mark-bchoi
Copy link

Hi,

There are many license issues by filtered by SPDX license policy which would be added by scarthgap branch. I modified them manually but, I am not sure it is correct approach.

License issues by SPDX license , for example, one of them :

NOTE: Executing Tasks
ERROR: apriltag-3.4.2-1-r0 do_create_spdx: Cannot find any text for license BSD
ERROR: Logfile of failure stored in: /home/bchoi/nvidia-yocto-bsp/bchoi-build/tmp/work/armv8a-oe4t-linux/apriltag/3.4.2-1/temp/log.do_create_spdx.1328371
ERROR: Task (/home/bchoi/nvidia-yocto-bsp/layers/meta-ros/meta-ros2-humble/generated-recipes/apriltag/apriltag_3.4.2-1.bb:do_create_spdx) failed with exit code '1'
NOTE: Tasks Summary: Attempted 5984 tasks of which 5959 didn't need to be rerun and 1 failed.

Summary: 1 task failed:
  /home/bchoi/nvidia-yocto-bsp/layers/meta-ros/meta-ros2-humble/generated-recipes/apriltag/apriltag_3.4.2-1.bb:do_create_spdx
Summary: There was 1 ERROR message, returning a non-zero exit code.

This error was from LICENSE = "BSD" which was not problematic for the previous branch, kirkstone.

Can you please let me know whether there is some way to turn checking SPDX license off or not?

My bitbake layer for ros2 and openembedded commit ids are :

meta-ros2-humble     = "HEAD:b96ba3d66488aca5868fa1a94643a7f52b2cfc07"
meta                 = "HEAD:bf3fe8c01c5cc00ada22049f4f0abb485e2a626f"

BR,
Mark

@mark-bchoi
Copy link
Author

I could change LICENSE = "BSD" to LICENSE = "0BSD" with command in advance.

grep -rn LICENSE\ \=\ \"BSD\" * | cut -d : -f 1 | xargs sed -i s/LICENSE\ \=\ \"BSD\"/LICENSE\ \=\ \"0BSD\"/g

@jiaxshi
Copy link
Contributor

jiaxshi commented Aug 28, 2024

A simple workaround is:

poky/meta$ cp files/common-licenses/BSD-3-Clause files/common-licenses/BSD

@yangpeng1995
Copy link

The community removed the BSD license in Scarthgap, so this broad license cannot be used in the future
https://git.yoctoproject.org/poky/commit/?h=scarthgap&id=56fe6d575e184ed44807fe27e069233da36962ad

The correct solution would be to change the license to concrete names such as "BSD-3-Clause".

Not to rename the BSD-3-Clause to BSD...

@jiaxshi
Copy link
Contributor

jiaxshi commented Sep 11, 2024

The community removed the BSD license in Scarthgap, so this broad license cannot be used in the future https://git.yoctoproject.org/poky/commit/?h=scarthgap&id=56fe6d575e184ed44807fe27e069233da36962ad

The correct solution would be to change the license to concrete names such as "BSD-3-Clause".

Not to rename the BSD-3-Clause to BSD...

As i mentioned in #1208, there are multiple BSD variant used by ROS packages. It's good to have a method to parse and use accurate LICENSEN in superflore process.

@shr-project
Copy link
Contributor

Please see ros-infrastructure/catkin_pkg#296 ros/rosdistro#26601 and ros-infrastructure/superflore#279

I don't work on ROS anymore but it indeed needs to be fixed by ROS component owners and QA check in catkin was supposed to incrementally motivate them to update license when updating the version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants