Skip to content

Commit

Permalink
Rename --mac-legacy to --mac-legacy-10.13-10.14 to prepare for future
Browse files Browse the repository at this point in the history
Electron 33 drops support for macOS 10.15 so we may
end up having two different macOS legacy versions.

While the macOS 10.13/10.14 version would run fine on
macOS 10.15 as well, that would be a significant downgrade
of Chromium which tends to result in data loss.

This is starting to get a bit out of hand.
  • Loading branch information
GarboMuffin committed Jan 29, 2025
1 parent 2ae7f66 commit b8ec3f8
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
if: runner.os == 'macOS'
run: |
node release-automation/build.js --mac --universal
node release-automation/build.js --mac-legacy --x64
node release-automation/build.js --mac-legacy-10.13-10.14 --x64
- name: Package Linux
if: runner.os == 'Linux'
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ jobs:
# @electron/notaraize documentation says key should be an absolute path
export APPLE_API_KEY="$(pwd)/$APPLE_API_KEY_NAME"
node release-automation/build.js --mac --universal --production
node release-automation/build.js --mac-legacy --x64 --production
node release-automation/build.js --mac-legacy-10.13-10.14 --x64 --production
# for safety
rm "$APPLE_API_KEY_NAME"
env:
Expand Down
1 change: 1 addition & 0 deletions docs/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -546,6 +546,7 @@ <h1 class="header">

<script>
// FIXME: when >= 1.13.2 is released, update the .deb packages to not hardcode 1.13.1
// FIXME: when >= 1.13.2 is released, macOS legacy download name may need to be updated
const VERSION = '1.13.0';

// OS-specific styles
Expand Down
4 changes: 2 additions & 2 deletions release-automation/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ You must specify which platform to build for:
--mac
Create app for macOS 10.15 and later.
--mac-legacy
Create app for macOS 10.13 and later (less secure than --mac).
--mac-legacy-10.13-10.14
Create app for macOS 10.13 and 10.14 (less secure than --mac).
--mac-dir
Generate executables for macOS 10.15 and later but don't package into a complete DMG installer.
Expand Down
6 changes: 3 additions & 3 deletions release-automation/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ const buildMac = () => build({
}
});

const buildMacLegacy = () => {
const buildMacLegacy10131014 = () => {
// This is the last release of Electron 26
// Electron 27 dropped support for macOS 10.13 and 10.14
const LEGACY_ELECTRON_VERSION = '26.6.10';
Expand All @@ -263,7 +263,7 @@ const buildMacLegacy = () => {
legacy: true,
extraConfig: {
mac: {
artifactName: '${productName} Legacy Setup ${version}.${ext}'
artifactName: '${productName} Legacy 10.13 10.14 Setup ${version}.${ext}'
}
},
prepare: async (archName) => {
Expand Down Expand Up @@ -318,7 +318,7 @@ const run = async () => {
'--windows-dir': buildWindowsDir,
'--microsoft-store': buildMicrosoftStore,
'--mac': buildMac,
'--mac-legacy': buildMacLegacy,
'--mac-legacy-10.13-10.14': buildMacLegacy10131014,
'--mac-dir': buildMacDir,
'--debian': buildDebian,
'--tarball': buildTarball,
Expand Down

0 comments on commit b8ec3f8

Please sign in to comment.