[libcamera-devel] [PATCH] clang-format: Remove unsupported option

Laurent Pinchart laurent.pinchart at ideasonboard.com
Fri Aug 27 13:38:04 CEST 2021


Hi Kieran,

Thank you for the patch.

On Fri, Aug 27, 2021 at 12:01:09PM +0100, Kieran Bingham wrote:
> The CaseSensitive option requires clang-format 12 which is not widely
> available.
> 
> Remove the option, to fix clang-format uses until a solution which
> supports clang-format version specific configurations can be merged.

Out of curiosity, have you run into blocking issue when trying to
implementing this through a symlink, or is that still the plan ?

> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  .clang-format | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/.clang-format b/.clang-format
> index ff60b928affc..5b8857dae924 100644
> --- a/.clang-format
> +++ b/.clang-format
> @@ -77,7 +77,6 @@ IncludeCategories:
>    # Qt includes (match before C++ standard library)
>    - Regex:           '<Q([A-Za-z0-9\-_])+>'
>      Priority:        9
> -    CaseSensitive:   true
>    # Headers in <> with an extension. (+system libraries)
>    - Regex:           '<([A-Za-z0-9\-_])+\.h>'
>      Priority:        2

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list