[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/5] Inline the effect of am/pre-inst-guile
From: |
Neil Jerram |
Subject: |
Re: [PATCH 4/5] Inline the effect of am/pre-inst-guile |
Date: |
Tue, 22 Mar 2011 22:08:45 +0000 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Jan Nieuwenhuizen <address@hidden> writes:
[in rearranged order...]
>> - "$(preinstguile)" -l "$(srcdir)/$(snarf_doc).scm" -c "
>> \
>> + "$(top_builddir_absolute)/meta/guile" -l "$(srcdir)/$(snarf_doc).scm"
>> \
>> + -c "
>> \
>
> How do you suggest this works during cross compiling?
Because meta/guile itself uses $GUILE_FOR_BUILD when $cross_compiling is
"yes".
> Ah, but in my cross build recipe, i have something like
>
> preinstguile=$GUILE_FOR_BUILD
Is that because you're cross-building in a way that doesn't set
$cross_compiling to "yes"?
I can understand if you are, because I think I was doing that when last
working on mingw building, and using Wine and binfmt to run the built
executables.
> I know this isn't nice, I think preinstguile should go
> and we should use $GUILE_FOR_BUILD throughout.
>
> It's just one variable, but one that you can override,
I think the right thing might be to ensure that $cross_compiling is set
to "yes" for your build, even if the default ./configure mechanisms
(which I presume are based on the --target option) don't set it. Could
you do a grep-find for "cross_compiling", and see if any of the things
that depend on [ $cross_compiling = yes ] would _not_ be appropriate for
your kind of build?
(I think this would have worked for the mingw build that I was doing.
The difference would have been that the non-installed intermediate
executables, like guile_filter_doc_snarfage, were built (using
CC_FOR_BUILD) and run host-natively, instead of being target-built and
then Wine-run.)
Regards,
Neil
- Some simple build simplification patches, Neil Jerram, 2011/03/21
- [PATCH 3/5] Make explicit that GUILE_FOR_BUILD is only used when cross-compiling, Neil Jerram, 2011/03/21
- [PATCH 2/5] GUILE_FOR_BUILD is only needed by meta/guile.in, not by Makefiles, Neil Jerram, 2011/03/21
- [PATCH 1/5] Remove unused definition of preinstguiletool, Neil Jerram, 2011/03/21
- [PATCH 4/5] Inline the effect of am/pre-inst-guile, Neil Jerram, 2011/03/21
- [PATCH 5/5] Remove statements about scripts/* that are no longer true, Neil Jerram, 2011/03/21
- Re: Some simple build simplification patches, Andy Wingo, 2011/03/25