[Pkg-openssl-devel] Bug#465248: long term mass bug filing for cross

[Neil Williams]

On Mon, 2008-02-11 at 22:31 +0100, Kurt Roeckx wrote:
> build support Reply-To: 
> In-Reply-To: <handler.s.C.1202735413773.transcript at bugs.debian.org>
> 
> tags 465248 - patchs
> thanks
> 
> Hi,
> 
> I think it's unlikely that the patch you provided actually works
> properly. 

It does - however I have limited it to the basic autotools-dev
requirements as there are other issues outstanding about how things like
test suites are disabled. Let's deal with these things one step at a
time - expecting a patch that solves everything in one bug is like
asking me to run before I can walk. Besides, the other changes needed
for cross-building in general and Emdebian specifically are likely to
change between now and implementation so there really is no point
implementing more than this autotools-dev support at this time.

The patch makes no difference to the Debian build - it is completely
unobtrusive in that sense - yet it will vastly simplify my cross
building workload. :-)

>  For instance the "make test" shouldn't be run when cross
> compiling since it expects to be able to run native code. 

I know, that is handled separately. Don't worry about it right now. Once
I have a few more things changed upstream, I will be able to start a
mass bug filing for the 'nocheck' DEB_BUILD_OPTION to achieve this. It
is not only when cross-compiling that 'make check' should be omitted.

This patch is exclusively for cross compiling support. Other changes are
specific to Emdebian or useful for crossbuilding but also useful at
other times, e.g. make test omission.

Feel free to wrap 'make test' in support for 'nocheck' passed via
DEB_BUILD_OPTIONS in a similar manner to noopt. Emdebian crossbuilding
tools set 'nocheck' during every build so if you do this, openssl will
be ahead of the game. :-)

>  There
> might be other things I'm not aware off that might cause problems.
> I can not apply this patch as is.

Please do apply it. I have tested with this patch and it suitable as-is.
Other issues can (and will) be handled separately but these are *much*
easier to patch into place than this autotools support. I need this
patch applied soon - the rest really can wait. Honest. After all, dpkg
doesn't support cross-building without assistance from packages like
dpkg-cross and emdebian-tools yet anyway and it seems unlikely that
dpkg-cross will be merged into dpkg before Lenny. There are bigger
issues here but if I can get this patch into packages like openssl, then
updating the crossbuilt packages becomes a lot easier and I can work on
the other problems in the build tools and package sets themselves.

Things like '$(MAKE) check' can be processed very easily with automation
scripts. It really is not a problem. Honest. (Yes, I know openssl uses
'make test' but that is trivial to script as a regular expression.
emdebian-tools already uses this method to drop dh_installman and other
unwanted operations that cannot be implemented in Debian at this time.
Yes, I am working on those too - 'nodocs' support is one option, dpkg
filtering another and dpkg variants is on the horizon (via
debian-xcontrol).)

See http://www.emdebian.org/bugs.php for links to the actual patches
used by Emdebian. Note that we drop things like documentation and
manpages that would cause problems with the "upstream" Debian package
which is why I have not submitted the complete Emdebian patch set.

http://buildd.emdebian.org/svn/browser/current/target/trunk/o/openssl/trunk/

http://buildd.emdebian.org/svn/browser/current/target/trunk/o/openssl/trunk/emdebian-rules.patch

This is an incremental process. If we wait until all the pieces are in
place nothing will ever progress. I need to reduce the work involved in
updating the cross built packages for Emdebian and this patch will make
a vast difference to my openssl workload.

That's why this is a long term mass bug filing for cross build support -
I expect the process to be continuing after Lenny is in old-stable.
Please don't wait that long to implement this patch. ;-)

-- 


Neil Williams
=============
http://www.data-freedom.org/
http://www.nosoftwarepatents.com/
http://www.linux.codehelp.co.uk/


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.alioth.debian.org/pipermail/pkg-openssl-devel/attachments/20080211/a89440ca/attachment.pgp