Seems review went unnoticed, due to using signature. On Sat, 25 Dec 2010, maximilian attems wrote: > > +# Build header package > > +find . -name Makefile -o -name Kconfig\* -o -name \*.pl > /tmp/files$$ > > +find arch/x86/include include scripts -type f >> /tmp/files$$ > x86 centric? this will break on any non x86 arch, what is the correct variable for $ARCH? > > +(cd $objtree; find .config Module.symvers include scripts -type f >> /tmp/objfiles$$) > > +destdir=$hdrdir/usr/src/linux-headers-$version > > +mkdir -p "$destdir" "$hdrdir/DEBIAN" "$hdrdir/usr/share/doc/$hdrpackagename" > could move up to the other mkdir calls. we did that already. > > +tar -c -f - -T /tmp/files$$ | (cd $destdir; tar -xf -) > > +(cd $objtree; tar -c -f - -T /tmp/objfiles$$) | (cd $destdir; tar -xf -) > > +rm -f /tmp/files$$ /tmp/objfiles$$ > > +arch=$(dpkg --print-architecture) > > + > > +cat <<EOF >> debian/control > > + > > +Package: $hdrpackagename > > +Provides: linux-headers, linux-headers-2.6 > > +Architecture: $arch > not sure if that will interfere with the new crosscompile feature > of deb-pkg in linux-next. aboves arch call seems redundant with what we already have. > > +Description: Linux kernel headers for $KERNELRELEASE on $arch > > + This package provides kernel header files for $KERNELRELEASE on $arch > > + . > > + This is useful for people who need to build external modules > > +EOF > > + > > +create_package "$hdrpackagename" "$hdrdir" > > + > > # Do we have firmware? Move it out of the way and build it into a package. > > if [ -e "$tmpdir/lib/firmware" ]; then > > mv "$tmpdir/lib/firmware" "$fwdir/lib/" -- maks -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html