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

dependencies: update glfw to 20200625191551 #1136

Merged
merged 2 commits into from Jun 26, 2020
Merged

dependencies: update glfw to 20200625191551 #1136

merged 2 commits into from Jun 26, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jun 26, 2020

changes in the glfw:

  • pr-285 Fixes a compiler error by using null joystick on freebsd
  • pr-282 fix regression from pr-256 GetClipboardString No longer returns an error
  • pr-280 fix: go mod vendor
  • pr-273 Added missing functions for compatibility with Vulkan
  • pr-270 .travis.yml: use latest stable Go (1.13.8 and 1.12.17 at this time)

Description:

pulling in external dependency with the fixes preventing fyne crashes

Fixes #(issue)
#1132
#1123
#743 (claimed as fixed but the real fix is in the update that is missing)

Checklist:

  • Tests included.
  • Lint and formatter run with no errors. (no code changes, just deps)
  • Tests all pass.

Where applicable:

  • Public APIs match existing style.
  • Any breaking changes have a deprecation path or have been discussed.

  changes:
  pr-285 Fixes a compiler error by using null joystick on freebsd
  pr-282 fix regression from pr-256 GetClipboardString No longer returns an error
  pr-280 fix: go mod vendor
  pr-273 Added missing functions for compatibility with Vulkan
  pr-270 .travis.yml: use latest stable Go (1.13.8 and 1.12.17 at this time)
@andydotxyz
Copy link
Member

I think the vendor files must be out of date after this? Using “fyne vendor” will ensure the updated C and H files are included as well as the Go.

@ghost
Copy link
Author

ghost commented Jun 26, 2020

if i understood you correctly that was my mistake in he first try.
i've made force-push to update vendor\modules.txt in the same commit.
strange that only one build has failed because of that.

@andydotxyz
Copy link
Member

If you look at the changes tab for this PR there are 0 code changes present, so something doesn’t quite add up... the vendor folder should reflect all the code improvements you have listed I would think?

@ghost
Copy link
Author

ghost commented Jun 26, 2020

oh! i missed vendor is in the repo and not received basing on the txt file. sorry
made that change

Copy link
Member

@andydotxyz andydotxyz left a comment

Choose a reason for hiding this comment

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

Great thanks, let's get testing the related issues now :)

@andydotxyz andydotxyz merged commit 2c280e7 into fyne-io:develop Jun 26, 2020
@ghost ghost deleted the dependencies-update-glfw branch June 26, 2020 11:16
Jacalz added a commit to Jacalz/fyne that referenced this pull request Nov 23, 2023
See fyne-io#1136. The clipboard no longer crashes when pasting empy content (also verified locally). The clipboard also no longer require access to the window struct. Clean up the code. As the clipboard no longer needs the viewport, we can also remove a possible crash when getting a clipboard from a window that has yet to be set up.
@Jacalz Jacalz mentioned this pull request Nov 23, 2023
3 tasks
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

1 participant