-
Notifications
You must be signed in to change notification settings - Fork 33
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
Ignore not-yet-decided IPLab issues #419
Comments
I've approved the issue on behalf of the IP Team so that you're not blocked. The line will still change because the authority of the approval has changed; this is by design and I don't intend to change it. The first thing that I need to sort out is why the new issue was created in the first place. If you got an In parallel, I'll investigate what it would take to introduce a third |
There were actually 20 new IPLab issues created for dependencies previously approved by ClearlyDefined, see https://gitlab.eclipse.org/eclipse/escet/escet/-/issues/1088#note_2934129. 18/20 got auto-approved. Only 2/20 were restricted dependencies, of which one still remains: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/17667. |
The IP Team has processed that one as well. So, AFAIK, you are not blocked at this time. |
Indeed. Thanks! |
I get the following diff on Dash license check output in the Eclipse ESCET project:
Previously, the dependency (that we get via Eclipse Platform or so), was OK via ClearlyDefined. Now, it is no longer OK. Now, there is an IPLab issue. I don't know who created it, or why, but I didn't. The new IPLab issue is currently under evaluation, so there is no conclusion. This is now interpreted by the Dash license check tool as being restricted.
I think such not-yet-decided IPLab issues should be ignored. They don't forbid the use of the dependency, and ClearlyDefined should then be queried. That is, a not-yet-decided IPLab issue should be treated like a non-existing IPLab issue, I think.
With this change, the automated dependency check wouldn't fail in these kind of situations.
The text was updated successfully, but these errors were encountered: