helloadhavan/github_issues huggingface dataset:
Repository: ollama/ollama
Pull request: models: llama4 multimodal

This change adds Meta's [Llama 4](https://huggingface.co/collections/meta-llama/llama-4-67f0c30d9fe03840bc9d0164) model to Ollama. This has mainly been tested with the 16E Scout model.

Key model features:

* Mixture of expert
* Multimodal (vision, text)

Of note, there are a couple things of note:

* The vision model is trained on tiled images which requires special handling for both the pixel values and rotary embeddings. Both require 5D tensors which is not supported by ggml so the operation has been split into multiple operations
* The text model uses chunked attention mask for select layers and causal attention for other layers. This requires a minor change to `kvcache.Causal` similar to sliding window attention

Changed files:
filename: fs/ggml/ggml.go
patch: @@ -430,7 +430,7 @@ func (f GGML) GraphSize(context, batch uint64, numParallel int, kvCacheType stri
 	}
 
 	switch f.KV().Architecture() {
-	case "llama":
+	case "llama", "llama4":
 		fullOffload = max(
 			4*batch*(1+4*embedding+context*(1+heads)),
 			4*batch*(embedding+vocab),
@@ -444,7 +444,7 @@ func (f GGML) GraphSize(context, batch uint64, numParallel int, kvCacheType stri
 
 		if ffnGateExpsWeight, ok := layers["blk.0"]["ffn_gate_exps.weight"]; ok {
 			// mixtral 8x22b
-			ff := uint64(f.KV()["llama.feed_forward_length"].(uint32))
+			ff := uint64(f.KV().Uint("feed_forward_length"))
 			partialOffload = max(
 				3*ffnGateExpsWeight.Size()+4*batch*(2*ff+headsKV+embedding+context+embeddingHeads*headsKV),
 				4*(context*batch*heads+context*embeddingHeads*headsKV+batch*1024+embeddingHeads*headsKV*batch),
@@ -640,6 +640,9 @@ func (llm GGML) VisionGraphSize() (weights, graphSize uint64) {
 		graphSize = 4 * (imageSize*imageSize*numChannels +
 			embeddingLength*patchSize +
 			numPatches*numPatches*headCount)
+	case "llama4":
+		// vision graph is computed independently in the same schedule
+		// and is negligible compared to the worst case text graph
 	}
 
 	return weights, graphSize
Repository: vuejs/vue
Pull request: [automated] Patreon sponsors update

This is an automated pull request.

Changed files:
filename: BACKERS.md
patch: @@ -209,6 +209,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/yakaz.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://www.bacancytechnology.com" target="_blank">
+          <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/bacancy_technology.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>

filename: README.md
patch: @@ -214,6 +214,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/yakaz.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://www.bacancytechnology.com" target="_blank">
+          <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/bacancy_technology.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>
Repository: ggml-org/llama.cpp
Pull request: Reset graph on control vector change

This PR makes an existing context pick up a change to its control vector configuration via  `llama_context::set_adapter_cvec`.

The issue in short:
- Initial call to `set_adapter_cvec` works, steering vector applies to generation.
- Any further calls to it after the initial generation, including attempting to unset it via nullptr vector or chaging layers/strength, do not apply

Background: When I ran a [script](https://github.com/withcatai/node-llama-cpp/compare/master...iimez:node-llama-cpp:controlVectors#diff-2f60cdc91bc3c40f0e81aee1bdcf7e0606296a04ac6b2b8a63b7de38e773e693R978-R997) to sweep control vector configurations I noticed that *only the initial* configuration would be picked up, with repeated calls to `llama_context::set_adapter_cvec` (in between generations) being ignored and generation continuing with the initial config.

Recreating the context for every iteration in a sweep would have been an alternative, but much slower than necessary.

While looking to speed it up I saw the TODO in `set_adapter_cvec`, compared with the LoRA logic and tried around to find the minimal invalidation required to make generation pick up the changes. Resetting the graph appears to be enough for cvec and I'm suspecting the TODO could be removed.

I tested this change (extensively while sweeping steering vector configurations) on various model architectures and cpu/vulkan/cuda backends.

AI Disclaimer: Opus suggested this.

Changed files:
filename: src/llama-context.cpp
patch: @@ -1165,9 +1165,11 @@ bool llama_context::set_adapter_cvec(
                 int32_t   il_end) {
     LLAMA_LOG_DEBUG("%s: il_start = %d, il_end = %d\n", __func__, il_start, il_end);
 
-    // TODO: should we reserve?
+    bool res = cvec->apply(model, data, len, n_embd, il_start, il_end);
 
-    return cvec->apply(model, data, len, n_embd, il_start, il_end);
+    sched_need_reserve = true;
+
+    return res;
 }
 
 llm_graph_result * llama_context::process_ubatch(const llama_ubatch & ubatch, llm_graph_type gtype, llama_memory_context_i * mctx, ggml_status & ret) {
Repository: ollama/ollama
Pull request: fs: generic ggml.array

this updates ggml.array to use generics which makes it easier to use and avoids type asserting the array values

Changed files:
filename: sample/samplers_test.go
patch: @@ -74,7 +74,6 @@ func modelHelper(t testing.TB) model.BytePairEncoding {
 		t.Fatal(err)
 	}
 
-	types := make([]uint32, len(vocab))
 	tokens := make([]string, len(vocab))
 	for token, id := range vocab {
 		tokens[id] = token
@@ -86,7 +85,7 @@ func modelHelper(t testing.TB) model.BytePairEncoding {
 		``,
 		&model.Vocabulary{
 			Values: tokens,
-			Types:  types,
+			Types:  make([]int32, len(vocab)),
 			Merges: merges,
 		},
 	)
Repository: nodejs/node
Pull request: doc: add CVE delay mention


Changed files:
filename: SECURITY.md
patch: @@ -348,6 +348,21 @@ Security notifications will be distributed via the following methods.
 * <https://groups.google.com/group/nodejs-sec>
 * <https://nodejs.org/en/blog/vulnerability>
 
+### CVE publication timeline
+
+When security releases are published, there is a built-in delay before the
+corresponding CVEs are publicly disclosed. This delay occurs because:
+
+1. After the security release, we request the vulnerability reporter to disclose
+   the details on HackerOne.
+2. If the reporter does not disclose within one day, we proceed with forced
+   disclosure to publish the CVEs.
+3. The disclosure then goes through HackerOne's approval process before the CVEs
+   become publicly available.
+
+As a result, CVEs may not be immediately available when security releases are
+published, but will typically be disclosed within a few days of the release.
+
 ## Comments on this policy
 
 If you have suggestions on how this process could be improved, please visit
Repository: vuejs/vue
Pull request: [automated] Patreon backers update (May 2019

This is an automated pull request.

Changed files:
filename: BACKERS.md
patch: @@ -374,113 +374,106 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
 - Shawn Wildermuth
 - Samuel Smith
 - donny
-- Simon East
+- Sean Washington
 - Keisuke Kita
 - kazupon
-- Sean Washington
 - Lars Andreas Ness
+- Simon East
 - Benjamin Listwon
 - Kirk Lewis
-- Stephen Michael Hartley
 - Victor Tolbert
+- Stephen Michael Hartley
 - Wen-Tien Chang
-- Karol
 - Phan An
-- James J. Ye
 - Barbara Liau
+- James J. Ye
 - Luiz
 - Matsumoto Takamasa
-- Jarek Tkaczyk
+- Shinya Katayama
 - Niannian Modisette
+- Guy Gavergun
 - Matt Jones
 - Duncan Kenzie
-- Guy Gavergun
-- Shinya Katayama
-- Mike Margerum
 - Bernhard E. Reiter
-- Chengzhi Yin
 - Zoran Knezevic
-- Jon Hobbs-Smith
+- Chengzhi Yin
 - Pierre Vanhulst
+- Jon Hobbs-Smith
 - Akiho Nagao
+- Anthony Estebe
 - Haim Yulzari
 - Asaf Yishai
-- Anthony Estebe
-- Jeremy Tan
 - Jim Raden
+- Jeremy Tan
 - IMGNRY
 - Tai Shi Ling
 - Yong Jun Thong
 - Tyler
-- Vivekanandhan Natarajan
-- Matias Verdier
 - Rafael Belvederese
 - Chih-Hsuan Fan
-- Guilherme S L de Souza
+- Jamie McElwain
+- Vivekanandhan Natarajan
 - Daniel Waghorn
+- Guilherme S L de Souza
+- Matias Verdier
 - Rob Yedlin
 - Mickaël Andrieu
-- Jamie McElwain
-- Jordan Oroshiba
 - Cliff Hess
 - Joe Gregory
-- Richard Simpson
+- Jordan Oroshiba
 - Marcos Moura
+- Richard Simpson
 - Jessie Hernandez
 - Ivan Sieder
-- Eric
 - Aparajita Fishman
+- Eric
+- Christo Crampton
 - Romain Lienard
 - Bohdan Kokotko
-- Christo Crampton
-- David Ang
-- Donald Fischer
-- Alexander Weiher
 - Oskar Lindgren
 - Jere Sjöroos
+- David Ang
+- Alexander Weiher
+- Donald Fischer
 - Wakana Seki
-- David Kaplan
-- Ben Hong
 - Domenico Gaudioso
+- David Kaplan
 - Jaeyoung Lee
+- Ben Hong
 - Andy
 - Joe Cochran
 - Matt Sencenbaugh
+- Colt Borg
 - Juan Bermudez
 - Hannes Kochniß
-- Colt Borg
 - Alberto T. Payero Mota
+- Elon Hung
 - Daniel Mattingley
 - Teon Ooi
 - Chris Calo
-- Elon Hung
 - Soichiro Isshiki
-- Milan Zivkovic
-- Christopher Bemister
-- Ed Linklater
-- Garion Herman
 - Marko Boskovic
+- Garion Herman
+- Ed Linklater
 - Rob
 - Andrew Willis
 - Yusuke Kawabata
 - Princeyesuraj Edward
-- shimbaco
 - 龙腾道
-- 极 蔡
+- Shapovalov Pavel
 - Peter Matkovsky
+- 极 蔡
 - Nick Dandakis
 - Fabien GuySake Ungerer
 - Fedor Indutny
 - Masato Miura
-- Nathan Mallison
-- Shapovalov Pavel
-- David McGuigan
-- Max Fletcher
 - Bill Condo
+- David McGuigan
+- Nathan Mallison
 - Kevin Batdorf
-- Ivan Buncic
-- Pierre Lebrun
 - Sheree Peña
+- Pierre Lebrun
+- John Thompson
 <!--10 end-->
 
 <h2 align="center">Backers via OpenCollective</h2>
Repository: ollama/ollama
Pull request: fix superfluous call to WriteHeader

the first call to `http.ResponseWriter.Write` implicitly calls `WriteHeader` with `http.StatusOK` if it hasn't already been called. once `WriteHeader` has been called, subsequent calls has no effect. Write is called when JSON encoding `progressUpdateJSON{}`. calls to `http.ResponseWriter.WriteHeader` after the first encode is useless and produces a warning:

```
http: superfluous response.WriteHeader call from github.com/ollama/ollama/server/internal/registry.(*statusCodeRecorder).WriteHeader (server.go:77)
```

Changed files:
filename: server/internal/registry/server.go
patch: @@ -73,8 +73,13 @@ type statusCodeRecorder struct {
 func (r *statusCodeRecorder) WriteHeader(status int) {
 	if r._status == 0 {
 		r._status = status
+		r.ResponseWriter.WriteHeader(status)
 	}
-	r.ResponseWriter.WriteHeader(status)
+}
+
+func (r *statusCodeRecorder) Write(b []byte) (int, error) {
+	r._status = r.status()
+	return r.ResponseWriter.Write(b)
 }
 
 var (
Repository: facebook/react
Pull request: [compiler] Enable additional lints by default


Enable more validations to help catch bad patterns, but only in the linter. These rules are already enabled by default in the compiler _if_ violations could produce unsafe output.

---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/react/pull/33752).
* #33981
* #33777
* #33767
* #33765
* #33760
* #33759
* #33758
* #33751
* __->__ #33752
* #33753

Changed files:
filename: compiler/packages/eslint-plugin-react-compiler/src/rules/ReactCompilerRule.ts
patch: @@ -107,6 +107,12 @@ const COMPILER_OPTIONS: Partial<PluginOptions> = {
   flowSuppressions: false,
   environment: validateEnvironmentConfig({
     validateRefAccessDuringRender: false,
+    validateNoSetStateInRender: true,
+    validateNoSetStateInEffects: true,
+    validateNoJSXInTryStatements: true,
+    validateNoImpureFunctionsInRender: true,
+    validateStaticComponents: true,
+    validateNoFreezingKnownMutableFunctions: true,
   }),
 };
 

filename: packages/eslint-plugin-react-hooks/src/rules/ReactCompiler.ts
patch: @@ -109,6 +109,12 @@ const COMPILER_OPTIONS: Partial<PluginOptions> = {
   flowSuppressions: false,
   environment: validateEnvironmentConfig({
     validateRefAccessDuringRender: false,
+    validateNoSetStateInRender: true,
+    validateNoSetStateInEffects: true,
+    validateNoJSXInTryStatements: true,
+    validateNoImpureFunctionsInRender: true,
+    validateStaticComponents: true,
+    validateNoFreezingKnownMutableFunctions: true,
   }),
 };
Repository: vercel/next.js
Pull request: Prettier-ignore changes in `next-env.d.ts` files in all top-level apps

Specifically, the `lint` job is currently failing in CI for `apps/bundle-analyzer/next-env.d.ts`, e.g.:
https://github.com/vercel/next.js/actions/runs/21444929031/job/61758061793?pr=89175

Changed files:
filename: .prettierignore
patch: @@ -78,6 +78,7 @@ test/integration/typescript-app-type-declarations/next-env.strictRouteTypes.d.ts
 /turbopack/crates/turbopack-tracing/tests/node-file-trace/test/unit/tsx/lib.tsx
 
 /apps/docs/.source/*
+/apps/*/next-env.d.ts
 
 # Symlink files
 readme.md
Repository: facebook/react
Pull request: [DevTools] Feature detect createSidebarPane

Same as #33987 but for the sidebar pane creation.

Changed files:
filename: packages/react-devtools-extensions/src/main/index.js
patch: @@ -317,7 +317,7 @@ function createSourcesEditorPanel() {
   }
 
   const sourcesPanel = chrome.devtools.panels.sources;
-  if (!sourcesPanel) {
+  if (!sourcesPanel || !sourcesPanel.createSidebarPane) {
     // Firefox doesn't currently support extending the source panel.
     return;
   }
Repository: vuejs/vue
Pull request: [automated] Patreon sponsors update

This is an automated pull request.

Changed files:
filename: BACKERS.md
patch: @@ -57,6 +57,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="222px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/nativescript.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://teeplusplusclth.com/" target="_blank">
+          <img width="222px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/tee__.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>

filename: README.md
patch: @@ -67,6 +67,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="222px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/nativescript.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://teeplusplusclth.com/" target="_blank">
+          <img width="222px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/tee__.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>
Repository: vuejs/vue
Pull request: [automated] Patreon sponsors update

This is an automated pull request.

Changed files:
filename: BACKERS.md
patch: @@ -300,6 +300,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/earthlink.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://twitter.com/philipjbasile" target="_blank">
+          <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/philip_john_basile.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>
Repository: vercel/next.js
Pull request: [ci] Silence `baseline-browser-mapping` warnings

The warnings from `baseline-browser-mapping` about outdated data fail some tests that don't expect the CLI output. By updating the dependency we avoid showing the warning for now, and by setting the environment variable `BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA` we should silence any future warnings as well.

related:
- #86625
- #86653

Changed files:
filename: .github/workflows/build_reusable.yml
patch: @@ -118,6 +118,10 @@ env:
   # defaults to 256, but we run a lot of tests in parallel, so the limit should be lower
   NEXT_TURBOPACK_IO_CONCURRENCY: 64
 
+  # Disable warnings from baseline-browser-mapping
+  # https://github.com/web-platform-dx/baseline-browser-mapping/blob/ec8136ae9e034b332fab991d63a340d2e13b8afc/README.md?plain=1#L34
+  BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA: 1
+
 jobs:
   build:
     timeout-minutes: ${{ inputs.timeout_minutes }}

filename: packages/next/package.json
patch: @@ -99,7 +99,7 @@
   "dependencies": {
     "@next/env": "16.2.0-canary.14",
     "@swc/helpers": "0.5.15",
-    "baseline-browser-mapping": "^2.8.3",
+    "baseline-browser-mapping": "^2.9.19",
     "caniuse-lite": "^1.0.30001579",
     "postcss": "8.4.31",
     "styled-jsx": "5.1.6"

filename: pnpm-lock.yaml
patch: @@ -1091,8 +1091,8 @@ importers:
         specifier: 0.5.15
         version: 0.5.15
       baseline-browser-mapping:
-        specifier: ^2.8.3
-        version: 2.8.32
+        specifier: ^2.9.19
+        version: 2.9.19
       caniuse-lite:
         specifier: 1.0.30001746
         version: 1.0.30001746
@@ -7811,8 +7811,8 @@ packages:
   base64-js@1.5.1:
     resolution: {integrity: sha512-AKpaYlHn8t4SVbOHCy+b5+KKgvR4vrsD8vbvrbiQJps7fKDTkjkDry6ji0rUJjC0kzbNePLwzxq8iypo41qeWA==}
 
-  baseline-browser-mapping@2.8.32:
-    resolution: {integrity: sha512-OPz5aBThlyLFgxyhdwf/s2+8ab3OvT7AdTNvKHBwpXomIYeXqpUUuT8LrdtxZSsWJ4R4CU1un4XGh5Ez3nlTpw==}
+  baseline-browser-mapping@2.9.19:
+    resolution: {integrity: sha512-ipDqC8FrAl/76p2SSWKSI+H9tFwm7vYqXQrItCuiVPt26Km0jS+NzSsBWAaBusvSbQcfJG+JitdMm+wZAgTYqg==}
     hasBin: true
 
   batch@0.6.1:
@@ -25413,7 +25413,7 @@ snapshots:
 
   base64-js@1.5.1: {}
 
-  baseline-browser-mapping@2.8.32: {}
+  baseline-browser-mapping@2.9.19: {}
 
   batch@0.6.1: {}
 
@@ -25607,7 +25607,7 @@ snapshots:
 
   browserslist@4.28.0:
     dependencies:
-      baseline-browser-mapping: 2.8.32
+      baseline-browser-mapping: 2.9.19
       caniuse-lite: 1.0.30001746
       electron-to-chromium: 1.5.262
       node-releases: 2.0.27
Repository: ggml-org/llama.cpp
Pull request: ggml blas: set mkl threads from thread context

Commit 1: Set number of threads for MKL
Commit 2: Add way to run blas builds through local CI.


Changed files:
filename: ci/run.sh
patch: @@ -25,7 +25,13 @@
 # # with KLEIDIAI support
 # GG_BUILD_KLEIDIAI=1 bash ./ci/run.sh ./tmp/results ./tmp/mnt
 #
-# # with OPENVINO support
+# # with BLAS support
+# GG_BUILD_BLAS=1 bash ./ci/run.sh ./tmp/results ./tmp/mnt
+#
+# with BLAS support (custom vendor)
+# GG_BUILD_BLAS=1 GG_BUILD_BLAS_VENDOR=Intel10_64lp bash ./ci/run.sh ./tmp/results ./tmp/mnt
+#
+# with OPENVINO support
 # GG_BUILD_OPENVINO=1 GG_BUILD_LOW_PERF=1 GGML_OPENVINO_DEVICE=CPU bash ./ci/run.sh ./tmp/results ./tmp/mnt
 #
 
@@ -169,6 +175,10 @@ if [ -n "${GG_BUILD_KLEIDIAI}" ]; then
         -DBUILD_SHARED_LIBS=OFF"
 fi
 
+if [ ! -z ${GG_BUILD_BLAS} ]; then
+    CMAKE_EXTRA="${CMAKE_EXTRA} -DGGML_BLAS=ON -DGGML_BLAS_VENDOR=${GG_BUILD_BLAS_VENDOR:-OpenBLAS}"
+fi
+
 if [ ! -z ${GG_BUILD_OPENVINO} ]; then
     if [ -z ${OpenVINO_DIR} ]; then
         echo "OpenVINO_DIR not found, please install OpenVINO via archives and enable it by:"

filename: ggml/src/ggml-blas/ggml-blas.cpp
patch: @@ -121,6 +121,8 @@ static void ggml_backend_blas_mul_mat(ggml_backend_blas_context * ctx, struct gg
     bli_thread_set_num_threads(ctx->n_threads);
 #elif defined(GGML_BLAS_USE_NVPL)
     nvpl_blas_set_num_threads(ctx->n_threads);
+#elif defined(GGML_BLAS_USE_MKL)
+    mkl_set_num_threads(ctx->n_threads);
 #endif
 
     for (int64_t i13 = 0; i13 < ne13; i13++) {
Repository: vuejs/vue
Pull request: [automated] Patreon sponsors update

This is an automated pull request.

Changed files:
filename: BACKERS.md
patch: @@ -231,6 +231,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/storekit.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://staffaugmentation.ro/en" target="_blank">
+          <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/staff_augmentation.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>

filename: README.md
patch: @@ -236,6 +236,11 @@ Funds donated via Patreon go directly to support Evan You's full-time work on Vu
           <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/storekit.png">
         </a>
       </td>
+      <td align="center" valign="middle">
+        <a href="https://staffaugmentation.ro/en" target="_blank">
+          <img width="148px" src="https://raw.githubusercontent.com/vuejs/vuejs.org/master/themes/vue/source/images/staff_augmentation.png">
+        </a>
+      </td>
     </tr><tr></tr>
   </tbody>
 </table>
Repository: nodejs/node
Pull request: gyp: aix: change gcc version so CXX="ccache g++" works

Closes https://github.com/nodejs/node/issues/61443

<!--
Before submitting a pull request, please read:

- the CONTRIBUTING guide at https://github.com/nodejs/node/blob/HEAD/CONTRIBUTING.md
- the commit message formatting guidelines at
  https://github.com/nodejs/node/blob/HEAD/doc/contributing/pull-requests.md#commit-message-guidelines

For code changes:
1. Include tests for any bug fixes or new features.
2. Update documentation if relevant.
3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes.

If you believe this PR should be highlighted in the Node.js CHANGELOG
please add the `notable-change` label.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.
-->


Changed files:
filename: common.gypi
patch: @@ -598,7 +598,7 @@
             ],
           }, {                                             # else it's `AIX`
             'variables': {
-              'gcc_major': '<!(<(python) -c "import os; import subprocess; CXX=os.environ.get(\'CXX\', \'g++\'); subprocess.run([CXX, \'-dumpversion\'])")'
+              'gcc_major': '<!(sh -c "${CXX:-g++} -dumpversion")'
             },
             # Disable the following compiler warning:
             #