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

Re: [PATCH] nearly successful powerpc build



I assume you're talking woody here?  I've been working on potato boot
floppies for the past week steadily and not had time to look at the
woody set.

On Sat, Apr 14, 2001 at 01:12:55AM -0400, james@netcsi.com wrote:
> I got a build that ran all the way up to release.sh before failing
> to install some documentation.  Here are the relevant patches.  I'm
> not too sure about the second one, because I don't understand what
> '$system' is supposed to be for (it's currently always passed in as
> null from the higher-level scripts, and the value the lower-level
> scripts want is in the subarch, i took a wild guess which to fix).


> powerpc-prep-apus.diff
>     - Added powerpc/rules, to deal with prep and apus oddities.
>     - Modernized bootprep.sh.
>     - config: Changed default powerpc kernel version to 2.2.17.

Why on earth would you do that???  It should have been
2.2.18presomething in the archive; the 2.2.18pre was there for a
reason.

It should also just be changed to 2.2.19 now.  They're in incoming.


> powerpc-hacks.diff
>     - rescue.sh: Switch on $subarch, not $system for powerpc.
>     - rootdisk.sh: Added 'charset=latin1' (HACK--where should
>       this come from?)
> 
> -- 
> James Deikun, Techie(tm), CSI Multimedia
> The opinions expressed &c.



>  # Patch kernel source - FIXME: allow for other patches
>  cd kernel-source-$kver
> -zcat $builddir/usr/src/kernel-patches/powerpc/*.diff.gz \
> - | patch -l -s -p1
> +# zcat $builddir/usr/src/kernel-patches/powerpc/*.diff.gz \
> +#  | patch -l -s -p1

Deleting lines of code is usually a bad idea.

In this case, a very bad idea.  This comes from
kernel-patch-<ver>-powerpc; I assume it was part of your attempt to go
to 2.2.17?

>  ifeq "$(architecture)" "powerpc"
> -    kver	:= 2.2.18
> -    pcmcia_kver := 2.2.18-pmac
> +    kver	:= 2.2.17
> +    pcmcia_kver := 2.2.17-pmac
>      apuskver	:= 2.2.10
>  endif

Like I said.  Why on earth do you want to do a downgrade?

>  ifeq "$(architecture)" "sparc"
> @@ -57,7 +57,7 @@
>  # Top-level directory where we will look for external stuff required
>  # to build the boot-floppies.  It has to be an absolute PATH or else
>  # the basedisks.sh script will fail.
> -ftp_archive	:= /archive/debian
> +ftp_archive	:= /space/james/archive/debian

Careful of including this with submitted patches!  If you feel like
adding a config.local, more power to you... please...

-- 
Daniel Jacobowitz                           Debian GNU/Linux Developer
Monta Vista Software                              Debian Security Team
                         "I am croutons!"



Reply to: