Hi Chris.

Thanks for working on this.
 
quoted text
> > > +if [ "$srctree" != "$objtree" ] ; then
> > > +    cp --parents $(find -type f -name "Makefile*" -o -name "Kconfig*" -not -ipath "$objtree/*Makefile" ) ${tgtdir}
> >                                                                                                ^
> > Why this wildcard??? (objtree/>*<Makefile)
> > Seems to be a typing error.
> 
> no, it's so we catch things like
> 
> linux/build/Makefile
> 
>  *and*
> 
> linux/build/foo/bar/Makefile
Obviously - I missed the find when I looked.
Thanks for the explanation. And no comments needed in the script -
I should just have read the full command line.

quoted text
> 
> > > +#rm -rf ${tgtdir}/Documentation
> > Remove this line since it is commented out
> 
> right, actually, it can be uncommented but the 'make help' fails; i'm
> not sure if we need make help to work since most of the other targets
> won't anyhow
> 
> so should i remove it (leaving 'make help' as usable) or remove it?
Un-comment it so we do not break make help.

quoted text
> > Something less hardcoded are preferred. Maybe like:
> > cp -a `ls | grep -v ^asm` asm-generic $(tgtdir}/include
> 
> ok (though i'm not a big fan of ls | grep as a rule, it tends to be
> fragile when people do dumb things)

find . -maxdepth 1 -! -name 'asm*'
seems to do the trick and it is better than ls.

quoted text
> 
>     sed -n "s/^ALTARCH[[:space:]]:=[[:space:]]\(.*$\)\+//p"
> 
> (i'm sure there is a better way though)

Oleg (added to mail) - you are quite confident in sed et al.
Let us know if there is a simpler way.

(We want to pick i386 from include/asm-x86_64/Kbuild
and similar for the architectures that uses ARCHDEF).

	Sam
-
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/