[PATCHv2] Fix copyright year
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Tue Oct 15 13:39:19 CEST 2024
Hi Bernhard,
Thank you for the patch.
On Tue, Oct 15, 2024 at 10:20:25AM +0200, libcamerabmw+ml at lsmod.de wrote:
> From: "Bernhard M. Wiedemann" <bwiedemann at suse.de>
>
> without this patch, openSUSE's debugsource package would vary over time,
> breaking reproducible builds.
> Also a copyright of 2025 is factually wrong,
> because nobody did anything copyright-worthy in that year.
> If updating the year manually is too hard (it is optional anyway),
> using the mtime of input file(s) could work as well.
>
> By moving the year to tracepoints.h.in it can easily be updated
> along with the content.
>
> The diff was:
> /usr/src/debug/libcamera-0.3.1/x86_64-suse-linux/include/libcamera/internal/tracepoints.h 2024-08-20 09:37:46.000000000 +0200
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: LGPL-2.1-or-later */
> /*
> - * Copyright (C) 2024, Google Inc.
> + * Copyright (C) 2025, Google Inc.
I'm a bit confused, where does this diff come from ?
>
> Signed-off-by: Bernhard M. Wiedemann <bwiedemann at suse.de>
> ---
> include/libcamera/internal/tracepoints.h.in | 2 +-
> utils/codegen/gen-tp-header.py | 4 +---
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/include/libcamera/internal/tracepoints.h.in b/include/libcamera/internal/tracepoints.h.in
> index f0962091..ff5c581b 100644
> --- a/include/libcamera/internal/tracepoints.h.in
> +++ b/include/libcamera/internal/tracepoints.h.in
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: LGPL-2.1-or-later */
> /*
> - * Copyright (C) {{year}}, Google Inc.
> + * Copyright (C) 2024, Google Inc.
This should be 2020, the file hasn't received any copyrightable change
since then.
> *
> * Tracepoints with lttng
> *
> diff --git a/utils/codegen/gen-tp-header.py b/utils/codegen/gen-tp-header.py
> index 83606c32..6769c7ce 100755
> --- a/utils/codegen/gen-tp-header.py
> +++ b/utils/codegen/gen-tp-header.py
> @@ -6,7 +6,6 @@
> #
> # Generate header file to contain lttng tracepoints
>
> -import datetime
> import jinja2
> import pathlib
> import os
> @@ -20,7 +19,6 @@ def main(argv):
> output = argv[2]
> template = argv[3]
>
> - year = datetime.datetime.now().year
> path = pathlib.Path(output).absolute().relative_to(argv[1])
>
> source = ''
> @@ -28,7 +26,7 @@ def main(argv):
> source += open(fname, 'r', encoding='utf-8').read() + '\n\n'
>
> template = jinja2.Template(open(template, 'r', encoding='utf-8').read())
> - string = template.render(year=year, path=path, source=source)
> + string = template.render(path=path, source=source)
Looks good.
>
> f = open(output, 'w', encoding='utf-8').write(string)
>
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list