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@xxxxxxxxxx
_______________________________________________
Fedora-kernel-list mailing list
Fedora-kernel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-kernel-list
