avoid creating symlink to clang format config

resolves #14 Change-Id: I1ac2d54dee77c9d9ecd93bdd3b38a6b0856caf0f
oliverlee · Jun 28, 2024 · 7764e9d · 7764e9d
1 parent bf274e2
commit 7764e9d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -134,7 +134,7 @@ build:clang-format-base --@bazel_clang_format//:ignore=//:clang-format-ignore
 ## Requirements
 
 - Bazel ???
-- clang-format ???
+- ClangFormat 14
 
 I'm not sure what the minimum versions are but please let me know if you are
 using a version that doesn't work.
diff --git a/defs.bzl b/defs.bzl
@@ -25,13 +25,11 @@ def _do_format(ctx, f, format_options, execution_reqs):
         command = """
 set -euo pipefail
 
-test -e .clang-format || ln -s -f {config} .clang-format
-
 # https://github.com/llvm/llvm-project/issues/46336
 # although newer versions of clang-format (e.g. 18.1.4) *do* appear to work
 # with symlinks
 #
-{binary} {format_options} $(readlink --canonicalize {infile})
+{binary} -style=file:{config} {format_options} $(readlink --canonicalize {infile})
 
 touch {outfile}
 """.format(