-
Notifications
You must be signed in to change notification settings - Fork 9
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
how to disable NO_LICENSE_IN_DEPENDENCY
?
#40
Comments
For this, ORT has the concept of package curations. Community-maintained curations are part of the
It's actually the opposite: If something does not explicitly state a license, you must not use it at all. That's common legal interpretation and part of many Open Source usage guidelines, like e.g. the one from Zalando which states "You cannot use unlicensed code anywhere".
If you still want to ignore this error (which is highly discouraged), then you can do so via rule violation resolutions. |
@sschuberth great explanation, thanks! I can't figure out how to provide the location of |
The project's README is the wrong location for user-facing guides. Instead, please refer to the docs on the website. |
@sschuberth this page is not related to the GitHub Actions plugin, if I'm not mistaken. It explains how ORT can be used inside Jenkins, via manual configuration. The README file, instead, explains how to configure YAML for GHA -- that's why I'm reading it (it's pretty well written, by the way!) |
Well, those are the generic docs that apply to any way ORT is run. Also when running ORT via the GitHub action, it looks for its configuration files (and that includes package curations) in the default locations. That said, the GitHub action could have some special additional logic that I'm not aware of, as I'm not maintaining it. @tsteenbe can probably tell you more. But I'd assume you two options are: Put curations inside of an |
@sschuberth placing curations into the
What is my next step? How to allow curations in the This is my
|
Again see the docs, it's in the big "NOTE" box at the top: You need to enable the
|
@sschuberth once again, thanks for your help! I did what you suggested, but the warning is still the same: log. Any ideas? |
@yegor256 If you want to disable a rule you need set up your own ort-config repository with your own rules and then set Your ort,yml is incorrect - in meetings now but will shoot in a PR to your repo to fix things |
@tsteenbe thanks! Is it possible to enable some curations without creating a new repository? (this looks like an overkill). Would be great if your fix to our |
@yegor256 Fixed up your .ort.yml in yegor256/tojos#110 - note that the ort scan will still with PACKAGE_CURATION_IN_ORT_YML - The use of package curations is not allowed in the *.ort.yml file rule violations. In the rules in our ort-config repository we do not allow package curations in .ort.yml. I will make a PR against ort-config with my curations so curations in your .ort.yml are no longer needed. An alternative is that you clone ort-config within your own github.com/yegor256 and then you can change whatever you want but you loose the benefit of easily getting curations from other ORT users. This also why we block in the ort-config policy rules the use curations in the .ort.yml - curations in .ort.yml are only applied to one project instead of being reusable by anyone (within your org/world). |
OK I fixed things up
@yegor256 Once 2 is merged then PR in 3 should pass |
@tsteenbe just curious, why do we need (2) if (1) will be fixed? |
@yegor256 The fix in 1) is not yet part of a JQF release so 2) is needed to fix the declared licenses for all versions before the JQF release containing 1. Your project github.com/yegor256/tojos is currently using verions of JQF that have not be fixed with 1 so 2) fixes that up. In ORT community we try to fix up issue upstream if time permits so future release of FOSS projects won't have problems we detected with ORT. Curations and package configurations are mechanism in ORT to patch up already publish releases - you can't go FOSS maintainers to ask "Could you please fix up all your past release so licensing is right". |
I'm using this plugin with a pretty large repository, where we have many third-party dependencies, some of which simply don't have licenses. There is no possible way to fix that, since they are not ours. Moreover, if they don't have license, we can safely assume that they don't care who and how is using their products.
Thus, the question: how can I ignore this particular type of error?
The text was updated successfully, but these errors were encountered: