Skip to content

Conversation

@AkihiroSuda
Copy link
Contributor

What type of PR is this?

  • bug (In my understanding)

What this PR does / why we need it:

Replaces https://github.com/antzucaro/matchr (GPL2, introduced in v2.7.0) with https://github.com/xrash/smetrics (MIT License).

This is needed because downstream projects may use GPL2-incompatible licenses such as Apache License 2.0.

Which issue(s) this PR fixes:

Fixes #1404

Release Notes

Remove GPL2 dependency introduced in v2.7.0

@AkihiroSuda
Copy link
Contributor Author

CI failure seems irrelevant to the commit

go: golang.org/x/[email protected] requires
	golang.org/x/[email protected]: missing go.sum entry; to add it:
	go mod download golang.org/x/tools

https://github.com/urfave/cli/runs/6567968972?check_suite_focus=true

@AkihiroSuda AkihiroSuda force-pushed the remove-gpl2-dependency branch from eee1066 to 805c7b2 Compare May 24, 2022 06:26
This commit replaces github.com/antzucaro/matchr (GPL2, introduced in v2.7.0)
with github.com/xrash/smetrics (MIT License).

Fixes issue 1404

Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda AkihiroSuda force-pushed the remove-gpl2-dependency branch from 805c7b2 to edb9cc4 Compare May 24, 2022 06:28
@AkihiroSuda AkihiroSuda force-pushed the remove-gpl2-dependency branch from edb9cc4 to 3e31c9b Compare May 24, 2022 06:30
@meatballhat meatballhat merged commit 5ff4e2a into urfave:main May 24, 2022
@meatballhat
Copy link
Member

@AkihiroSuda Thank you!

This was referenced Mar 5, 2025
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.

>= v2.7.0 can't be used with GPL2-incompatible projects

3 participants