helloadhavan/github_issues huggingface dataset:
Repository: electron/electron
Pull request: build: fix clang format location helper (again)

#### Description of Change

Similar to #42527, Chromium recently:
* [Moved `clang-format` to `buildtools`](https://chromium-review.googlesource.com/c/chromium/src/+/5506857)
  * This seems to be on the most recent M126
* [Then, later changed the path of `clang-format`](https://chromium-review.googlesource.com/c/chromium/src/+/5581721), moving from `{platform}/format` to `{platform}-format` (`/` to `-`).
  * This change has not landed on M126

#### Backport strategy

I'm going to backport this change to `32-x-y` and `31-x-y`. This should cleanly backport to `32-x-y`, but since `31-x-y` only got the first change we will have to subtly adjust the `util.py` change to join `format` as a path segment instead of as part of the platform.

#### Checklist
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->

- [x] PR description included and stakeholders cc'd
- [x] `npm test` passes
- [x] [PR release notes](https://github.com/electron/clerk/blob/main/README.md) describe the change in a way relevant to app developers, and are [capitalized, punctuated, and past tense](https://github.com/electron/clerk/blob/main/README.md#examples).

#### Release Notes

Notes: none


Changed files:
filename: script/lib/util.py
patch: @@ -194,6 +194,9 @@ def get_buildtools_executable(name):
     chromium_platform = 'linux64'
   else:
     raise Exception(f"Unsupported platform: {sys.platform}")
+  
+  if name == 'clang-format':
+    chromium_platform += '-format'
 
   path = os.path.join(buildtools, chromium_platform, name)
   if sys.platform == 'win32':

filename: script/run-clang-format.py
patch: @@ -23,7 +23,7 @@
 import tempfile
 
 from functools import partial
-from lib.util import get_depot_tools_executable
+from lib.util import get_buildtools_executable
 
 DEFAULT_EXTENSIONS = 'c,h,C,H,cpp,hpp,cc,hh,c++,h++,cxx,hxx,mm'
 DEFAULT_CLANG_FORMAT_IGNORE = '.clang-format-ignore'
@@ -203,7 +203,7 @@ def main():
         '--clang-format-executable',
         metavar='EXECUTABLE',
         help='path to the clang-format executable',
-        default=get_depot_tools_executable('clang-format'))
+        default=get_buildtools_executable('clang-format'))
     parser.add_argument(
         '--extensions',
         help='comma-separated list of file extensions'
Repository: denoland/deno
Pull request: fix(fmt/js): `else` was moved to wrong `if` sometimes when formatting minified code

* https://github.com/dprint/dprint-plugin-typescript/pull/633


Changed files:
filename: .dprint.json
patch: @@ -57,7 +57,7 @@
     "ext/websocket/autobahn/reports"
   ],
   "plugins": [
-    "https://plugins.dprint.dev/typescript-0.90.4.wasm",
+    "https://plugins.dprint.dev/typescript-0.90.5.wasm",
     "https://plugins.dprint.dev/json-0.19.2.wasm",
     "https://plugins.dprint.dev/markdown-0.17.0.wasm",
     "https://plugins.dprint.dev/toml-0.6.1.wasm",

filename: Cargo.lock
patch: @@ -2241,9 +2241,9 @@ checksum = "1aaf95b3e5c8f23aa320147307562d361db0ae0d51242340f558153b4eb2439b"
 
 [[package]]
 name = "dprint-core"
-version = "0.66.1"
+version = "0.66.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "317f9cd1da390ab12e1858b3a71c7d6c8e8ad633454cb0b73ce5589964d9c177"
+checksum = "f3ab0dd2bedc109d25f0d21afb09b7d329f6c6fa83b095daf31d2d967e091548"
 dependencies = [
  "anyhow",
  "bumpalo",
@@ -2308,9 +2308,9 @@ dependencies = [
 
 [[package]]
 name = "dprint-plugin-typescript"
-version = "0.90.4"
+version = "0.90.5"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9ee5e8dfbcb0bc8f360b7287380a57b3600ce8af791d09a0085953bcaa9bdf97"
+checksum = "d7c3c339020ebbbbbe5fc049350935ee2ea2ba5a3fc01f753588639a30404cda"
 dependencies = [
  "anyhow",
  "deno_ast",

filename: cli/Cargo.toml
patch: @@ -99,7 +99,7 @@ dotenvy = "0.15.7"
 dprint-plugin-json = "=0.19.2"
 dprint-plugin-jupyter = "=0.1.3"
 dprint-plugin-markdown = "=0.17.0"
-dprint-plugin-typescript = "=0.90.4"
+dprint-plugin-typescript = "=0.90.5"
 env_logger = "=0.10.0"
 fancy-regex = "=0.10.0"
 faster-hex.workspace = true
Repository: vercel/next.js
Pull request: Use `NEXT_PRIVATE_DEBUG_CACHE` env variable for cache handler debug logs

Instead of `NEXT_PRIVATE_DEBUG_USE_CACHE`, we're now using the env variable `NEXT_PRIVATE_DEBUG_CACHE`, which is already used in other places for cache debugging.

In addition, `process.pid` is not logged anymore. This prevents Node.js API usage errors in #77577 when bundling with Turbopack.

Changed files:
filename: packages/next/src/server/use-cache/handlers.ts
patch: @@ -1,9 +1,9 @@
 import DefaultCacheHandler from '../lib/cache-handlers/default'
 import type { CacheHandlerCompat } from '../lib/cache-handlers/types'
 
-const debug = process.env.NEXT_PRIVATE_DEBUG_USE_CACHE
+const debug = process.env.NEXT_PRIVATE_DEBUG_CACHE
   ? (message: string, ...args: any[]) => {
-      console.log(`use-cache[${process.pid}]: ${message}`, ...args)
+      console.log(`use-cache: ${message}`, ...args)
     }
   : () => {}
Repository: electron/electron
Pull request: build: fix clang format location helper

Manual Backport of https://github.com/electron/electron/pull/42911

This incorporates the changes from https://github.com/electron/electron/pull/42527 and the change to `script/lib/util.py` from https://github.com/electron/electron/pull/42118.

Notes: none

Changed files:
filename: .github/workflows/pipeline-electron-lint.yml
patch: @@ -53,10 +53,10 @@ jobs:
       run: |
         chromium_revision="$(grep -A1 chromium_version src/electron/DEPS | tr -d '\n' | cut -d\' -f4)"
 
-        sha1_path='buildtools/linux64/clang-format.sha1'
-        curl -sL "https://chromium.googlesource.com/chromium/src/+/${chromium_revision}/${sha1_path}?format=TEXT" | base64 -d > "src/${sha1_path}"
+        mkdir -p src/buildtools
+        curl -sL "https://chromium.googlesource.com/chromium/src/+/${chromium_revision}/buildtools/DEPS?format=TEXT" | base64 -d > src/buildtools/DEPS
 
-        download_from_google_storage.py --no_resume --no_auth --bucket chromium-clang-format -s "src/${sha1_path}"
+        gclient sync --spec="solutions=[{'name':'src/buildtools','url':None,'deps_file':'DEPS','custom_vars':{'process_deps':True},'managed':False}]"
     - name: Run Lint
       shell: bash
       run: |

filename: script/lib/util.py
patch: @@ -4,6 +4,7 @@
 import errno
 import json
 import os
+import platform
 import shutil
 import subprocess
 import sys
@@ -184,14 +185,20 @@ def get_electron_exec():
 
 def get_buildtools_executable(name):
   buildtools = os.path.realpath(os.path.join(ELECTRON_DIR, '..', 'buildtools'))
-  chromium_platform = {
-    'darwin': 'mac',
-    'linux': 'linux64',
-    'linux2': 'linux64',
-    'win32': 'win',
-    'cygwin': 'win',
-  }[sys.platform]
-  path = os.path.join(buildtools, chromium_platform, name)
+
+  if sys.platform == 'darwin':
+    chromium_platform = 'mac_arm64' if platform.machine() == 'arm64' else 'mac'
+  elif sys.platform in ['win32', 'cygwin']:
+    chromium_platform = 'win'
+  elif sys.platform in ['linux', 'linux2']:
+    chromium_platform = 'linux64'
+  else:
+    raise Exception(f"Unsupported platform: {sys.platform}")
+
+  if name == 'clang-format':
+    path = os.path.join(buildtools, chromium_platform, 'format', name)
+  else:
+    path = os.path.join(buildtools, chromium_platform, name)
   if sys.platform == 'win32':
     path += '.exe'
   return path
Repository: nodejs/node
Pull request: doc: add release key for aduh95

Refs: https://github.com/nodejs/Release/issues/999
<!--
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: README.md
patch: @@ -768,6 +768,8 @@ responding to new issues.
 
 Primary GPG keys for Node.js Releasers (some Releasers sign with subkeys):
 
+* **Antoine du Hamel** <<duhamelantoine1995@gmail.com>>
+  `C0D6248439F1D5604AAFFB4021D900FFDB233756`
 * **Beth Griggs** <<bethanyngriggs@gmail.com>>
   `4ED778F539E3634C779C87C6D7062848A1AB005C`
 * **Bryan English** <<bryan@bryanenglish.com>>
@@ -793,6 +795,7 @@ To import the full set of trusted release keys (including subkeys possibly used
 to sign releases):
 
 ```bash
+gpg --keyserver hkps://keys.openpgp.org --recv-keys C0D6248439F1D5604AAFFB4021D900FFDB233756 # Antoine du Hamel
 gpg --keyserver hkps://keys.openpgp.org --recv-keys 4ED778F539E3634C779C87C6D7062848A1AB005C # Beth Griggs
 gpg --keyserver hkps://keys.openpgp.org --recv-keys 141F07595B7B3FFE74309A937405533BE57C7D57 # Bryan English
 gpg --keyserver hkps://keys.openpgp.org --recv-keys 74F12602B6F1C4E913FAA37AD3A89613643B6201 # Danielle Adams
Repository: facebook/react
Pull request: act: Add test for bypassing queueMicrotask

Test for fix added in #21740

Changed files:
filename: packages/react-reconciler/src/__tests__/ReactIsomorphicAct-test.js
patch: @@ -0,0 +1,50 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @jest-environment node
+ */
+
+// sanity tests for act()
+
+let React;
+let ReactNoop;
+let act;
+let DiscreteEventPriority;
+
+describe('isomorphic act()', () => {
+  beforeEach(() => {
+    React = require('react');
+    ReactNoop = require('react-noop-renderer');
+    DiscreteEventPriority = require('react-reconciler/constants')
+      .DiscreteEventPriority;
+    act = React.unstable_act;
+  });
+
+  // @gate __DEV__
+  test('bypasses queueMicrotask', async () => {
+    const root = ReactNoop.createRoot();
+
+    // First test what happens without wrapping in act. This update would
+    // normally be queued in a microtask.
+    ReactNoop.unstable_runWithPriority(DiscreteEventPriority, () => {
+      root.render('A');
+    });
+    // Nothing has rendered yet
+    expect(root).toMatchRenderedOutput(null);
+    // Flush the microtasks by awaiting
+    await null;
+    expect(root).toMatchRenderedOutput('A');
+
+    // Now do the same thing but wrap the update with `act`. No
+    // `await` necessary.
+    act(() => {
+      ReactNoop.unstable_runWithPriority(DiscreteEventPriority, () => {
+        root.render('B');
+      });
+    });
+    expect(root).toMatchRenderedOutput('B');
+  });
+});
Repository: ggml-org/llama.cpp
Pull request: llama : support OLMo-2-0325-32B (#12376)

The K-norm dimensions were calculated wrong (assumed to be the same as Q-norm).

Fixes #12376

Changed files:
filename: src/llama-model.cpp
patch: @@ -1005,6 +1005,7 @@ void llama_model::load_hparams(llama_model_loader & ml) {
                     case 16: type = LLM_TYPE_1B; break;
                     case 32: type = LLM_TYPE_7B; break;
                     case 40: type = LLM_TYPE_13B; break;
+                    case 64: type = LLM_TYPE_32B; break;
                     default: type = LLM_TYPE_UNKNOWN;
                 }
             } break;
@@ -2726,6 +2727,8 @@ bool llama_model::load_tensors(llama_model_loader & ml) {
                 } break;
             case LLM_ARCH_OLMO2:
                 {
+                    const int64_t n_embd_head = n_embd / n_head;
+
                     tok_embd = create_tensor(tn(LLM_TENSOR_TOKEN_EMBD, "weight"), {n_embd, n_vocab}, 0);
 
                     // output
@@ -2740,7 +2743,7 @@ bool llama_model::load_tensors(llama_model_loader & ml) {
                         layer.wv = create_tensor(tn(LLM_TENSOR_ATTN_V,   "weight", i), {n_embd, n_embd_gqa}, 0);
                         layer.wo = create_tensor(tn(LLM_TENSOR_ATTN_OUT, "weight", i), {n_embd, n_embd}, 0);
                         layer.attn_q_norm = create_tensor(tn(LLM_TENSOR_ATTN_Q_NORM, "weight", i), {n_embd}, 0);
-                        layer.attn_k_norm = create_tensor(tn(LLM_TENSOR_ATTN_K_NORM, "weight", i), {n_embd}, 0);
+                        layer.attn_k_norm = create_tensor(tn(LLM_TENSOR_ATTN_K_NORM, "weight", i), {n_head_kv * n_embd_head}, 0);
                         layer.attn_post_norm = create_tensor(tn(LLM_TENSOR_ATTN_POST_NORM, "weight", i), {n_embd}, 0);
 
                         layer.ffn_gate = create_tensor(tn(LLM_TENSOR_FFN_GATE, "weight", i), {n_embd,   n_ff}, 0);
Repository: ggml-org/llama.cpp
Pull request: context : fix init of n_outputs

ref https://github.com/ggml-org/llama.cpp/pull/12181#issuecomment-2725851877

Some minor macrodata refinement of the context initialization to prevent having invalid `n_outputs` value after the worst-cache graph reserve.

Changed files:
filename: src/llama-context.cpp
patch: @@ -285,11 +285,15 @@ llama_context::llama_context(
 
     // reserve worst-case graph
     if (!hparams.vocab_only) {
-        uint32_t n_seqs = 1; // TODO: worst-case number of sequences
-        uint32_t n_tokens = std::min(cparams.n_ctx, cparams.n_ubatch);
+        const uint32_t n_seqs = 1; // TODO: worst-case number of sequences
+        const uint32_t n_tokens = std::min(cparams.n_ctx, cparams.n_ubatch);
 
         llama_token token = model.vocab.token_bos(); // not actually used by llama_build_graph, but required to choose between token and embedding inputs graph
 
+        // restore later
+        // TODO: something cleaner
+        const auto n_outputs_save = n_outputs;
+
         // max number of outputs
         n_outputs = n_tokens;
 
@@ -341,6 +345,8 @@ llama_context::llama_context(
             }
         }
 
+        n_outputs = n_outputs_save;
+
         for (size_t i = 0; i < backend_ptrs.size(); ++i) {
             ggml_backend_t             backend = backend_ptrs[i];
             ggml_backend_buffer_type_t buft    = backend_buft[i];
Repository: huggingface/transformers
Pull request: Remove `dist": "loadfile"` for `pytest` for CircleCI jobs

# What does this PR do?

This pytest option forces all tests  from a test file to be run in the same CPU process when using `-n X` (from `pytest-xdist` pluging) to distribute tests across multiple CPU processes.

However, this leads some worker(s) to run much longer if a test file have tests that are slower. Removing `dist": "loadfile"` allows those tests are also being distributed to multiple processes.

Results: 

tests_tokenization: `main` | 1 run on PR

- the longest runtime among 8 paralle CircleCI runners: 8m30s | 5m
- total runtime from all 8 paralle CircleCI runners: 43m | 27m
- total idle time from 8 parallel CircleCI runners: 25m40s | 12m

tests_torch: `main` | 2 runs on PR

- the longest runtime among 8 paralle CircleCI runners: 6m40s | (6m | 5m10s)
- total runtime from all 8 paralle CircleCI runners: 31m15s | (30m | 28m)
- total idle time from 8 parallel CircleCI runners: 22m 10s | (18m | 13m15s)

tests_non_model: `main` | 1 run on PR

- the longest runtime among 8 paralle CircleCI runners: 6m | 2m50
- total runtime from all 8 paralle CircleCI runners: 21m20 | 14m50
- total idle time from 8 parallel CircleCI runners: 26m 50s | 7m 55s

Changed files:
filename: .circleci/create_circleci_config.py
patch: @@ -30,7 +30,7 @@
     "RUN_PIPELINE_TESTS": False,
 }
 # Disable the use of {"s": None} as the output is way too long, causing the navigation on CircleCI impractical
-COMMON_PYTEST_OPTIONS = {"max-worker-restart": 0, "dist": "loadfile", "vvv": None, "rsfE":None}
+COMMON_PYTEST_OPTIONS = {"max-worker-restart": 0, "vvv": None, "rsfE":None}
 DEFAULT_DOCKER_IMAGE = [{"image": "cimg/python:3.8.12"}]
 
 # Strings that commonly appear in the output of flaky tests when they fail. These are used with `pytest-rerunfailures`
@@ -269,13 +269,15 @@ def job_name(self):
     docker_image=[{"image":"huggingface/transformers-examples-torch"}],
     # TODO @ArthurZucker remove this once docker is easier to build
     install_steps=["uv venv && uv pip install . && uv pip install -r examples/pytorch/_tests_requirements.txt"],
+    pytest_num_workers=4,
 )
 
 
 examples_tensorflow_job = CircleCIJob(
     "examples_tensorflow",
     additional_env={"OMP_NUM_THREADS": 8},
     docker_image=[{"image":"huggingface/transformers-examples-tf"}],
+    pytest_num_workers=2,
 )
Repository: ggml-org/llama.cpp
Pull request: llama-tts : add '-o' option

llama-tts : add '-o' option

* added -o option to specify an output file name

* llama-tts returns ENOENT in case of file write error

note : PR #12042 is closed as superseded with this one.


Changed files:
filename: common/arg.cpp
patch: @@ -1889,7 +1889,7 @@ common_params_context common_params_parser_init(common_params & params, llama_ex
         [](common_params & params, const std::string & value) {
             params.out_file = value;
         }
-    ).set_examples({LLAMA_EXAMPLE_IMATRIX, LLAMA_EXAMPLE_CVECTOR_GENERATOR, LLAMA_EXAMPLE_EXPORT_LORA}));
+    ).set_examples({LLAMA_EXAMPLE_IMATRIX, LLAMA_EXAMPLE_CVECTOR_GENERATOR, LLAMA_EXAMPLE_EXPORT_LORA, LLAMA_EXAMPLE_TTS}));
     add_opt(common_arg(
         {"-ofreq", "--output-frequency"}, "N",
         string_format("output the imatrix every N iterations (default: %d)", params.n_out_freq),

filename: examples/tts/tts.cpp
patch: @@ -87,11 +87,11 @@ struct wav_header {
     uint32_t data_size;
 };
 
-static void save_wav16(const std::string & fname, const std::vector<float> & data, int sample_rate) {
+static bool save_wav16(const std::string & fname, const std::vector<float> & data, int sample_rate) {
     std::ofstream file(fname, std::ios::binary);
     if (!file) {
-        LOG_ERR("%s: Failed to open file '%s' for writing", __func__, fname.c_str());
-        return;
+        LOG_ERR("%s: Failed to open file '%s' for writing.\n", __func__, fname.c_str());
+        return false;
     }
 
     wav_header header;
@@ -108,7 +108,7 @@ static void save_wav16(const std::string & fname, const std::vector<float> & dat
         file.write(reinterpret_cast<const char*>(&pcm_sample), sizeof(pcm_sample));
     }
 
-    file.close();
+    return file.good();
 }
 
 static void fill_hann_window(int length, bool periodic, float * output) {
@@ -536,6 +536,7 @@ static std::string audio_data_from_speaker(json speaker, const outetts_version t
 int main(int argc, char ** argv) {
     common_params params;
 
+    params.out_file = "output.wav";
     params.prompt = "";
 
     params.n_predict = 4096;
@@ -1060,8 +1061,6 @@ lovely<|t_0.56|>






<|14
     }
 #endif
 
-    const std::string fname = "output.wav";
-
     const int n_sr = 24000; // sampling rate
 
     // zero out first 0.25 seconds
@@ -1072,11 +1071,15 @@ lovely<|t_0.56|>






<|14
     LOG_INF("%s: time for spectral ops: %.3f ms\n", __func__, (ggml_time_us() - t_spec_start) / 1000.0f);
     LOG_INF("%s: total time:            %.3f ms\n", __func__, (ggml_time_us() - t_main_start) / 1000.0f);
 
-    save_wav16(fname, audio, n_sr);
+    int retval = 0;
 
-    LOG_INF("%s: audio written to file '%s'\n", __func__, fname.c_str());
+    if (save_wav16(params.out_file, audio, n_sr)) {
+        LOG_INF("%s: audio written to file '%s'\n", __func__, params.out_file.c_str());
+    } else {
+        retval = ENOENT;
+    }
 
     llama_backend_free();
 
-    return 0;
+    return retval;
 }
Repository: denoland/deno
Pull request: fix(publish): public api - trace parent classes & interfaces when following a method

* https://github.com/denoland/deno_graph/pull/465

Closes https://github.com/denoland/deno/issues/23658

Changed files:
filename: Cargo.lock
patch: @@ -1477,9 +1477,9 @@ dependencies = [
 
 [[package]]
 name = "deno_graph"
-version = "0.74.3"
+version = "0.74.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "330a02cf6b5fe78f8b7a4256dc6289d8d45fd2ec8ef7b1b2e8b4ffe1fff8284a"
+checksum = "d70d7b300e5a4863d817db6fdcd914b57ff5165b481b4288437fe63d48757765"
 dependencies = [
  "anyhow",
  "async-trait",

filename: cli/Cargo.toml
patch: @@ -69,7 +69,7 @@ deno_config = "=0.16.3"
 deno_core = { workspace = true, features = ["include_js_files_for_snapshotting"] }
 deno_doc = { version = "=0.129.0", features = ["html", "syntect"] }
 deno_emit = "=0.40.2"
-deno_graph = { version = "=0.74.3", features = ["tokio_executor"] }
+deno_graph = { version = "=0.74.4", features = ["tokio_executor"] }
 deno_lint = { version = "=0.58.4", features = ["docs"] }
 deno_lockfile.workspace = true
 deno_npm = "=0.18.0"
Repository: electron/electron
Pull request: build: fix clang format location helper (again)

Backport of #42911

See that PR for details.


Notes: none


Changed files:
filename: script/lib/util.py
patch: @@ -194,6 +194,9 @@ def get_buildtools_executable(name):
     chromium_platform = 'linux64'
   else:
     raise Exception(f"Unsupported platform: {sys.platform}")
+  
+  if name == 'clang-format':
+    chromium_platform += '-format'
 
   path = os.path.join(buildtools, chromium_platform, name)
   if sys.platform == 'win32':

filename: script/run-clang-format.py
patch: @@ -23,7 +23,7 @@
 import tempfile
 
 from functools import partial
-from lib.util import get_depot_tools_executable
+from lib.util import get_buildtools_executable
 
 DEFAULT_EXTENSIONS = 'c,h,C,H,cpp,hpp,cc,hh,c++,h++,cxx,hxx,mm'
 DEFAULT_CLANG_FORMAT_IGNORE = '.clang-format-ignore'
@@ -203,7 +203,7 @@ def main():
         '--clang-format-executable',
         metavar='EXECUTABLE',
         help='path to the clang-format executable',
-        default=get_depot_tools_executable('clang-format'))
+        default=get_buildtools_executable('clang-format'))
     parser.add_argument(
         '--extensions',
         help='comma-separated list of file extensions'