Skip to content

Conversation

@deepak1556
Copy link
Collaborator

@deepak1556 deepak1556 commented Jan 9, 2025

Fixes #237437
Fixes #238609

@deepak1556 deepak1556 self-assigned this Jan 9, 2025
@deepak1556 deepak1556 added this to the February 2025 milestone Jan 9, 2025
@deepak1556 deepak1556 force-pushed the electron-34.x.y branch 2 times, most recently from 2b1d33d to 93764df Compare January 10, 2025 10:39
@bpasero
Copy link
Member

bpasero commented Jan 11, 2025

@deepak1556 do we need to act on this breaking change for custom protocol handlers: https://www.electronjs.org/docs/latest/breaking-changes#behavior-changed-custom-protocol-url-handling-on-windows ?

@deepak1556 deepak1556 force-pushed the electron-34.x.y branch 3 times, most recently from de464b5 to 373539b Compare January 16, 2025 04:38
@deepak1556
Copy link
Collaborator Author

Thanks @bpasero , the issue only affects non standard custom schemes and data urls. The ones we use vscode-file, vscode-webview for loadURL are standard schemes, will continue to work as expected.

@deepak1556 deepak1556 force-pushed the electron-34.x.y branch 6 times, most recently from 1d3cfe2 to 52b5822 Compare January 24, 2025 03:36
@deepak1556 deepak1556 force-pushed the electron-34.x.y branch 2 times, most recently from 2c55d08 to f4a78c4 Compare February 11, 2025 13:18
@deepak1556 deepak1556 requested a review from bpasero February 11, 2025 19:56
@deepak1556 deepak1556 marked this pull request as ready for review February 11, 2025 19:56
bpasero
bpasero previously approved these changes Feb 11, 2025
@deepak1556 deepak1556 merged commit f94b270 into main Feb 12, 2025
8 checks passed
@deepak1556 deepak1556 deleted the electron-34.x.y branch February 12, 2025 09:36
@alerque
Copy link

alerque commented Feb 12, 2025

Any comments on an expected time frame for this to be released? Not to pester, I'm just evaluating how to respond to the backport requests to apply this as a patch to the existing release version on Arch Linux.

@deepak1556
Copy link
Collaborator Author

Unless we need to revert for major regressions, this change will be available in our 1.98 release in the beginning of March.

@samnn
Copy link

samnn commented Feb 22, 2025

Any comments on an expected time frame for this to be released? Not to pester, I'm just evaluating how to respond to the backport requests to apply this as a patch to the existing release version on Arch Linux.

With the icu update it is now relatively difficult to fix paste for code. Downgrading electron32 requires several dependencies be downgraded, including icu. Downgrading icu breaks the system badly.

Attempting to downgrade code itself to 1.94.2 to fix the issue also runs into an icu dependency issue.

@yannleretaille
Copy link

With the icu update it is now relatively difficult to fix paste for code. Downgrading electron32 requires several dependencies be downgraded, including icu. Downgrading icu breaks the system badly.

Attempting to downgrade code itself to 1.94.2 to fix the issue also runs into an icu dependency issue.

@samn For arch users, I recommend installing icu75 from the AUR. You might still need to downgrade flac to 1.4.0 (and potentially packages that depend on it) as well, but that does not cause the aforementioned system breakage.

On my system, I was able to fix this by installing icu75:

# paru/yay -S icu75

and downgrading the following packages:

# sudo downgrade electron32 flac audacity electron27 electron31 electron33 libsndfile

to the following versions:

audacity (1:3.7.1-2 => 1:3.7.1-1)
electron27 (27.3.11-9 => 27.3.11-8)
electron31 (31.7.7-3 => 31.7.7-1)
electron32 (32.3.1-3 => 32.2.8-3)
electron33 (33.4.1-3 => 33.4.0-1)
flac (1.5.0-1 => 1.4.3-2)
libsndfile (1.2.2-3 => 1.2.2-2)

You can check which packages need to be downgraded by just running sudo downgrade electron32 and selecting version 32.2.8-3.

@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Mar 29, 2025
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.

Paste no longer works with Electron >= 32.3.0 Electron 34 update

7 participants