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

chore: update to Electron 32 #218245

Merged
merged 41 commits into from
Oct 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
718a04b
chore: update [email protected]
deepak1556 Jul 17, 2024
14ef600
chore: bump [email protected]
deepak1556 Jul 17, 2024
b9fcdd1
chore: update sysroot download url
deepak1556 Aug 6, 2024
115e09f
chore: bump [email protected]
deepak1556 Aug 6, 2024
8530320
chore: update rpm deps for arm64
deepak1556 Aug 6, 2024
da879af
chore: bump [email protected]
deepak1556 Aug 23, 2024
bf31c0c
chore: bump [email protected]
deepak1556 Sep 4, 2024
07f039c
fix: compile stage
deepak1556 Sep 4, 2024
d9c36d8
fix: install on windows
deepak1556 Sep 4, 2024
d52606b
fix: override node-addon-api to support c++20 with gcc-11
deepak1556 Sep 4, 2024
7a5dc72
ci: use gnu++2a for gcc <=9
deepak1556 Sep 4, 2024
0539dcf
ci: fix sdl pipeline
deepak1556 Sep 4, 2024
ad0fb1e
ci: web stage
deepak1556 Sep 4, 2024
0bbe0b4
ci: fix alpine stage
deepak1556 Sep 4, 2024
79405a7
ci: fix legacy server stage
deepak1556 Sep 4, 2024
bf0c4ca
fix: node-addon-api resolution for remote
deepak1556 Sep 4, 2024
95dba4f
ci: fix legacy arm stage
deepak1556 Sep 4, 2024
999f497
chore: sync npm dependencies
deepak1556 Sep 10, 2024
1c9c02f
chore: bump [email protected]
deepak1556 Sep 10, 2024
d42132a
ci: fix monaco editor checks
deepak1556 Sep 10, 2024
98f2d8d
ci: fix legacy server stage
deepak1556 Sep 10, 2024
bc21d2c
ci: possible concurrent execution
deepak1556 Sep 10, 2024
1cbbb25
ci: use foreground scripts on windows
deepak1556 Sep 10, 2024
463a85f
chore: bump [email protected]
deepak1556 Sep 17, 2024
121b095
fix: compilation
deepak1556 Sep 18, 2024
a0f537a
Revert "wco - hardcode devtools location on Linux (#227084)"
deepak1556 Sep 18, 2024
37b05bd
chore: bump [email protected]
deepak1556 Sep 25, 2024
85122d5
ci: remove workaround for missing c++20 flags
deepak1556 Sep 25, 2024
740921e
ci: use gnu++2a for gcc <=9
deepak1556 Sep 26, 2024
c16b745
chore: bump distro
deepak1556 Oct 2, 2024
181c4c2
ci: use gnu++17 for legacy arm servers
deepak1556 Oct 2, 2024
cea5c99
fixup! override node-addon-api to support c++20 with gcc-11
deepak1556 Oct 2, 2024
ce3d6ff
chore: bump @vscode/policy-watcher
deepak1556 Oct 2, 2024
a8ce7b5
chore: bump @vscode/sqlite3
deepak1556 Oct 2, 2024
3ed3973
chore: bump node.js v20.17.0
deepak1556 Oct 2, 2024
78393e3
chore: bump distro
deepak1556 Oct 2, 2024
426f6d0
chore: update @vscode/windows-ca-certs
deepak1556 Oct 2, 2024
a1b4706
chore: update builds for [email protected]
deepak1556 Oct 3, 2024
fd7e7b1
chore: bump distro
deepak1556 Oct 3, 2024
7526f41
chore: bump dependencies
deepak1556 Oct 3, 2024
ddbc882
ci: fix arm legacy servers
deepak1556 Oct 4, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ jobs:
runs-on: windows-2022
timeout-minutes: 60
env:
CHILD_CONCURRENCY: "1"
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
steps:
- uses: actions/checkout@v4
Expand Down Expand Up @@ -57,6 +56,7 @@ jobs:
- name: Execute npm
if: ${{ steps.cacheNodeModules.outputs.cache-hit != 'true' }}
env:
npm_config_foreground_scripts: "true"
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
run: npm ci
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/rich-navigation.yml.off
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ jobs:
if: steps.caching-stage.outputs.cache-hit != 'true'
run: npm ci
env:
CHILD_CONCURRENCY: 1
npm_config_foreground_scripts: "true"

- uses: microsoft/[email protected]
with:
Expand Down
4 changes: 2 additions & 2 deletions .npmrc
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
disturl="https://electronjs.org/headers"
target="30.5.1"
ms_build_id="10306386"
target="32.1.2"
ms_build_id="10323173"
runtime="electron"
build_from_source="true"
legacy-peer-deps="true"
Expand Down
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
20.16.0
20.17.0
1 change: 1 addition & 0 deletions build/azure-pipelines/alpine/product-build-alpine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ steps:

- script: |
set -e

for i in {1..5}; do # try 5 times
npm ci && break
if [ $i -eq 5 ]; then
Expand Down
8 changes: 1 addition & 7 deletions build/azure-pipelines/darwin/product-build-darwin.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,16 +75,10 @@ steps:
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'), ne(variables['NPM_REGISTRY'], 'none'))
displayName: Setup NPM Authentication

- script: |
set -e
# Refs https://github.com/microsoft/vscode/issues/219893#issuecomment-2209313109
sudo xcode-select --switch /Applications/Xcode_15.2.app
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'))
displayName: Switch to Xcode >= 15.1

- script: |
set -e
c++ --version
xcode-select -print-path
python3 -m pip install setuptools

for i in {1..5}; do # try 5 times
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,28 @@ steps:
imageName: vscode-linux-build-agent:centos7-devtoolset8-$(VSCODE_ARCH)
containerCommand: uname

- ${{ if or(eq(parameters.VSCODE_ARCH, 'arm64'), eq(parameters.VSCODE_ARCH, 'armhf')) }}:
- script: |
set -e
includes=$(cat << 'EOF'
{
"target_defaults": {
"conditions": [
["OS=='linux'", {
'cflags_cc!': [ '-std=gnu++20' ],
'cflags_cc': [ '-std=gnu++2a' ],
}]
]
}
}
EOF
)
if [ ! -d "$HOME/.gyp" ]; then
mkdir -p "$HOME/.gyp"
fi
echo "$includes" > "$HOME/.gyp/include.gypi"
displayName: Override gnu target for arm64 and arm

- script: |
set -e

Expand All @@ -101,7 +123,7 @@ steps:
set -e

export VSCODE_SYSROOT_PREFIX='-glibc-2.17'
source ./build/azure-pipelines/linux/setup-env.sh --only-remote
source ./build/azure-pipelines/linux/setup-env.sh --skip-sysroot

for i in {1..5}; do # try 5 times
npm ci && break
Expand Down
40 changes: 22 additions & 18 deletions build/azure-pipelines/linux/product-build-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,28 @@ steps:
GITHUB_TOKEN: "$(github-distro-mixin-password)"
displayName: Download vscode sysroots

- ${{ if or(eq(parameters.VSCODE_ARCH, 'arm64'), eq(parameters.VSCODE_ARCH, 'armhf')) }}:
- script: |
set -e
includes=$(cat << 'EOF'
{
"target_defaults": {
"conditions": [
["OS=='linux'", {
'cflags_cc!': [ '-std=gnu++20' ],
'cflags_cc': [ '-std=gnu++2a' ],
}]
]
}
}
EOF
)
if [ ! -d "$HOME/.gyp" ]; then
mkdir -p "$HOME/.gyp"
fi
echo "$includes" > "$HOME/.gyp/include.gypi"
displayName: Override gnu target for arm64 and arm

- script: |
set -e

Expand Down Expand Up @@ -156,20 +178,6 @@ steps:
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'))
displayName: Mixin distro node modules

- ${{ else }}:
# Ref https://github.com/microsoft/vscode/issues/189019
# for the node-gyp rebuild step
- script: |
set -e

cd node_modules/native-keymap && npx [email protected] -y rebuild --debug
cd ../.. && ./.github/workflows/check-clean-git-state.sh
env:
npm_config_arch: $(NPM_ARCH)
GITHUB_TOKEN: "$(github-distro-mixin-password)"
displayName: Rebuild debug version of native modules (OSS)
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'))

- script: |
set -e
node build/azure-pipelines/common/listNodeModules.js .build/node_modules_list.txt
Expand Down Expand Up @@ -248,8 +256,6 @@ steps:
- script: |
set -e

source ./build/azure-pipelines/linux/setup-env.sh

EXPECTED_GLIBC_VERSION="2.28" \
EXPECTED_GLIBCXX_VERSION="3.4.25" \
./build/azure-pipelines/linux/verify-glibc-requirements.sh
Expand All @@ -263,8 +269,6 @@ steps:
- script: |
set -e

source ./build/azure-pipelines/linux/setup-env.sh

EXPECTED_GLIBC_VERSION="2.28" \
EXPECTED_GLIBCXX_VERSION="3.4.26" \
./build/azure-pipelines/linux/verify-glibc-requirements.sh
Expand Down
24 changes: 13 additions & 11 deletions build/azure-pipelines/linux/setup-env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,8 @@ else
fi

if [ "$npm_config_arch" == "x64" ]; then
if [ "$(echo "$@" | grep -c -- "--only-remote")" -eq 0 ]; then
# Download clang based on chromium revision used by vscode
curl -s https://raw.githubusercontent.com/chromium/chromium/124.0.6367.243/tools/clang/scripts/update.py | python - --output-dir=$PWD/.build/CR_Clang --host-os=linux
curl -s https://raw.githubusercontent.com/chromium/chromium/128.0.6613.162/tools/clang/scripts/update.py | python - --output-dir=$PWD/.build/CR_Clang --host-os=linux

# Download libcxx headers and objects from upstream electron releases
DEBUG=libcxx-fetcher \
Expand All @@ -30,22 +29,23 @@ if [ "$npm_config_arch" == "x64" ]; then

# Set compiler toolchain
# Flags for the client build are based on
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/124.0.6367.243:build/config/arm.gni
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/124.0.6367.243:build/config/compiler/BUILD.gn
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/124.0.6367.243:build/config/c++/BUILD.gn
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/128.0.6613.162:build/config/arm.gni
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/128.0.6613.162:build/config/compiler/BUILD.gn
# https://source.chromium.org/chromium/chromium/src/+/refs/tags/128.0.6613.162:build/config/c++/BUILD.gn
export CC="$PWD/.build/CR_Clang/bin/clang --gcc-toolchain=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu"
export CXX="$PWD/.build/CR_Clang/bin/clang++ --gcc-toolchain=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu"
export CXXFLAGS="-nostdinc++ -D__NO_INLINE__ -I$PWD/.build/libcxx_headers -isystem$PWD/.build/libcxx_headers/include -isystem$PWD/.build/libcxxabi_headers/include -fPIC -flto=thin -fsplit-lto-unit -D_LIBCPP_ABI_NAMESPACE=Cr -D_LIBCPP_HARDENING_MODE=_LIBCPP_HARDENING_MODE_EXTENSIVE --sysroot=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot"
export LDFLAGS="-stdlib=libc++ --sysroot=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot -fuse-ld=lld -flto=thin -L$PWD/.build/libcxx-objects -lc++abi -L$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot/usr/lib/x86_64-linux-gnu -L$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot/lib/x86_64-linux-gnu -Wl,--lto-O0"

if [ "$(echo "$@" | grep -c -- "--skip-sysroot")" -eq 0 ]; then
# Set compiler toolchain for remote server
export VSCODE_REMOTE_CC=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/bin/x86_64-linux-gnu-gcc
export VSCODE_REMOTE_CXX=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/bin/x86_64-linux-gnu-g++
export VSCODE_REMOTE_CXXFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot"
export VSCODE_REMOTE_LDFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot -L$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot/usr/lib/x86_64-linux-gnu -L$VSCODE_SYSROOT_DIR/x86_64-linux-gnu/x86_64-linux-gnu/sysroot/lib/x86_64-linux-gnu"
fi
elif [ "$npm_config_arch" == "arm64" ]; then
if [ "$(echo "$@" | grep -c -- "--only-remote")" -eq 0 ]; then
if [ "$(echo "$@" | grep -c -- "--skip-sysroot")" -eq 0 ]; then
# Set compiler toolchain for client native modules
export CC=$VSCODE_SYSROOT_DIR/aarch64-linux-gnu/bin/aarch64-linux-gnu-gcc
export CXX=$VSCODE_SYSROOT_DIR/aarch64-linux-gnu/bin/aarch64-linux-gnu-g++
Expand All @@ -59,11 +59,13 @@ elif [ "$npm_config_arch" == "arm64" ]; then
export VSCODE_REMOTE_LDFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/aarch64-linux-gnu/aarch64-linux-gnu/sysroot -L$VSCODE_SYSROOT_DIR/aarch64-linux-gnu/aarch64-linux-gnu/sysroot/usr/lib/aarch64-linux-gnu -L$VSCODE_SYSROOT_DIR/aarch64-linux-gnu/aarch64-linux-gnu/sysroot/lib/aarch64-linux-gnu"
fi
elif [ "$npm_config_arch" == "arm" ]; then
# Set compiler toolchain for client native modules
export CC=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/bin/arm-rpi-linux-gnueabihf-gcc
export CXX=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/bin/arm-rpi-linux-gnueabihf-g++
export CXXFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot"
export LDFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot -L$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot/usr/lib/arm-linux-gnueabihf -L$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot/lib/arm-linux-gnueabihf"
if [ "$(echo "$@" | grep -c -- "--skip-sysroot")" -eq 0 ]; then
# Set compiler toolchain for client native modules
export CC=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/bin/arm-rpi-linux-gnueabihf-gcc
export CXX=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/bin/arm-rpi-linux-gnueabihf-g++
export CXXFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot"
export LDFLAGS="--sysroot=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot -L$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot/usr/lib/arm-linux-gnueabihf -L$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/arm-rpi-linux-gnueabihf/sysroot/lib/arm-linux-gnueabihf"
fi

# Set compiler toolchain for remote server
export VSCODE_REMOTE_CC=$VSCODE_SYSROOT_DIR/arm-rpi-linux-gnueabihf/bin/arm-rpi-linux-gnueabihf-gcc
Expand Down
8 changes: 4 additions & 4 deletions build/azure-pipelines/oss/product-build-pr-cache-win32.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,15 +49,15 @@ steps:

- powershell: |
. build/azure-pipelines/win32/exec.ps1
. build/azure-pipelines/win32/retry.ps1
$ErrorActionPreference = "Stop"
$env:npm_config_arch="$(VSCODE_ARCH)"
$env:CHILD_CONCURRENCY="1"
retry { exec { npm ci } }
exec { npm ci }
env:
npm_config_arch: $(VSCODE_ARCH)
npm_config_foreground_scripts: "true"
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
GITHUB_TOKEN: "$(github-distro-mixin-password)"
retryCountOnTaskFailure: 5
displayName: Install dependencies
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'))

Expand Down
1 change: 1 addition & 0 deletions build/azure-pipelines/web/product-build-web.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ steps:

- script: |
set -e

for i in {1..5}; do # try 5 times
npm ci && break
if [ $i -eq 5 ]; then
Expand Down
5 changes: 2 additions & 3 deletions build/azure-pipelines/win32/product-build-win32-cli-sign.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,11 +36,10 @@ steps:

- powershell: |
. azure-pipelines/win32/exec.ps1
. azure-pipelines/win32/retry.ps1
$ErrorActionPreference = "Stop"
$env:CHILD_CONCURRENCY="1"
retry { exec { npm ci } }
exec { npm ci }
workingDirectory: build
retryCountOnTaskFailure: 5
displayName: Install build dependencies

- template: ../cli/cli-win32-sign.yml@self
Expand Down
6 changes: 3 additions & 3 deletions build/azure-pipelines/win32/product-build-win32.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,15 +91,15 @@ steps:

- powershell: |
. build/azure-pipelines/win32/exec.ps1
. build/azure-pipelines/win32/retry.ps1
$ErrorActionPreference = "Stop"
retry { exec { npm ci } }
exec { npm ci }
env:
npm_config_arch: $(VSCODE_ARCH)
CHILD_CONCURRENCY: 1
npm_config_foreground_scripts: "true"
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
GITHUB_TOKEN: "$(github-distro-mixin-password)"
retryCountOnTaskFailure: 5
displayName: Install dependencies
condition: and(succeeded(), ne(variables.NODE_MODULES_RESTORED, 'true'))

Expand Down
6 changes: 3 additions & 3 deletions build/azure-pipelines/win32/sdl-scan-win32.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,15 @@ steps:

- powershell: |
. build/azure-pipelines/win32/exec.ps1
. build/azure-pipelines/win32/retry.ps1
$ErrorActionPreference = "Stop"
retry { exec { npm ci } }
exec { npm ci }
env:
npm_config_arch: ${{ parameters.VSCODE_ARCH }}
CHILD_CONCURRENCY: 1
npm_config_foreground_scripts: "true"
ELECTRON_SKIP_BINARY_DOWNLOAD: 1
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 1
GITHUB_TOKEN: "$(github-distro-mixin-password)"
retryCountOnTaskFailure: 5
displayName: Install dependencies

- script: node build/azure-pipelines/distro/mixin-npm
Expand Down
Loading
Loading