lkml.org 
[lkml]   [2022]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] kbuild: Restore .version auto-increment behaviour for Debian/RPM packages
On Tue, 15 Nov 2022 03:36:05 +0000,
Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> On Tue, Nov 15, 2022 at 9:09 AM Masahiro Yamada <masahiroy@kernel.org> wrote:
> >
> > On Mon, Nov 14, 2022 at 1:02 AM Marc Zyngier <maz@kernel.org> wrote:
> > >
> > > diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian
> > > index 60a2a63a5e90..e5c983afddab 100755
> > > --- a/scripts/package/mkdebian
> > > +++ b/scripts/package/mkdebian
> > > @@ -90,7 +90,7 @@ if [ -n "$KDEB_PKGVERSION" ]; then
> > > packageversion=$KDEB_PKGVERSION
> > > revision=${packageversion##*-}
> > > else
> > > - revision=$(cat .version 2>/dev/null||echo 1)
> > > + revision=$(init/build-version)
> >
> >
> > This does not work for out-of-tree builds
> > because init/build-version is a check-in source file.
> >
> >
> >
> > For example, "make O=/tmp/foo bindeb-pkg" fails with:
> > .../linux/scripts/package/mkdebian: 93: init/build-version: not found
> >
> >
> > The correct code is:
> >
> >
> > revision=$($srctree/init/build-version)
> >
> >
> >
> >
> > > packageversion=$version-$revision
> > > fi
> > > sourcename=$KDEB_SOURCENAME
> > > diff --git a/scripts/package/mkspec b/scripts/package/mkspec
> > > index 70392fd2fd29..9cbd45f497ba 100755
> > > --- a/scripts/package/mkspec
> > > +++ b/scripts/package/mkspec
> > > @@ -42,7 +42,7 @@ sed -e '/^DEL/d' -e 's/^\t*//' <<EOF
> > > Name: kernel
> > > Summary: The Linux Kernel
> > > Version: $__KERNELRELEASE
> > > - Release: $(cat .version 2>/dev/null || echo 1)
> > > + Release: $(init/build-version)
> >
> >
> > Ditto.
> >
> > Release: $($srctree/init/build-version)
>
>
>
> No, mkspec needs no change because binrpm-pkg builds the kernel
> _before_ generating the spec file.
>
> If you increment the .version file here again, you would have
> the revision mismatch between the package and the actual vmlinuz.
>
>
>
> Please send the change for mkdebian only.

Yup, good point on both count. Patch incoming shortly.

Thanks,

M.

--
Without deviation from the norm, progress is not possible.

\
 
 \ /
  Last update: 2022-11-15 20:19    [W:0.111 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site