On Mon, 2008-02-11 at 22:31 +0100, Kurt Roeckx wrote: > build support Reply-To: > In-Reply-To: <[EMAIL PROTECTED]> > > 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/
signature.asc
Description: This is a digitally signed message part
