Re: [RFC PATCH v3] kbuild: fix error when building from src rpm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 6/5/2013 6:17 AM, Marciniszyn, Mike wrote:
> I sent a different version today that used %defines for each arch,

I didn't look through the original patch (especially since you said you
are sending a new one).  However, please keep in mind that we need to
either put firmware in a kernel version specific directory or split it
off to a separate patch.  Otherwise you can't install more than one
kernel package at a time.

> but this idea is better and doesn't create a maintenance issue.  The
> scope would be kbuild wide so I assume that the cc list needs to
> expand?
> 
> Also, I will also want to add a new package to create a kernel-devel
> package and will be submitting patch for that shortly.
> 
> Thanks, Mike
> 
> 
>> -----Original Message----- From: Michal Marek
>> [mailto:mmarek@xxxxxxx] Sent: Wednesday, June 05, 2013 5:54 AM To:
>> Marciniszyn, Mike Cc: linux-kbuild@xxxxxxxxxxxxxxx; Doug Ledford 
>> Subject: Re: [RFC PATCH v3] kbuild: fix error when building from
>> src rpm
>> 
>> On 4.6.2013 22:12, Marciniszyn, Mike wrote:
>>> Sorry for the confusion with the patch.
>>> 
>>> The latest version I've changed to an RFC because of some
>>> testing issues I've encountered.
>>> 
>>> The /sbin/installkernel doesn't behave very nicely when
>>> INSTALL_PATH is modified on RH6.4.
>> [snip]
>>> I'm seeing when spec file tries to do its own install...
>>> 
>>> I'm open to suggestions for making this part of the spec file
>>> portable and the src RPM usable.
>> 
>> You could introduce a new Makefile target, say image_name, that
>> would simply print the image name after including the arch
>> Makefile. Then in the spec file, you would simply do
>> 
>> KBUILD_IMAGE=$(make -s image_name)
>> 
>> I think that this target would be a useful addition to the already
>> existing kernelversion and kernelrelease targets. And BTW, we could
>> use make -s kernelrelease in the specfile as well, instead of
>> hardcoding the value there.
>> 
>> Michal


Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux