Re: [PATCH v3 2/2] kbuild: rpm-pkg: introduce a simple changelog section for kernel.spec

[Masahiro Yamada]

On Sun, Jul 7, 2024 at 1:45 AM Rafael Aquini <aquini@xxxxxxxxxx> wrote:
>
> Fix the following rpmbuild warning:
>
>   $ make srcrpm-pkg
>   ...
>   RPM build warnings:
>       source_date_epoch_from_changelog set but %changelog is missing
>
> Signed-off-by: Rafael Aquini <aquini@xxxxxxxxxx>
> ---
>  scripts/package/mkspec | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>
> diff --git a/scripts/package/mkspec b/scripts/package/mkspec
> index ce201bfa8377..e45fdb12fbc7 100755
> --- a/scripts/package/mkspec
> +++ b/scripts/package/mkspec
> @@ -28,3 +28,26 @@ cat<<EOF
>  EOF
>
>  cat "${srctree}/scripts/package/kernel.spec"
> +
> +# collect the user's name and email address for the changelog entry
> +if [ "$(command -v git)" ]; then
> +       name=$(git config user.name) || true
> +       email=$(git config user.email) || true
> +fi
> +
> +if [ ! "${name:+set}" ]; then
> +       name=${KBUILD_BUILD_USER:-$(id -nu)}
> +fi
> +
> +if [ ! "${email:+set}" ]; then
> +       buildhost=${KBUILD_BUILD_HOST:-$(hostname -f 2>/dev/null || hostname)}
> +       builduser=${KBUILD_BUILD_USER:-$(id -nu)}
> +       email="${builduser}@${buildhost}"
> +fi
> +
> +cat << EOF
> +
> +%changelog
> +* $(LC_ALL=C; date +'%a %b %d %Y') ${name} <${email}> - ${KERNELRELEASE}


I am not sure whether the version is required or not.

In the following guide, not all entries have the version.

  https://jfearn.fedorapeople.org/en-US/RPM/4/html/RPM_Guide/ch09s06.html


If you want to add the version, perhaps is it better to
follow the fedora convention?


The spec file of Fedora looks as follows.
The version is enclosed in the square brackets.

%changelog
* Wed Dec 13 2023 Augusto Caringi <acaringi@xxxxxxxxxx> [6.6.7-0]
- Add rhbz#2253632 rhbz#2253633 to BugsFixed (Justin M. Forbes)
- Turn on DRM_ACCEL drivers for Fedora (Justin M. Forbes)
- Linux v6.6.7




Or, is this not important because there is no strict format?



> +- Custom built Linux kernel.
> +EOF
> --
> 2.45.1
>
>


-- 
Best Regards
Masahiro Yamada