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

Update x11-dl #378

Merged
merged 1 commit into from
Jan 9, 2018
Merged

Update x11-dl #378

merged 1 commit into from
Jan 9, 2018

Conversation

GabrielMajeri
Copy link
Contributor

Fixes #376.

@tomaka
Copy link
Contributor

tomaka commented Jan 8, 2018

This PR is unfortunately not very useful because a user can also apply the fix by running cargo update.
If we merge this PR and bump winit, then the user will have to run cargo update as well anyway.

@GabrielMajeri
Copy link
Contributor Author

@tomaka I understand, should we add a warning to the README or something? Or is there any way to force Cargo to update a dependency? If we yank the old version, will it display a warning message?

@tomaka
Copy link
Contributor

tomaka commented Jan 8, 2018

I suppose one solution would be to yank the old version of x11-dl.

@elinorbgr
Copy link
Contributor

elinorbgr commented Jan 9, 2018

Still, we can merge this PR (without necessarily bumping winit version now). This will ensure any new dependency on winit will not accidentally pull a bad version of x11-dl. No?

@tomaka tomaka merged commit dddd9de into rust-windowing:master Jan 9, 2018
@GabrielMajeri GabrielMajeri deleted the update-deps branch January 9, 2018 11:47
tmfink pushed a commit to tmfink/winit that referenced this pull request Jan 5, 2022
Fix Metal memory management and the `canvas_metal_minimal`, `canvas_nanovg`, and `macos_app` examples.

`winit` does not create an autorelease pool, so the Metal backend had not taken
the presence of one into account. Now the Metal backend creates and flushes
autorelease pools as necessary.

Closes rust-windowing#334.
Closes rust-windowing#376.
madsmtm pushed a commit to madsmtm/winit that referenced this pull request Jun 11, 2022
Co-authored-by: amrbashir <amrbashir@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants