From bab5af61e45260e9509e1ff9e93cf1d674d2d087 Mon Sep 17 00:00:00 2001
From: Boris Verkhovskiy <boris.verk@gmail.com>
Date: Thu, 22 Jun 2023 13:22:01 +0100
Subject: [PATCH] Don't prebuild on Node 10-14 in CI

---
 .github/workflows/build.yml | 33 +++++++++++++--------------------
 binding.gyp                 |  7 -------
 2 files changed, 13 insertions(+), 27 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index d4468557..7be752ab 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -51,34 +51,27 @@ jobs:
           - macos-latest
           - ubuntu-latest
         node:
-          - 10
-          - 12
-          - 14
-          - 16
-          - 18
-          - 20
-        exclude:
+          # Node 10-14 fails to prebuild for Node 18+
+          #
           # macos-latest's Python version is too new to work with the node-gyp
           # bundled by prebuild and the "overrides" in package.json for node-gyp
           # doesn't work before npm 8.3 (Node 16)
-          - os: macos-latest
-            node: 10
-          - os: macos-latest
-            node: 12
-          - os: macos-latest
-            node: 14
-          # Similar issue but in this case the error is more direct:
           #
+          # ubuntu-latest has a similar issue but in this case the error is more direct:
           # #error "It looks like you are building this native module without
           # using the right config.gypi.  This normally means that you need to
           # update electron-rebuild (>=3.2.8) or node-gyp (>=8.4.0) if you're
           # building modules directly."
-          - os: ubuntu-latest
-            node: 10
-          - os: ubuntu-latest
-            node: 12
-          - os: ubuntu-latest
-            node: 14
+          #
+          # windows-2019 fails with
+          # gyp: name 'llvm_version' is not defined while evaluating condition
+          # 'llvm_version=="0.0"' in binding.gyp while trying to load binding.gyp
+          # - 10
+          # - 12
+          # - 14
+          - 16
+          - 18
+          - 20
       fail-fast: false
     name: Prebuild with Node ${{ matrix.node }} on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
diff --git a/binding.gyp b/binding.gyp
index 17808cb5..8b1379de 100644
--- a/binding.gyp
+++ b/binding.gyp
@@ -42,13 +42,6 @@
               'RuntimeLibrary': 0,
             },
           },
-          'variables': {
-            # fix this error when prebuilding for Node 18 on Node 14 or older
-            #
-            # gyp: name 'llvm_version' is not defined while evaluating condition
-            # 'llvm_version=="0.0"' in binding.gyp while trying to load binding.gyp
-            'llvm_version': 0,
-          }
         }]
       ],
     },