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

Electron 2.0.8 #57457

Closed
wants to merge 1 commit into from
Closed

Electron 2.0.8 #57457

wants to merge 1 commit into from

Conversation

K900
Copy link

@K900 K900 commented Aug 28, 2018

This fixes a crash on Linux systems running Glibc 2.28 (see electron/electron#13972, #55934 etc.). All the tests pass, and no more crashes on Arch Linux, even with the offending glibc patch un-reverted.

This fixes a crash on Linux systems running Glibc 2.28
(see electron/electron#13972)
@msftclas
Copy link

msftclas commented Aug 28, 2018

CLA assistant check
All CLA requirements met.

@K900
Copy link
Author

K900 commented Aug 28, 2018

Also, this might be worth backporting to stable, but I'd leave that decision up to the maintainers.

@bpasero bpasero self-assigned this Aug 29, 2018
@bpasero bpasero added this to the September 2018 milestone Aug 29, 2018
@bpasero
Copy link
Member

bpasero commented Aug 29, 2018

VSCode is using a custom Electron build for the official releases and we have not yet updated that to 2.0.8 because we are waiting for a possible 2.0.9 release that includes more changes we want. We cannot accept this PR until we have done these updates to our build.

@bpasero bpasero modified the milestones: September 2018, Backlog Sep 1, 2018
@jonathangaldino
Copy link

I'm waiting on this PR since I can't use code. @K900 can you pass me the build?

@bpasero
Copy link
Member

bpasero commented Sep 10, 2018

I am closing this, we want to pick up 2.0.9 when its out. This should happen during this month 👍

@bpasero bpasero closed this Sep 10, 2018
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants