From 3ec11a1089a3af7b52a338ee92d41bcd78b748f6 Mon Sep 17 00:00:00 2001 From: Henning Makholm Date: Thu, 13 Feb 2020 15:30:15 +0100 Subject: [PATCH] Don't chain to ./codeql in .codeqlmanifest.json This entry in `.codeqlmanifest.json` was intended to allow unpacking the CodeQL CLI as a subdirectory of `ql`, and things would Just Work. However, it is not necessary anymore because recent releases of the CLI will search their own directory as a fallback _independently_ of the parent directory. On the contrary, removing this link will make internal testing easier because you then run a test build of the CLI with `--search-path` pointing to the `ql` checkout without inadvertently making extractors in a _different_ build that is unpacked there visible. --- .codeqlmanifest.json | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.codeqlmanifest.json b/.codeqlmanifest.json index 421284bc4b9..020c9ce9244 100644 --- a/.codeqlmanifest.json +++ b/.codeqlmanifest.json @@ -2,5 +2,4 @@ "*/ql/test/qlpack.yml", "*/upgrades/qlpack.yml", "misc/legacy-support/*/qlpack.yml", - "misc/suite-helpers/qlpack.yml", - "codeql/.codeqlmanifest.json" ] } + "misc/suite-helpers/qlpack.yml" ] }