>>>> Gunnar Evermann writes:
Gunnar> Andreas Jaeger <aj(a)suse.de> writes:
> I'm now checking in your patch as discussed (moved target
"all" to the
> front) - without the patches to libs/xemacs-devel/Makefile etc. which
> are just wrong.
Gunnar> I missed that discussion, but this snippet from the Makefile certainly
Gunnar> looks not quite right.
Gunnar> -------------
Gunnar> ifeq ($(BUILD_WITHOUT_MULE),)
Gunnar> SUBDIRS += mule
Gunnar> endif
Gunnar> all:
Gunnar> for dir in $(SUBDIRS); do \
Gunnar> $(MAKE) $(MFLAGS) -C $${dir} autoloads; \
Gunnar> done
Gunnar> for dir in $(SUBDIRS); do \
Gunnar> $(MAKE) $(MFLAGS) -C $${dir} bytecompile; \
Gunnar> done
Gunnar> include Local.rules.mk
Gunnar> -include Local.rules
Gunnar> --------------
Gunnar> for example isn't Local.rules the place where one is supposed to set
Gunnar> BUILD_WITHOUT_MULE ?
Your argument looks right. How do we fix this? Rename all to
all-dirs, move the includes before all-dirs and add
all-dirs: all
What do you think? I can make a patch later.
Andreas
--
Andreas Jaeger
SuSE Labs aj(a)suse.de
private aj(a)arthur.inka.de