Skip to content

Conversation

@pboling
Copy link
Contributor

@pboling pboling commented Sep 13, 2025

  • regression test: Ruby library projects with no runtime dependencies do not include development dependencies

…encies are empty

- regression test: Ruby library projects with no runtime dependencies do not include development dependencies
@pboling pboling force-pushed the feat/library-ignores-dev-dependency branch from d429d13 to 7ef9083 Compare September 13, 2025 07:54
@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

@kezhenxu94 @wu-sheng this is done!

Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

Cool!

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

@kezhenxu94 @wu-sheng I sent an email requesting to be added to the SkyWalking slack. 😸

@kezhenxu94 kezhenxu94 merged commit 18cb362 into apache:main Sep 13, 2025
1 check passed
@pboling pboling deleted the feat/library-ignores-dev-dependency branch September 13, 2025 08:14
@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

@kezhenxu94 This is so cool! I am not sure what I need to do next though. It is still failing.

INFO Loading configuration from file: .licenserc.yaml 
ERROR the following licenses are unknown or incompatible with the main license, please check manually: MIT
Dependency       | License
---------------- | -------
faraday          |     MIT
multi_xml        |     MIT
version_gem      |     MIT
faraday-net_http |     MIT
jwt              |     MIT
rack             |     MIT
snaky_hash       |     MIT
hashie           |     MIT
git              |     MIT
json             |    Ruby 
ERROR one or more errors occurred checking license compatibility 
Error: Process completed with exit code 1.

Ref: https://github.com/ruby-oauth/oauth2/actions/runs/17692969800/job/50291221524#step:3:129

IMO, these licenses are all compatible with MIT, especially the ones that are MIT. 😆

Perhaps we need to configure the MIT license as compatible with other licenses?

@kezhenxu94
Copy link
Member

Perhaps we need to configure the MIT license as compatible with other licenses?

Yes exactly! We need to config the Ruby license (if it’s a license) to be compatible with MIT, AND to make a license X compatible with itself X 😄

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

Ruby license is a license, and many of the standard ruby libraries (default, bundled, core) use it!
https://stdgems.org/compare/
I'll work on another PR :)
Did you see my request to join the slack?

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

Also, the ruby license doesn't have a standard license header... is that a problem?

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

@kezhenxu94 Where can I find documentation on the difference between compatible and weak-compatible?

@kezhenxu94
Copy link
Member

@kezhenxu94 Where can I find documentation on the difference between compatible and weak-compatible?

https://www.apache.org/legal/resolved.html#category-b this is ASF specific, the weak-compatibility might not be applicable to other license, though.

@kezhenxu94
Copy link
Member

Also, the ruby license doesn't have a standard license header... is that a problem?

In terms of dependency license check, that’s not a problem. For license header, we can just use the spdx id as the license header.

@kezhenxu94
Copy link
Member

Did you see my request to join the slack?

I don’t see any email, did you sent to [email protected]?

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

apache.org/legal/resolved.html#category-b this is ASF specific, the weak-compatibility might not be applicable to other license, though.

  1. True, but it is the best information we have to build on, and since this project isn't a lawyer it must sit on someone's shoulders.
  2. Logically if an Apache project can include code from any and all licenses from category A in a compatible manner, it means they are all compatible with each other according to their definition of compatible.
  3. If we don't have a better definition of compatible we should use theirs.
  4. Similar with the weak-compatible. If any and all of the weak compatible licenses can be included (for binary code) in an Apache project, it means they must be, according to their definition, compatible with each other, not just with Apache.
  5. This is an Apache project! So I think it would be reasonably expected to give primacy to the ASF definitions of compatible. :)

I don’t see any email, did you sent to [email protected]?

Yes. I will send it again!

@pboling
Copy link
Contributor Author

pboling commented Sep 13, 2025

Sent another request. Subject Request to join SkyWalking slack, and it is from my public gmail: peter.boling at gmail dot com

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

Successfully merging this pull request may close these issues.

2 participants