Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Crate proc-macro-warning adds GPL 3.0 license dependency #13872

Closed
2 tasks done
haerdib opened this issue Apr 11, 2023 · 3 comments · Fixed by #13876
Closed
2 tasks done

Crate proc-macro-warning adds GPL 3.0 license dependency #13872

haerdib opened this issue Apr 11, 2023 · 3 comments · Fixed by #13876
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.

Comments

@haerdib
Copy link
Contributor

haerdib commented Apr 11, 2023

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

Description of bug

The newly added proc-macro-warning crate adds a GPL 3.0 dependency to substrate (added in this PR)
To the best of my knowledge (not a lawyer) this violates the GPL v3.0 license.

Steps to reproduce

Noticed after a cargo update via license check in our CI. Check out https://github.com/scs/substrate-api-client/actions/runs/4664840889/jobs/8259110692

@github-actions github-actions bot added the J2-unconfirmed Issue might be valid, but it’s not yet known. label Apr 11, 2023
@bkchr
Copy link
Member

bkchr commented Apr 11, 2023

CC @ggwpez

ggwpez added a commit to ggwpez/proc-macro-warning that referenced this issue Apr 11, 2023
Turns out we need this for Substrate, see
paritytech/substrate#13872

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
ggwpez added a commit that referenced this issue Apr 11, 2023
Closes #13872

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez
Copy link
Member

ggwpez commented Apr 11, 2023

We should probably add that automated dependency license check to our CI as well, WDYT @bkchr?

@bkchr
Copy link
Member

bkchr commented Apr 11, 2023

I think we once had one, not sure what happened to it.

bkchr pushed a commit that referenced this issue Apr 13, 2023
* Update proc-macro-warning

Closes #13872

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update to v0.3.1 (including syn 2.0)

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Revert "Update to v0.3.1 (including syn 2.0)"

CI is red since it Polkadot uses a different syn version…
Going to update it lean.

This reverts commit 3240d37.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
nathanwhit pushed a commit to nathanwhit/substrate that referenced this issue Jul 19, 2023
* Update proc-macro-warning

Closes paritytech#13872

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update to v0.3.1 (including syn 2.0)

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Revert "Update to v0.3.1 (including syn 2.0)"

CI is red since it Polkadot uses a different syn version…
Going to update it lean.

This reverts commit 3240d37.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
taqtiqa-mark pushed a commit to subversive-upstream/polkadot-sdk-substrate-frame-support that referenced this issue Sep 14, 2023
* Update proc-macro-warning

Closes paritytech/substrate#13872

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Update to v0.3.1 (including syn 2.0)

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Revert "Update to v0.3.1 (including syn 2.0)"

CI is red since it Polkadot uses a different syn version…
Going to update it lean.

This reverts commit 3240d379b72f6f722eaf502ed49e9a1b0f79db4b.

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
J2-unconfirmed Issue might be valid, but it’s not yet known.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants