shared /boot support. bz 197065

[Jarod Wilson]

On Monday 24 March 2008 03:32:37 pm Dave Jones wrote:
> I took a stab at bz 197065 and arrived at the patch below.
> Would appreciate some eyeballs before I commit from people
> familiar with the macro goo in the specfile. (Hi Roland!)
>
> Aparently pm-utils will need a change to cope with the changed
> filename, but I think that should be the limit of the damage.
> (oprofile will need to append the archname on the end of System.map-$ver
> filenames, but they're user-passed anyway, and not coded anywhere afaik
> Hmm. Not sure about Systemtap).
>
> Comments?

Yep!

> -    install -m 644 .config $RPM_BUILD_ROOT/boot/config-$KernelVer
> -    install -m 644 System.map $RPM_BUILD_ROOT/boot/System.map-$KernelVer
> -    touch $RPM_BUILD_ROOT/boot/initrd-$KernelVer.img
> +    install -m 644 .config $RPM_BUILD_ROOT/boot/config-$KernelVer.%{_arch}
> +    install -m 644 System.map $RPM_BUILD_ROOT/boot/System.map-$KernelVer.%{_arch}
> +    touch $RPM_BUILD_ROOT/boot/initrd-$KernelVer.img.%{_arch}

For the sake of consistency, I'd try to keep $KernelVer and %{_arch} together. The above would give 
us for example config-2.6.25-1.fc9.x86_64, System.map-2.6.25-1.fc9.x86_64 and 
initrd-2.6.25-1.fc9.img.x86_64. I'd make the initrd either initrd-2.6.25-1.fc9.x86_64.img or 
initrd.img-2.6.25-1.fc9.x86_64.


> -/sbin/new-kernel-pkg --package kernel%{?-v:-%{-v*}} --rpmposttrans %{?1} %{KVERREL}%{?-v*} || exit 
$?\
> +/sbin/new-kernel-pkg --package kernel%{?-v:-%{-v*}} --rpmposttrans %{?1} %{KVERREL}%{?-v*}.
%{_arch} || exit $?\ %{nil}

I suspect this will make new-kernel-package very unhappy, since I think its expecting to be fed what 
essentially amounts to $(uname -r).


-- 
Jarod Wilson
jwilson at redhat.com