Skip to content

Revert "Add built-in GUI to display license notices"#103906

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
adamscott:revert-builtin-gui-license-notice
Mar 11, 2025
Merged

Revert "Add built-in GUI to display license notices"#103906
Repiteo merged 1 commit intogodotengine:masterfrom
adamscott:revert-builtin-gui-license-notice

Conversation

@adamscott
Copy link
Member

@adamscott adamscott commented Mar 10, 2025

After internal discussion based on the feedback received, this reverts commit daa6198 / #79599.

I will work on a follow-up PR to achieve the same goal.

Special thanks to @Calinou for their original PR. We need something like this in order to make games made with Godot at least more compliant.

Fixes #103830 (partially).

@adamscott adamscott requested review from a team as code owners March 10, 2025 14:42
@Calinou Calinou added this to the 4.x milestone Mar 10, 2025
Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Given the pushback on the original implementation, I agree we should take a step back and rework the proposal.

I still think we need to do more to make it trivial for users to comply with third-party licenses Godot relies on, but let's go back to the drawing board and make a proposal that takes into account the user feedback.

@akien-mga akien-mga modified the milestones: 4.x, 4.5 Mar 10, 2025
@Repiteo Repiteo merged commit 78c9f8d into godotengine:master Mar 11, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Mar 11, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The third-party license window does not accomplish its intentions, and should be an addon instead of in core.

4 participants