[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: |
Jan Nieuwenhuizen |
|
Subject: |
Re: [PATCH 4/5] Inline the effect of am/pre-inst-guile |
|
Date: |
Tue, 22 Mar 2011 21:46:25 +0100 |
Neil Jerram schreef op ma 21-03-2011 om 22:19 [+0000]:
> It's just one variable definition, and in my opinion it confuses,
> rather than helps, the overall build picture to have two names
> (preinstguile and meta/guile) for the same thing.
Ah, but in my cross build recipe, i have something like
preinstguile=$GUILE_FOR_BUILD
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,
> - "$(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?
Greetings,
Jan.
--
Jan Nieuwenhuizen <address@hidden> | GNU LilyPond http://lilypond.org
Freelance IT http://JoyofSource.com | Avatar® http://AvatarAcademy.nl
- 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
- Re: [PATCH 4/5] Inline the effect of am/pre-inst-guile,
Jan Nieuwenhuizen <=
- [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