Skip to content
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

sublime-text4: update license_checksum #37589

Closed
wants to merge 1 commit into from

Conversation

iotku
Copy link

@iotku iotku commented Jun 16, 2022

Testing the changes

  • I tested the changes in this PR: YES

Previously, the build would fail as the checksum of the license has changed.

I'm not aware of any notice of changes to the text, so it's likely just a minor formatting change in the html which is being check-summed. Additionally, sublime-merge-bin (sublime-text's sister project) uses the same EULA and respective checksum as provided here.

Pull request resubmitted to update the package revision as the package version has not changed.

@classabbyamp
Copy link
Member

I'm not aware of any notice of changes to the text, so it's likely just a minor formatting change in the html which is being check-summed. Additionally, sublime-merge-bin (sublime-text's sister project) uses the same EULA and respective checksum as provided here.

The change was adding an allowance for redistribution of the linux version of the software.

Pull request resubmitted to update the package revision as the package version has not changed.

no need to do that in the future, you can use git commit --amend and git push --force to update the PR

@classabbyamp
Copy link
Member

I'll take care of this in #37593, there are a few more changes that I wanted to make due to the EULA change

@iotku iotku deleted the sublime-text4 branch June 18, 2022 23:35
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.

None yet

2 participants