[LTP] [PATCH] Add compile_flags.txt to gitignore file

[LTP] [PATCH] Add compile_flags.txt to gitignore file

[Andrea Cervesato]

From: Andrea Cervesato <andrea.cervesato@suse.com>

Add rule to ignore clangd LSP configuration from local repository.

Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
---
Clangd is extensively used by IDEs and generic LSP wrappers to setup
clangd configuration. This patch removes it from gitignore so we can
safely avoid it gets into the patch-set when it's sent.
---
 .gitignore | 1 +
 1 file changed, 1 insertion(+)

diff --git a/.gitignore b/.gitignore
index 24f4a4ea834e0b919e640e879fe5e5b5d7aec466..6cd409489f266c4805dcf4633957892b10ea032a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -60,3 +60,4 @@ patches/
 logfile.*
 
 /utils/benchmark/ebizzy-0.3/ebizzy
+/compile_flags.txt

---
base-commit: b874b6a264cf4f343c17d543ebe4ff91fbb89042
change-id: 20260325-compile_flags_gitignore-c12d8f65ef2d

Best regards,
-- 
Andrea Cervesato <andrea.cervesato@suse.com>


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH] Add compile_flags.txt to gitignore file

[Petr Vorel]

Hi Andrea,

> Add rule to ignore clangd LSP configuration from local repository.

Acked-by: Petr Vorel <pvorel@suse.cz>

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH] Add compile_flags.txt to gitignore file

[Jan Polensky]

> From: Andrea Cervesato <andrea.cervesato@suse.com>
>
> Add rule to ignore clangd LSP configuration from local repository.
>
> Signed-off-by: Andrea Cervesato <andrea.cervesato@suse.com>
> ---
> Clangd is extensively used by IDEs and generic LSP wrappers to setup
> clangd configuration. This patch removes it from gitignore so we can
> safely avoid it gets into the patch-set when it's sent.
> ---
>  .gitignore | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/.gitignore b/.gitignore
> index 24f4a4ea834e0b919e640e879fe5e5b5d7aec466..6cd409489f266c4805dcf4633957892b10ea032a 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -60,3 +60,4 @@ patches/
>  logfile.*
>
>  /utils/benchmark/ebizzy-0.3/ebizzy
> +/compile_flags.txt
Hi Andrea,

thanks for the patch!

One thought: `compile_commands.json` (the compilation database) is also
commonly generated locally (e.g. with bear, compiledb), and not
necessarily in the repository root (often in a build directory). Since
the intent here is to prevent accidental inclusion in patch series, would
it make sense to ignore that filename as well?

In `.gitignore` we could do this without anchoring to the repo root, e.g.:
  compile_commands.json

Thanks & best regards,
Jan
>
> ---
> base-commit: b874b6a264cf4f343c17d543ebe4ff91fbb89042
> change-id: 20260325-compile_flags_gitignore-c12d8f65ef2d
>
> Best regards,
> --
> Andrea Cervesato <andrea.cervesato@suse.com>
>
>
> --
> Mailing list info: https://lists.linux.it/listinfo/ltp

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH] Add compile_flags.txt to gitignore file

[Andrea Cervesato via ltp]

Hi Jan,

> One thought: `compile_commands.json` (the compilation database) is also
> commonly generated locally (e.g. with bear, compiledb), and not
> necessarily in the repository root (often in a build directory). Since
> the intent here is to prevent accidental inclusion in patch series, would
> it make sense to ignore that filename as well?

I think it's a good idea, thanks. Will send v2.

--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp