linux-kernel - Re: [PATCH] kbuild/mkspec: remove firmware from rpm package

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1542341479.7952083.1505889541234.JavaMail.zimbra@redhat.com>
Date:   Wed, 20 Sep 2017 02:39:01 -0400 (EDT)
From:   Shu Wang <shuwang@...hat.com>
To:     Greg KH <gregkh@...uxfoundation.org>
Cc:     yamada masahiro <yamada.masahiro@...ionext.com>, mmarek@...e.com,
        linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org,
        chuhu@...hat.com, yizhan@...hat.com
Subject: Re: [PATCH] kbuild/mkspec: remove firmware from rpm package

Hi

Do you mean the patch for deb package? The patch didn't fix rpm
package. or there's another patch I didn't get on the mailing list.

https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1492398.html

Thanks

----- Original Message -----
> From: "Greg KH" <gregkh@...uxfoundation.org>
> To: shuwang@...hat.com
> Cc: "yamada masahiro" <yamada.masahiro@...ionext.com>, mmarek@...e.com, linux-kbuild@...r.kernel.org,
> linux-kernel@...r.kernel.org, chuhu@...hat.com, yizhan@...hat.com
> Sent: Wednesday, September 20, 2017 2:10:12 PM
> Subject: Re: [PATCH] kbuild/mkspec: remove firmware from rpm package
> 
> On Wed, Sep 20, 2017 at 10:35:29AM +0800, shuwang@...hat.com wrote:
> > From: Shu Wang <shuwang@...hat.com>
> > 
> > Commit 5620a0d1aacd ("firmware: delete in-kernel firmware") deleted
> > in-kernel firmware support, including the firmware install command.
> > 
> > So make binrpm-pkg will failed as no firmware_install make target.
> > 
> > Signed-off-by: Shu Wang <shuwang@...hat.com>
> > ---
> >  scripts/package/mkspec | 4 ----
> >  1 file changed, 4 deletions(-)
> > 
> > diff --git a/scripts/package/mkspec b/scripts/package/mkspec
> > index bb43f153fd8e..8f34e31d8474 100755
> > --- a/scripts/package/mkspec
> > +++ b/scripts/package/mkspec
> > @@ -88,11 +88,8 @@ echo 'mkdir -p $RPM_BUILD_ROOT/boot/efi
> > $RPM_BUILD_ROOT/lib/modules'
> >  echo "%else"
> >  echo 'mkdir -p $RPM_BUILD_ROOT/boot $RPM_BUILD_ROOT/lib/modules'
> >  echo "%endif"
> > -echo 'mkdir -p $RPM_BUILD_ROOT'"/lib/firmware/$KERNELRELEASE"
> >  
> >  echo 'INSTALL_MOD_PATH=$RPM_BUILD_ROOT make %{?_smp_mflags} KBUILD_SRC=
> >  mod-fw= modules_install'
> > -echo 'INSTALL_FW_PATH=$RPM_BUILD_ROOT'"/lib/firmware/$KERNELRELEASE"
> > -echo 'make INSTALL_FW_PATH=$INSTALL_FW_PATH' firmware_install
> >  echo "%ifarch ia64"
> >  echo 'cp $KBUILD_IMAGE $RPM_BUILD_ROOT'"/boot/efi/vmlinuz-$KERNELRELEASE"
> >  echo 'ln -s '"efi/vmlinuz-$KERNELRELEASE" '$RPM_BUILD_ROOT'"/boot/"
> > @@ -154,7 +151,6 @@ echo '%defattr (-, root, root)'
> >  echo "/lib/modules/$KERNELRELEASE"
> >  echo "%exclude /lib/modules/$KERNELRELEASE/build"
> >  echo "%exclude /lib/modules/$KERNELRELEASE/source"
> > -echo "/lib/firmware/$KERNELRELEASE"
> >  echo "/boot/*"
> >  echo ""
> >  echo "%files headers"
> 
> I think this has already been posted to the list, right?
> 
> Sorry about it, it's pretty obvious that no build system tests kernel
> packages, and neither do I :)
> 
> thanks,
> 
> greg k-h
>