Re: kernel-package: some enhancements for all archs

[Manoj Srivastava <srivasta@datasync.com>]

Hi,

	I like the subarch script. Thanks. I also like your proposal
 for multiple .config files, which allows one to compile for several
 architectures from the same source tree (I guess). However, I have
 doscovered a flaw in the rules file you have:

______________________________________________________________________
# SILO can boot gzip'ed kernel; m68k has vmlinuz, just the kernel makefiles
# don't seem to realise it.
ifeq ($(strip $(architecture)),m68k)
        gzip -9vc vmlinux > debian/tmp-image/boot/vmlinuz-$(version)
        chmod 644 debian/tmp-image/boot/vmlinuz-$(version)
else
# ********************************* ERROR***********
ifeq ($(strip $(architecture)),powerpc)
        SUB=`awk '/machine/ { print $3 }' /proc/cpuinfo`
ifeq ($(strip $(SUB)),CHRP)
        cp arch/$(KERNEL_ARCH)/$(SUB)boot/$(kimage) \
                   debian/tmp-image/boot/vmlinuz-$(version)
else
        cp arch/$(KERNEL_ARCH)/boot/$(kimage) \
                   debian/tmp-image/boot/vmlinuz-$(version)
endif              
______________________________________________________________________

	See, if the architecture is not m68k, then the first case is
 taken. So, only architectures which say m68k go to the else part, so
 the powerpc branch is never taken. This does not seem right. Is this
 closer to what is correct?

______________________________________________________________________

# SILO can boot gzip'ed kernel; m68k has vmlinuz, just the kernel makefiles
# don't seem to realise it.
#ifneq ($(strip $(architecture)),m68k)
ifeq ($(strip $(filter m68k ppc powerpc,$(architecture))),)
        cp arch/$(architecture)/boot/$(kimage) \
                   debian/tmp-image/boot/vmlinuz-$(version)
        chmod 644 debian/tmp-image/boot/vmlinuz-$(version)
else
ifneq ($(strip $(filter ppc powerpc,$(architecture))),)
        SUB=`awk '/machine/ { print $3 }' /proc/cpuinfo`
ifeq ($(strip $(SUB)),CHRP)
        cp arch/$(KERNEL_ARCH)/$(SUB)boot/$(kimage) \
                   debian/tmp-image/boot/vmlinuz-$(version)
else
        cp arch/$(KERNEL_ARCH)/$(SUBARCH)/boot/$(kimage) \
                   debian/tmp-image/boot/vmlinuz-$(version)
        chmod 644 debian/tmp-image/boot/vmlinuz-$(version)
endif              
else
        gzip -9vc vmlinux > debian/tmp-image/boot/vmlinuz-$(version)
        chmod 644 debian/tmp-image/boot/vmlinuz-$(version)
endif
endif
______________________________________________________________________

	manoj

-- 
 "Americans like to talk about (or be told about) Democracy but, when
 put to the test, usually find it to be an 'inconvenience.'  We have
 opted instead for an authoritarian system *disguised* as a Democracy.
 We pay through the nose for an enormous joke-of-a-government, let it
 push us around, and then wonder how all those assholes got in there."
 Frank Zappa
Manoj Srivastava  <srivasta@acm.org> <http://www.datasync.com/%7Esrivasta/>
Key C7261095 fingerprint = CB D9 F4 12 68 07 E4 05  CC 2D 27 12 1D F5 E8 6E


--  
To UNSUBSCRIBE, email to debian-powerpc-request@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org