CMake style

[Daan De Meyer]

I agree that passing the version via the command line might be a bit
excessive. However, by moving the other defines out of the config.h, we can
only specify them for the targets that actually use them which feels like
an improvement to me. I'll keep the config file to store the version but
maybe it should be renamed to version.h to more accurately reflect its
contents now.

Regards,

Daan

On Wed, 24 Jul 2019 at 11:23, Alex Turbov <i.zaufi at gmail.com> wrote:

> Hi,
>
> On Wed, Jul 24, 2019 at 10:38 AM Daan De Meyer <daan.j.demeyer at gmail.com>
> wrote:
>
>> The KATE_VERSION variable was used in some source files but by passing
>> the version as a command line define via CMake allowed me to remove the
>> config.h file completely.
>>
>
>  The good practice IMO is completely opposite: move `#defines` from CLI to
> generated `*.h` file(s) as much as you can. As for me, it'll make the CLI
> shorter and readable by humans w/ less pain. But the main reason: various
> shells have different limitations on CI length...
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190724/8e3711b1/attachment.html>