linux-kernel - Re: [PATCH] kbuild: create linux-headers package in deb-pkg

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4D2F044E.7060607@suse.cz>
Date:	Thu, 13 Jan 2011 14:55:26 +0100
From:	Michal Marek <mmarek@...e.cz>
To:	maximilian attems <max@...o.at>
Cc:	linux-kbuild@...r.kernel.org, Theodore Ts'o <tytso@....edu>,
	linux-kernel@...r.kernel.org,
	Ralf Rösch <ralf.roesch@...gmbh.de>
Subject: Re: [PATCH] kbuild: create linux-headers package in deb-pkg

On 10.1.2011 18:46, maximilian attems wrote:
> Seems review went unnoticed, due to using signature.

Oh, indeed, I thought that was just full-quote in your email, sorry.

> 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?

This should be changed to arch/$SRCARCH/include.

>>> +(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.

Yes, please remove it. BTW, here is a bugreport I got privately about
the very same issue:
On 12.1.2011 10:02, Ralf Rösch wrote:
| [...]
| Attached you will find a patch to complete the build successfully for my
| mipsel target.
| The dpkg --print-architecture shows i386 which results in error:
| dpkg-gencontrol: Fehler: aktuelle Host-Architektur »mipsel« erscheint
| nicht in der Architekturliste (i386) des Pakets
| (current host architecture 'mipsel' does not appear in package's list
i386)
| so I changed "arch" temporary to "any".
|
| It would be nice if it would be possible to make the kernel-headers
| installable on any host.
| They are needed very frequently for cross-compiling. (I already possible
| with dpkg force).

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