Thorsten Leemhuis
2008-Mar-26 07:01 UTC
Re: rpms/kernel/devel kernel.spec, 1.537, 1.538 linux-2.6-firewire-git-pending.patch, 1.22, 1.23
One comment: On 25.03.2008 21:43, Jarod Wilson (jwilson) wrote:> Author: jwilson > > Update of /cvs/pkgs/rpms/kernel/devel > In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv22927 > > Modified Files: > kernel.spec linux-2.6-firewire-git-pending.patch > Log Message: > * Tue Mar 25 2008 Jarod Wilson <jwilson@redhat.com> > - Put %{_target_cpu} into kernel uname, and tack it onto assorted > files and directories. Makes it possible to do parallel installs > of say both i686 and x86_64 kernels of the same version on x86_64 > x86_64 hardware (#197065). > [...] > Index: kernel.spec > ==================================================================> RCS file: /cvs/pkgs/rpms/kernel/devel/kernel.spec,v > retrieving revision 1.537 > retrieving revision 1.538 > diff -u -r1.537 -r1.538 > --- kernel.spec 25 Mar 2008 18:42:30 -0000 1.537 > +++ kernel.spec 25 Mar 2008 20:42:18 -0000 1.538 > [...] > @@ -1243,11 +1243,11 @@ > # Pick the right config file for the kernel we''re building > if [ -n "$Flavour" ] ; then > Config=kernel-%{version}-%{_target_cpu}-$Flavour.config > - DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour-%{_target_cpu} > - DevelLink=/usr/src/kernels/%{KVERREL}$Flavour-%{_target_cpu} > + DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour > + DevelLink=/usr/src/kernels/%{KVERREL}$FlavourJust wondering: If we are working in this area why not finally chose one of the two variants ("%{KVERREL}-$Flavour" or "%{KVERREL}$Flavour") and stick to it everywhere as well? Or even use a "." as delimiter here as well? Then the duplication due to the "if [ -n "$Flavour" ]" could be removed in this area of the spec file by using something like this everywhere: DevelDir=/usr/src/kernels/%{KVERREL}${Flavour:+-${Flavour}} > [...] CU knurd
Jarod Wilson
2008-Mar-26 13:03 UTC
Re: rpms/kernel/devel kernel.spec, 1.537, 1.538 linux-2.6-firewire-git-pending.patch, 1.22, 1.23
On Wednesday 26 March 2008 03:01:29 am Thorsten Leemhuis wrote:> One comment: > > On 25.03.2008 21:43, Jarod Wilson (jwilson) wrote: > > Author: jwilson > > > > Update of /cvs/pkgs/rpms/kernel/devel > > In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv22927 > > > > Modified Files: > > kernel.spec linux-2.6-firewire-git-pending.patch > > Log Message: > > * Tue Mar 25 2008 Jarod Wilson <jwilson@redhat.com> > > - Put %{_target_cpu} into kernel uname, and tack it onto assorted > > files and directories. Makes it possible to do parallel installs > > of say both i686 and x86_64 kernels of the same version on x86_64 > > x86_64 hardware (#197065). > > [...] > > Index: kernel.spec > > ==================================================================> > RCS file: /cvs/pkgs/rpms/kernel/devel/kernel.spec,v > > retrieving revision 1.537 > > retrieving revision 1.538 > > diff -u -r1.537 -r1.538 > > --- kernel.spec 25 Mar 2008 18:42:30 -0000 1.537 > > +++ kernel.spec 25 Mar 2008 20:42:18 -0000 1.538 > > [...] > > @@ -1243,11 +1243,11 @@ > > # Pick the right config file for the kernel we''re building > > if [ -n "$Flavour" ] ; then > > Config=kernel-%{version}-%{_target_cpu}-$Flavour.config > > - DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour-%{_target_cpu} > > - DevelLink=/usr/src/kernels/%{KVERREL}$Flavour-%{_target_cpu} > > + DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour > > + DevelLink=/usr/src/kernels/%{KVERREL}$Flavour > > Just wondering: If we are working in this area why not finally chose one > of the two variants ("%{KVERREL}-$Flavour" or "%{KVERREL}$Flavour") and > stick to it everywhere as well? Or even use a "." as delimiter here as > well? > > Then the duplication due to the "if [ -n "$Flavour" ]" could be removed > in this area of the spec file by using something like this everywhere: > > DevelDir=/usr/src/kernels/%{KVERREL}${Flavour:+-${Flavour}}I thought about trying to yank that extra symlink and the if clause, but wasn''t sure exactly what they were there for, so I didn''t want to cause any further breakage... But yeah, I''d be all for simplifying there, if its not going to cause someone/something headaches. To be consistent with the rest of the paths laid down, it really should be %{KVERREL}${Flavour} without any "-" or "." (unless we also change all the other paths to also use "-" or ".".) -- Jarod Wilson jwilson@redhat.com
Thorsten Leemhuis
2008-Mar-26 13:19 UTC
Re: rpms/kernel/devel kernel.spec, 1.537, 1.538 linux-2.6-firewire-git-pending.patch, 1.22, 1.23
On 26.03.2008 14:03, Jarod Wilson wrote:> On Wednesday 26 March 2008 03:01:29 am Thorsten Leemhuis wrote: >> One comment: >> >> On 25.03.2008 21:43, Jarod Wilson (jwilson) wrote: >>> Author: jwilson >>> >>> Update of /cvs/pkgs/rpms/kernel/devel >>> In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv22927 >>> >>> Modified Files: >>> kernel.spec linux-2.6-firewire-git-pending.patch >>> Log Message: >>> * Tue Mar 25 2008 Jarod Wilson <jwilson@redhat.com> >>> - Put %{_target_cpu} into kernel uname, and tack it onto assorted >>> files and directories. Makes it possible to do parallel installs >>> of say both i686 and x86_64 kernels of the same version on x86_64 >>> x86_64 hardware (#197065). >>> [...] >>> Index: kernel.spec >>> ==================================================================>>> RCS file: /cvs/pkgs/rpms/kernel/devel/kernel.spec,v >>> retrieving revision 1.537 >>> retrieving revision 1.538 >>> diff -u -r1.537 -r1.538 >>> --- kernel.spec 25 Mar 2008 18:42:30 -0000 1.537 >>> +++ kernel.spec 25 Mar 2008 20:42:18 -0000 1.538 >>> [...] >>> @@ -1243,11 +1243,11 @@ >>> # Pick the right config file for the kernel we''re building >>> if [ -n "$Flavour" ] ; then >>> Config=kernel-%{version}-%{_target_cpu}-$Flavour.config >>> - DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour-%{_target_cpu} >>> - DevelLink=/usr/src/kernels/%{KVERREL}$Flavour-%{_target_cpu} >>> + DevelDir=/usr/src/kernels/%{KVERREL}-$Flavour >>> + DevelLink=/usr/src/kernels/%{KVERREL}$Flavour >> Just wondering: If we are working in this area why not finally chose one >> of the two variants ("%{KVERREL}-$Flavour" or "%{KVERREL}$Flavour") and >> stick to it everywhere as well? Or even use a "." as delimiter here as >> well? >> Then the duplication due to the "if [ -n "$Flavour" ]" could be removed >> in this area of the spec file by using something like this everywhere: >> DevelDir=/usr/src/kernels/%{KVERREL}${Flavour:+-${Flavour}} > I thought about trying to yank that extra symlink and the if clause, but > wasn''t sure exactly what they were there for, so I didn''t want to cause any > further breakage...Understood.> But yeah, I''d be all for simplifying there, if its not > going to cause someone/something headaches.+1 -- let''s remove it and see what breaks; we can easily re-add later if it''s needed somewhere> To be consistent with the rest of > the paths laid down, it really should be %{KVERREL}${Flavour} without any "-" > or "."+1> (unless we also change all the other paths to also use "-" or ".".)Hmmm. This would look cleaner IMHO. But it likely would create even more trouble for kmods and other kernel module stuff. But as there are adjustments needed already due to the other changes it might be a good idea to do this change now as well. But I''m not really sure if that''s really worth the effort. CU knurd
On Wednesday 26 March 2008 09:19:56 am Thorsten Leemhuis wrote: [...]> >> Just wondering: If we are working in this area why not finally chose one > >> of the two variants ("%{KVERREL}-$Flavour" or "%{KVERREL}$Flavour") and > >> stick to it everywhere as well? Or even use a "." as delimiter here as > >> well? > >> Then the duplication due to the "if [ -n "$Flavour" ]" could be removed > >> in this area of the spec file by using something like this everywhere: > >> DevelDir=/usr/src/kernels/%{KVERREL}${Flavour:+-${Flavour}} > > > > I thought about trying to yank that extra symlink and the if clause, but > > wasn''t sure exactly what they were there for, so I didn''t want to cause > > any further breakage... > > Understood. > > > But yeah, I''d be all for simplifying there, if its not > > going to cause someone/something headaches. > > +1 -- let''s remove it and see what breaks; we can easily re-add later if > it''s needed somewhere > > > To be consistent with the rest of > > the paths laid down, it really should be %{KVERREL}${Flavour} without any > > "-" or "." > > +1 > > > (unless we also change all the other paths to also use "-" or ".".) > > Hmmm. This would look cleaner IMHO. But it likely would create even more > trouble for kmods and other kernel module stuff. But as there are > adjustments needed already due to the other changes it might be a good > idea to do this change now as well. > > But I''m not really sure if that''s really worth the effort.I''m trying out a local build now that eliminates the DevelLink and uses your suggestion above for DevelDir, and makes use of %{KVERREL}-%{Flavour} right now, which as discussed on irc, looks more correct to both of us. :) i.e., this changes uname -r for kernel-PAE from 2.6.25-1.fc9.i686PAE to 2.6.25-1.fc9.i686-PAE, and all paths laid down are changed accordingly. -- Jarod Wilson jwilson@redhat.com