Re: [PATCH] docs: fix check-docs with WITH_BREAKING_CHANGES

From: Phillip Wood <phillip.wood123@gmail.com>
Date: Mar 9, 2025, 10:52 UTC
Message-ID: <b5fb3292-216a-4456-b456-e9ed38affc22@gmail.com>
In-Reply-To: <56cf842a-7c1f-4354-b191-35bcc1e139bd@gmail.com>

On 07/03/2025 15:07, Phillip Wood wrote:

Show 7 quoted lines

> On 07/03/2025 10:32, Phillip Wood wrote:
> 
> The diff below stops us from building pack-redundant with
> -Dbreaking_changes=true but still builds the documentation. I don't intend
> spending any more time one this
> 
> [...]

>   if get_option('breaking_changes')
>     build_options_config.set('WITH_BREAKING_CHANGES', 'YesPlease')
> +  add_project_arguments('-DWITH_BREAKING_CHANGES=YesPlease', language : 
> 'c')

Looking again at this I think it should probably be

     libgit_c_args += '-DWITH_BREAKING_CHANGES=YesPlease'

to match the rest of our meson.build. As a newcomer to meson I find it confusing that the CFLAGS for the build targets are set implicitly by their libgit dependency.

Best Wishes

Phillip